feat(api-gateway): OpenAPI - declare type field for Cube

packages/cubejs-api-gateway/openspec.yml

@@ -151,6 +151,7 @@ components:
       type: "object"
       required:
         - name
+        - type
         - measures
         - dimensions
         - segments
@@ -159,6 +160,8 @@ components:
           type: "string"
         title:
           type: "string"
+        type:
+          $ref: "#/components/schemas/V1CubeMetaType"
         description:
           type: "string"
         measures:
@@ -177,6 +180,11 @@ components:
           type: "array"
           items:
             $ref: "#/components/schemas/V1CubeMetaJoin"
+    V1CubeMetaType:
+      type: "string"
+      enum:
+        - "cube"
+        - "view"
     V1MetaResponse:
       type: "object"
       properties:

rust/cubesql/cubeclient/.openapi-generator/FILES

@@ -7,6 +7,7 @@ src/models/v1_cube_meta_dimension_granularity.rs
 src/models/v1_cube_meta_join.rs
 src/models/v1_cube_meta_measure.rs
 src/models/v1_cube_meta_segment.rs
+src/models/v1_cube_meta_type.rs
 src/models/v1_error.rs
 src/models/v1_load_request.rs
 src/models/v1_load_request_query.rs

rust/cubesql/cubeclient/src/models/mod.rs

@@ -10,6 +10,8 @@ pub mod v1_cube_meta_measure;
 pub use self::v1_cube_meta_measure::V1CubeMetaMeasure;
 pub mod v1_cube_meta_segment;
 pub use self::v1_cube_meta_segment::V1CubeMetaSegment;
+pub mod v1_cube_meta_type;
+pub use self::v1_cube_meta_type::V1CubeMetaType;
 pub mod v1_error;
 pub use self::v1_error::V1Error;
 pub mod v1_load_request;

rust/cubesql/cubeclient/src/models/v1_cube_meta.rs

@@ -14,6 +14,8 @@
     pub name: String,
     #[serde(rename = "title", skip_serializing_if = "Option::is_none")]
     pub title: Option<String>,
+    #[serde(rename = "type")]
+    pub r#type: crate::models::V1CubeMetaType,
     #[serde(rename = "description", skip_serializing_if = "Option::is_none")]
     pub description: Option<String>,
     #[serde(rename = "measures")]
@@ -29,13 +31,15 @@
 impl V1CubeMeta {
     pub fn new(
         name: String,
+        r#type: crate::models::V1CubeMetaType,
         measures: Vec<crate::models::V1CubeMetaMeasure>,
         dimensions: Vec<crate::models::V1CubeMetaDimension>,
         segments: Vec<crate::models::V1CubeMetaSegment>,
     ) -> V1CubeMeta {
         V1CubeMeta {
             name,
             title: None,
+            r#type,
             description: None,
             measures,
             dimensions,

rust/cubesql/cubeclient/src/models/v1_cube_meta_dimension.rs

@@ -15,17 +15,17 @@
     #[serde(rename = "description", skip_serializing_if = "Option::is_none")]
     pub description: Option<String>,
     #[serde(rename = "type")]
-    pub _type: String,
+    pub r#type: String,
     #[serde(rename = "granularities", skip_serializing_if = "Option::is_none")]
     pub granularities: Option<Vec<crate::models::V1CubeMetaDimensionGranularity>>,
 }
 
 impl V1CubeMetaDimension {
-    pub fn new(name: String, _type: String) -> V1CubeMetaDimension {
+    pub fn new(name: String, r#type: String) -> V1CubeMetaDimension {
         V1CubeMetaDimension {
             name,
             description: None,
-            _type,
+            r#type,
             granularities: None,
         }
     }

rust/cubesql/cubeclient/src/models/v1_cube_meta_measure.rs

@@ -17,18 +17,18 @@
     #[serde(rename = "description", skip_serializing_if = "Option::is_none")]
     pub description: Option<String>,
     #[serde(rename = "type")]
-    pub _type: String,
+    pub r#type: String,
     #[serde(rename = "aggType", skip_serializing_if = "Option::is_none")]
     pub agg_type: Option<String>,
 }
 
 impl V1CubeMetaMeasure {
-    pub fn new(name: String, _type: String) -> V1CubeMetaMeasure {
+    pub fn new(name: String, r#type: String) -> V1CubeMetaMeasure {
         V1CubeMetaMeasure {
             name,
             title: None,
             description: None,
-            _type,
+            r#type,
             agg_type: None,
         }
     }

rust/cubesql/cubeclient/src/models/v1_cube_meta_type.rs

@@ -0,0 +1,33 @@
+/*
+ * Cube.js
+ *
+ * Cube.js Swagger Schema
+ *
+ * The version of the OpenAPI document: 1.0.0
+ *
+ * Generated by: https://openapi-generator.tech
+ */
+
+///
+#[derive(Clone, Copy, Debug, Eq, PartialEq, Ord, PartialOrd, Hash, Serialize, Deserialize)]
+pub enum V1CubeMetaType {
+    #[serde(rename = "cube")]
+    Cube,
+    #[serde(rename = "view")]
+    View,
+}
+
+impl ToString for V1CubeMetaType {
+    fn to_string(&self) -> String {
+        match self {
+            Self::Cube => String::from("cube"),
+            Self::View => String::from("view"),
+        }
+    }
+}
+
+impl Default for V1CubeMetaType {
+    fn default() -> V1CubeMetaType {
+        Self::Cube
+    }
+}

rust/cubesql/cubesql/src/compile/rewrite/rules/members.rs

@@ -1688,7 +1688,7 @@ impl MemberRules {
                         .find_cube_with_name(time_dimension_name.split(".").next().unwrap())
                     {
                         if let Some(time_dimension) = cube.dimensions.iter().find(|d| {
-                            d._type == "time" && d.name.eq_ignore_ascii_case(&time_dimension_name)
+                            d.r#type == "time" && d.name.eq_ignore_ascii_case(&time_dimension_name)
                         }) {
                             for granularity in
                                 var_iter!(egraph[subst[granularity_var]], LiteralExprValue)

rust/cubesql/cubesql/src/compile/rewrite/rules/old_split.rs

@@ -5065,8 +5065,8 @@
                                 meta.find_cube_by_column(&alias_to_cube, &column)
                             {
                                 if let Some(dimension) = cube.lookup_dimension(&column.name) {
-                                    if is_time_dimension && dimension._type == "time"
-                                        || !is_time_dimension && dimension._type != "time"
+                                    if is_time_dimension && dimension.r#type == "time"
+                                        || !is_time_dimension && dimension.r#type != "time"
                                     {
                                         if let Some(expr_name) =
                                             original_expr_name(egraph, subst[arg_expr_var])