From e2b274f2a83ab5cf8d7826f7498cd28d703d9db3 Mon Sep 17 00:00:00 2001 From: Courtney Holcomb Date: Tue, 10 Sep 2024 11:00:19 -0500 Subject: [PATCH] Fix spelling --- metricflow/dataflow/optimizer/dataflow_optimizer_factory.py | 2 +- metricflow/engine/metricflow_engine.py | 4 ++-- tests_metricflow/query_rendering/compare_rendered_query.py | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/metricflow/dataflow/optimizer/dataflow_optimizer_factory.py b/metricflow/dataflow/optimizer/dataflow_optimizer_factory.py index fcf11eb06b..ac17abc8fe 100644 --- a/metricflow/dataflow/optimizer/dataflow_optimizer_factory.py +++ b/metricflow/dataflow/optimizer/dataflow_optimizer_factory.py @@ -30,7 +30,7 @@ def all_optimizations() -> FrozenSet[DataflowPlanOptimization]: return frozenset((DataflowPlanOptimization.SOURCE_SCAN, DataflowPlanOptimization.PREDICATE_PUSHDOWN)) @staticmethod - def enabled_obtimizations() -> FrozenSet[DataflowPlanOptimization]: + def enabled_optimizations() -> FrozenSet[DataflowPlanOptimization]: """Set of DataflowPlanOptimization that are currently enabled. Predicate pushdown optimizer is currently disabled. diff --git a/metricflow/engine/metricflow_engine.py b/metricflow/engine/metricflow_engine.py index 7c2b7f8260..ed0c52213d 100644 --- a/metricflow/engine/metricflow_engine.py +++ b/metricflow/engine/metricflow_engine.py @@ -111,7 +111,7 @@ class MetricFlowQueryRequest: order_by: Optional[Sequence[OrderByQueryParameter]] = None min_max_only: bool = False sql_optimization_level: SqlQueryOptimizationLevel = SqlQueryOptimizationLevel.O4 - dataflow_plan_optimizations: FrozenSet[DataflowPlanOptimization] = DataflowPlanOptimization.enabled_obtimizations() + dataflow_plan_optimizations: FrozenSet[DataflowPlanOptimization] = DataflowPlanOptimization.enabled_optimizations() query_type: MetricFlowQueryType = MetricFlowQueryType.METRIC @staticmethod @@ -130,7 +130,7 @@ def create_with_random_request_id( # noqa: D102 sql_optimization_level: SqlQueryOptimizationLevel = SqlQueryOptimizationLevel.O4, dataflow_plan_optimizations: FrozenSet[ DataflowPlanOptimization - ] = DataflowPlanOptimization.enabled_obtimizations(), + ] = DataflowPlanOptimization.enabled_optimizations(), query_type: MetricFlowQueryType = MetricFlowQueryType.METRIC, min_max_only: bool = False, ) -> MetricFlowQueryRequest: diff --git a/tests_metricflow/query_rendering/compare_rendered_query.py b/tests_metricflow/query_rendering/compare_rendered_query.py index af441a48e8..1d36ffce46 100644 --- a/tests_metricflow/query_rendering/compare_rendered_query.py +++ b/tests_metricflow/query_rendering/compare_rendered_query.py @@ -54,11 +54,11 @@ def render_and_check( # Run dataflow -> sql conversion with all optimizers if is_distinct_values_plan: optimized_plan = dataflow_plan_builder.build_plan_for_distinct_values( - query_spec, optimizations=DataflowPlanOptimization.enabled_obtimizations() + query_spec, optimizations=DataflowPlanOptimization.enabled_optimizations() ) else: optimized_plan = dataflow_plan_builder.build_plan( - query_spec, optimizations=DataflowPlanOptimization.enabled_obtimizations() + query_spec, optimizations=DataflowPlanOptimization.enabled_optimizations() ) conversion_result = dataflow_to_sql_converter.convert_to_sql_query_plan( sql_engine_type=sql_client.sql_engine_type,