From a37377ab791fb555c1744f54f9005f9bdc229009 Mon Sep 17 00:00:00 2001 From: Aleksandr Romanenko Date: Thu, 27 Nov 2025 15:22:26 +0100 Subject: [PATCH 1/5] chore(tesseract): Small sql-call building refactoring --- .../src/cube_bridge/base_tools.rs | 1 - .../src/cube_bridge/member_sql.rs | 9 ++++---- .../cubesqlplanner/src/planner/query_tools.rs | 2 +- .../src/planner/sql_evaluator/compiler.rs | 9 ++++---- .../planner/sql_evaluator/sql_call_builder.rs | 9 ++++---- .../cube_bridge/mock_base_tools.rs | 7 ------- .../cube_bridge/mock_member_sql.rs | 5 +++-- .../test_fixtures/cube_bridge/mock_schema.rs | 21 +++++++++++++++---- .../cube_bridge/mock_segment_definition.rs | 7 ++++++- .../src/test_fixtures/schemas/mod.rs | 8 ++++--- .../test_fixtures/schemas/visitors_schema.rs | 6 +++++- 11 files changed, 52 insertions(+), 32 deletions(-) diff --git a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs index 1633461e3f890..fbe8d80073cb1 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs @@ -18,7 +18,6 @@ use std::rc::Rc; pub trait BaseTools { fn driver_tools(&self, external: bool) -> Result, CubeError>; fn sql_templates(&self) -> Result, CubeError>; - fn security_context_for_rust(&self) -> Result, CubeError>; fn sql_utils_for_rust(&self) -> Result, CubeError>; fn generate_time_series( &self, diff --git a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs index 77589f9354956..2e8c2557daa6f 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs @@ -3,6 +3,7 @@ use super::{ security_context::{NativeSecurityContext, SecurityContext}, sql_utils::NativeSqlUtils, }; +use crate::cube_bridge::base_tools::BaseTools; use crate::cube_bridge::sql_utils::SqlUtils; use crate::planner::sql_evaluator::SqlCallArg; use crate::utils::UniqueVector; @@ -301,7 +302,7 @@ pub trait MemberSql { fn as_any(self: Rc) -> Rc; fn compile_template_sql( &self, - sql_utils: Rc, + base_tools: Rc, security_context: Rc, ) -> Result<(SqlTemplate, SqlTemplateArgs), CubeError>; } @@ -634,7 +635,7 @@ impl MemberSql for NativeMemberSql { fn compile_template_sql( &self, - sql_utils: Rc, + base_tools: Rc, security_context: Rc, ) -> Result<(SqlTemplate, SqlTemplateArgs), CubeError> { let state = ProxyState::new(); @@ -667,8 +668,8 @@ impl MemberSql for NativeMemberSql { context_obj, )? } else if arg == "SQL_UTILS" { - sql_utils - .clone() + base_tools + .sql_utils_for_rust()? .as_any() .downcast::>() .unwrap() diff --git a/rust/cubesqlplanner/cubesqlplanner/src/planner/query_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/planner/query_tools.rs index 4a4f7364c65a9..a32a4be8b465e 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/planner/query_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/planner/query_tools.rs @@ -135,7 +135,7 @@ impl QueryTools { }; let evaluator_compiler = Rc::new(RefCell::new(Compiler::new( cube_evaluator.clone(), - base_tools.sql_utils_for_rust()?, + base_tools.clone(), security_context.clone(), timezone.clone(), ))); diff --git a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs index 938a9891f15c6..fc9d982417cac 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs @@ -4,6 +4,7 @@ use super::{ CubeNameSymbolFactory, CubeTableSymbolFactory, DimensionSymbolFactory, MeasureSymbolFactory, SqlCall, SymbolFactory, TraversalVisitor, }; +use crate::cube_bridge::base_tools::{self, BaseTools}; use crate::cube_bridge::evaluator::CubeEvaluator; use crate::cube_bridge::join_hints::JoinHintItem; use crate::cube_bridge::member_sql::MemberSql; @@ -16,7 +17,7 @@ use std::collections::HashMap; use std::rc::Rc; pub struct Compiler { cube_evaluator: Rc, - sql_utils: Rc, + base_tools: Rc, security_context: Rc, timezone: Tz, /* (type, name) */ @@ -26,14 +27,14 @@ pub struct Compiler { impl Compiler { pub fn new( cube_evaluator: Rc, - sql_utils: Rc, + base_tools: Rc, security_context: Rc, timezone: Tz, ) -> Self { Self { cube_evaluator, security_context, - sql_utils, + base_tools, timezone, members: HashMap::new(), } @@ -132,7 +133,7 @@ impl Compiler { let call_builder = SqlCallBuilder::new( self, self.cube_evaluator.clone(), - self.sql_utils.clone(), + self.base_tools.clone(), self.security_context.clone(), ); let sql_call = call_builder.build(&cube_name, member_sql.clone())?; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs index 23300f81c67c7..c14cb2d927988 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs @@ -1,6 +1,7 @@ use super::symbols::MemberSymbol; use super::Compiler; use super::{SqlCall, SqlCallDependency, SqlCallFilterGroupItem, SqlCallFilterParamsItem}; +use crate::cube_bridge::base_tools::BaseTools; use crate::cube_bridge::evaluator::CubeEvaluator; use crate::cube_bridge::member_sql::*; use crate::cube_bridge::security_context::SecurityContext; @@ -13,7 +14,7 @@ use std::rc::Rc; pub struct SqlCallBuilder<'a> { compiler: &'a mut Compiler, cube_evaluator: Rc, - sql_utils: Rc, + base_tools: Rc, security_context: Rc, } @@ -21,13 +22,13 @@ impl<'a> SqlCallBuilder<'a> { pub fn new( compiler: &'a mut Compiler, cube_evaluator: Rc, - sql_utils: Rc, + base_tools: Rc, security_context: Rc, ) -> Self { Self { compiler, cube_evaluator, - sql_utils, + base_tools, security_context, } } @@ -38,7 +39,7 @@ impl<'a> SqlCallBuilder<'a> { member_sql: Rc, ) -> Result { let (template, template_args) = member_sql - .compile_template_sql(self.sql_utils.clone(), self.security_context.clone())?; + .compile_template_sql(self.base_tools.clone(), self.security_context.clone())?; let deps = template_args .symbol_paths diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs index d453a905286ac..1795c7856793a 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs @@ -29,9 +29,6 @@ pub struct MockBaseTools { #[builder(default = Rc::new(MockSqlTemplatesRender::default_templates()))] sql_templates: Rc, - #[builder(default = Rc::new(MockSecurityContext))] - security_context: Rc, - #[builder(default = Rc::new(MockSqlUtils))] sql_utils: Rc, } @@ -55,10 +52,6 @@ impl BaseTools for MockBaseTools { Ok(self.sql_templates.clone()) } - fn security_context_for_rust(&self) -> Result, CubeError> { - Ok(self.security_context.clone()) - } - fn sql_utils_for_rust(&self) -> Result, CubeError> { Ok(self.sql_utils.clone()) } diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs index 3707f98938d5a..a477492cc8b23 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs @@ -1,3 +1,4 @@ +use crate::cube_bridge::base_tools::BaseTools; use crate::cube_bridge::member_sql::{MemberSql, SqlTemplate, SqlTemplateArgs}; use crate::cube_bridge::security_context::SecurityContext; use crate::cube_bridge::sql_utils::SqlUtils; @@ -112,7 +113,7 @@ impl MemberSql for MockMemberSql { fn compile_template_sql( &self, - _sql_utils: Rc, + _base_tools: Rc, _security_context: Rc, ) -> Result<(SqlTemplate, SqlTemplateArgs), CubeError> { Ok(( @@ -240,7 +241,7 @@ mod tests { let mock = Rc::new(MockMemberSql::new("{CUBE.field} / {other.field}").unwrap()); let (template, args) = mock .compile_template_sql( - Rc::new(crate::test_fixtures::cube_bridge::MockSqlUtils), + Rc::new(crate::test_fixtures::cube_bridge::MockBaseTools::default()), Rc::new(crate::test_fixtures::cube_bridge::MockSecurityContext), ) .unwrap(); diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs index 0e71c14db0cce..893a79f7f37d1 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs @@ -507,6 +507,7 @@ mod tests { use crate::cube_bridge::join_item_definition::JoinItemDefinition; use crate::cube_bridge::measure_definition::MeasureDefinition; use crate::cube_bridge::segment_definition::SegmentDefinition; + use crate::test_fixtures::cube_bridge::MockBaseTools; #[test] fn test_basic_schema() { @@ -1103,7 +1104,10 @@ mod tests { // Compile template and check symbol_paths structure let (_template, args) = checkin_id_sql - .compile_template_sql(Rc::new(MockSqlUtils), Rc::new(MockSecurityContext)) + .compile_template_sql( + Rc::new(MockBaseTools::default()), + Rc::new(MockSecurityContext), + ) .unwrap(); // Should have exactly one symbol path @@ -1126,7 +1130,10 @@ mod tests { let checkin_count_sql = checkin_count_measure.sql().unwrap().unwrap(); let (_template, args) = checkin_count_sql - .compile_template_sql(Rc::new(MockSqlUtils), Rc::new(MockSecurityContext)) + .compile_template_sql( + Rc::new(MockBaseTools::default()), + Rc::new(MockSecurityContext), + ) .unwrap(); assert_eq!( @@ -1146,7 +1153,10 @@ mod tests { let id_sql = id_dim.sql().unwrap().unwrap(); let (_template, args) = id_sql - .compile_template_sql(Rc::new(MockSqlUtils), Rc::new(MockSecurityContext)) + .compile_template_sql( + Rc::new(MockBaseTools::default()), + Rc::new(MockSecurityContext), + ) .unwrap(); assert_eq!( @@ -1164,7 +1174,10 @@ mod tests { let count_sql = count_measure.sql().unwrap().unwrap(); let (_template, args) = count_sql - .compile_template_sql(Rc::new(MockSqlUtils), Rc::new(MockSecurityContext)) + .compile_template_sql( + Rc::new(MockBaseTools::default()), + Rc::new(MockSecurityContext), + ) .unwrap(); assert_eq!( diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs index e8900971d3d54..a2de32c59b2aa 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs @@ -41,6 +41,8 @@ impl SegmentDefinition for MockSegmentDefinition { #[cfg(test)] mod tests { + use crate::test_fixtures::cube_bridge::MockBaseTools; + use super::*; #[test] @@ -90,7 +92,10 @@ mod tests { use crate::test_fixtures::cube_bridge::{MockSecurityContext, MockSqlUtils}; let (template, args) = sql - .compile_template_sql(Rc::new(MockSqlUtils), Rc::new(MockSecurityContext)) + .compile_template_sql( + Rc::new(MockBaseTools::default()), + Rc::new(MockSecurityContext), + ) .unwrap(); match template { diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs index 2c20c4c3c85cb..2f26983e1d058 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs @@ -3,7 +3,9 @@ pub mod visitors_schema; pub use visitors_schema::create_visitors_schema; use crate::planner::sql_evaluator::Compiler; -use crate::test_fixtures::cube_bridge::{MockCubeEvaluator, MockSecurityContext, MockSqlUtils}; +use crate::test_fixtures::cube_bridge::{ + MockBaseTools, MockCubeEvaluator, MockSecurityContext, MockSqlUtils, +}; use chrono_tz::Tz; use std::rc::Rc; @@ -20,9 +22,9 @@ impl TestCompiler { /// Create a new TestCompiler with a specific timezone pub fn new_with_timezone(evaluator: Rc, timezone: Tz) -> Self { - let sql_utils = Rc::new(MockSqlUtils); + let base_tools = Rc::new(MockBaseTools::default()); let security_context = Rc::new(MockSecurityContext); - let compiler = Compiler::new(evaluator, sql_utils, security_context, timezone); + let compiler = Compiler::new(evaluator, base_tools, security_context, timezone); Self { compiler } } diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs index 553ccc9414cd4..e1125405614ba 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs @@ -194,6 +194,7 @@ mod tests { use super::*; use crate::cube_bridge::dimension_definition::DimensionDefinition; use crate::cube_bridge::segment_definition::SegmentDefinition; + use crate::test_fixtures::cube_bridge::MockBaseTools; #[test] fn test_schema_has_both_cubes() { @@ -240,7 +241,10 @@ mod tests { let sql = question_mark.sql().unwrap().unwrap(); // Verify SQL contains question marks let (template, _args) = sql - .compile_template_sql(Rc::new(MockSqlUtils), Rc::new(MockSecurityContext)) + .compile_template_sql( + Rc::new(MockBaseTools::default()), + Rc::new(MockSecurityContext), + ) .unwrap(); match template { crate::cube_bridge::member_sql::SqlTemplate::String(s) => { From f147a1f7d459169081470ec8f06da35cce1aa99b Mon Sep 17 00:00:00 2001 From: Aleksandr Romanenko Date: Thu, 27 Nov 2025 15:23:58 +0100 Subject: [PATCH 2/5] in work --- .../cubesqlplanner/src/cube_bridge/base_tools.rs | 1 - .../cubesqlplanner/src/planner/sql_evaluator/compiler.rs | 3 +-- .../src/planner/sql_evaluator/sql_call_builder.rs | 1 - .../src/test_fixtures/cube_bridge/mock_base_tools.rs | 3 +-- .../src/test_fixtures/cube_bridge/mock_member_sql.rs | 1 - .../src/test_fixtures/cube_bridge/mock_segment_definition.rs | 2 +- .../cubesqlplanner/src/test_fixtures/schemas/mod.rs | 2 +- .../src/test_fixtures/schemas/visitors_schema.rs | 2 +- 8 files changed, 5 insertions(+), 10 deletions(-) diff --git a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs index fbe8d80073cb1..a80f802569558 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/base_tools.rs @@ -1,7 +1,6 @@ use super::driver_tools::{DriverTools, NativeDriverTools}; use super::join_definition::{JoinDefinition, NativeJoinDefinition}; use super::pre_aggregation_obj::{NativePreAggregationObj, PreAggregationObj}; -use super::security_context::{NativeSecurityContext, SecurityContext}; use super::sql_templates_render::{NativeSqlTemplatesRender, SqlTemplatesRender}; use super::sql_utils::{NativeSqlUtils, SqlUtils}; use crate::cube_bridge::join_hints::JoinHintItem; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs index fc9d982417cac..e9f23022f765a 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/compiler.rs @@ -4,12 +4,11 @@ use super::{ CubeNameSymbolFactory, CubeTableSymbolFactory, DimensionSymbolFactory, MeasureSymbolFactory, SqlCall, SymbolFactory, TraversalVisitor, }; -use crate::cube_bridge::base_tools::{self, BaseTools}; +use crate::cube_bridge::base_tools::BaseTools; use crate::cube_bridge::evaluator::CubeEvaluator; use crate::cube_bridge::join_hints::JoinHintItem; use crate::cube_bridge::member_sql::MemberSql; use crate::cube_bridge::security_context::SecurityContext; -use crate::cube_bridge::sql_utils::SqlUtils; use crate::planner::sql_evaluator::sql_call_builder::SqlCallBuilder; use chrono_tz::Tz; use cubenativeutils::CubeError; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs index c14cb2d927988..aea777a0a3220 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/planner/sql_evaluator/sql_call_builder.rs @@ -5,7 +5,6 @@ use crate::cube_bridge::base_tools::BaseTools; use crate::cube_bridge::evaluator::CubeEvaluator; use crate::cube_bridge::member_sql::*; use crate::cube_bridge::security_context::SecurityContext; -use crate::cube_bridge::sql_utils::SqlUtils; use crate::planner::sql_evaluator::TimeDimensionSymbol; use crate::planner::GranularityHelper; use cubenativeutils::CubeError; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs index 1795c7856793a..6987862e6bfbf 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs @@ -3,11 +3,10 @@ use crate::cube_bridge::driver_tools::DriverTools; use crate::cube_bridge::join_definition::JoinDefinition; use crate::cube_bridge::join_hints::JoinHintItem; use crate::cube_bridge::pre_aggregation_obj::PreAggregationObj; -use crate::cube_bridge::security_context::SecurityContext; use crate::cube_bridge::sql_templates_render::SqlTemplatesRender; use crate::cube_bridge::sql_utils::SqlUtils; use crate::test_fixtures::cube_bridge::{ - MockDriverTools, MockSecurityContext, MockSqlTemplatesRender, MockSqlUtils, + MockDriverTools, MockJoinGraph, MockSqlTemplatesRender, MockSqlUtils, }; use cubenativeutils::CubeError; use std::any::Any; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs index a477492cc8b23..8d7aa45653db1 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_member_sql.rs @@ -1,7 +1,6 @@ use crate::cube_bridge::base_tools::BaseTools; use crate::cube_bridge::member_sql::{MemberSql, SqlTemplate, SqlTemplateArgs}; use crate::cube_bridge::security_context::SecurityContext; -use crate::cube_bridge::sql_utils::SqlUtils; use cubenativeutils::CubeError; use std::any::Any; use std::rc::Rc; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs index a2de32c59b2aa..273d13599365f 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_segment_definition.rs @@ -90,7 +90,7 @@ mod tests { let sql = segment.sql().unwrap(); assert_eq!(sql.args_names(), &vec!["CUBE"]); - use crate::test_fixtures::cube_bridge::{MockSecurityContext, MockSqlUtils}; + use crate::test_fixtures::cube_bridge::MockSecurityContext; let (template, args) = sql .compile_template_sql( Rc::new(MockBaseTools::default()), diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs index 2f26983e1d058..100cba97ecab1 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs @@ -4,7 +4,7 @@ pub use visitors_schema::create_visitors_schema; use crate::planner::sql_evaluator::Compiler; use crate::test_fixtures::cube_bridge::{ - MockBaseTools, MockCubeEvaluator, MockSecurityContext, MockSqlUtils, + MockBaseTools, MockCubeEvaluator, MockSecurityContext, }; use chrono_tz::Tz; use std::rc::Rc; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs index e1125405614ba..8548504fa5b6d 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/visitors_schema.rs @@ -206,7 +206,7 @@ mod tests { #[test] fn test_visitors_dimensions() { - use crate::test_fixtures::cube_bridge::{MockSecurityContext, MockSqlUtils}; + use crate::test_fixtures::cube_bridge::MockSecurityContext; use std::rc::Rc; let schema = create_visitors_schema(); From c686028310481157ed75f8eeba43494f200356ef Mon Sep 17 00:00:00 2001 From: Aleksandr Romanenko Date: Thu, 27 Nov 2025 15:24:22 +0100 Subject: [PATCH 3/5] fmt --- .../cubesqlplanner/src/test_fixtures/schemas/mod.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs index 100cba97ecab1..c8f09cda01c0c 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/schemas/mod.rs @@ -3,9 +3,7 @@ pub mod visitors_schema; pub use visitors_schema::create_visitors_schema; use crate::planner::sql_evaluator::Compiler; -use crate::test_fixtures::cube_bridge::{ - MockBaseTools, MockCubeEvaluator, MockSecurityContext, -}; +use crate::test_fixtures::cube_bridge::{MockBaseTools, MockCubeEvaluator, MockSecurityContext}; use chrono_tz::Tz; use std::rc::Rc; From 30ff5e7ca51e03bc77468c912bbe64648970748b Mon Sep 17 00:00:00 2001 From: Aleksandr Romanenko Date: Thu, 27 Nov 2025 15:25:24 +0100 Subject: [PATCH 4/5] in work --- .../cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs | 1 - .../src/test_fixtures/cube_bridge/mock_base_tools.rs | 2 +- .../cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs index 2e8c2557daa6f..d2d1bc523348a 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/cube_bridge/member_sql.rs @@ -4,7 +4,6 @@ use super::{ sql_utils::NativeSqlUtils, }; use crate::cube_bridge::base_tools::BaseTools; -use crate::cube_bridge::sql_utils::SqlUtils; use crate::planner::sql_evaluator::SqlCallArg; use crate::utils::UniqueVector; use cubenativeutils::wrappers::object::{NativeFunction, NativeStruct, NativeType}; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs index 6987862e6bfbf..8221886dbf989 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs @@ -6,7 +6,7 @@ use crate::cube_bridge::pre_aggregation_obj::PreAggregationObj; use crate::cube_bridge::sql_templates_render::SqlTemplatesRender; use crate::cube_bridge::sql_utils::SqlUtils; use crate::test_fixtures::cube_bridge::{ - MockDriverTools, MockJoinGraph, MockSqlTemplatesRender, MockSqlUtils, + MockDriverTools, MockSqlTemplatesRender, MockSqlUtils, }; use cubenativeutils::CubeError; use std::any::Any; diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs index 893a79f7f37d1..b74a9b9203776 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_schema.rs @@ -1044,7 +1044,7 @@ mod tests { #[test] fn test_view_with_multiple_long_join_paths() { - use crate::test_fixtures::cube_bridge::{MockSecurityContext, MockSqlUtils}; + use crate::test_fixtures::cube_bridge::MockSecurityContext; use std::rc::Rc; let schema = MockSchemaBuilder::new() From 5a2f00d31229a1867305403b1abb11cea75cd660 Mon Sep 17 00:00:00 2001 From: Aleksandr Romanenko Date: Thu, 27 Nov 2025 16:45:42 +0100 Subject: [PATCH 5/5] fmt --- .../src/test_fixtures/cube_bridge/mock_base_tools.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs index 8221886dbf989..616eddfdcef68 100644 --- a/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs +++ b/rust/cubesqlplanner/cubesqlplanner/src/test_fixtures/cube_bridge/mock_base_tools.rs @@ -5,9 +5,7 @@ use crate::cube_bridge::join_hints::JoinHintItem; use crate::cube_bridge::pre_aggregation_obj::PreAggregationObj; use crate::cube_bridge::sql_templates_render::SqlTemplatesRender; use crate::cube_bridge::sql_utils::SqlUtils; -use crate::test_fixtures::cube_bridge::{ - MockDriverTools, MockSqlTemplatesRender, MockSqlUtils, -}; +use crate::test_fixtures::cube_bridge::{MockDriverTools, MockSqlTemplatesRender, MockSqlUtils}; use cubenativeutils::CubeError; use std::any::Any; use std::rc::Rc;