Skip to content

Commit

Permalink
Update a few calls to LazyFormat to lazy-evaluate arguments.
Browse files Browse the repository at this point in the history
  • Loading branch information
plypaul committed Feb 7, 2025
1 parent 05be429 commit 05a9c99
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -328,8 +328,8 @@ def optimize(self, dataflow_plan: DataflowPlan) -> DataflowPlan: # noqa: D102
logger.debug(
LazyFormat(
"Optimized dataflow plan",
original_plan=dataflow_plan.sink_node.structure_text(),
optimized_plan=optimized_result.optimized_branch.structure_text(),
original_plan=dataflow_plan.sink_node.structure_text,
optimized_plan=optimized_result.optimized_branch.structure_text,
)
)

Expand Down
2 changes: 1 addition & 1 deletion metricflow/sql/optimizer/column_pruning/column_pruner.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ def optimize(self, node: SqlPlanNode) -> SqlPlanNode: # noqa: D102
logger.error(
LazyFormat(
"The columns required at this node can't be determined, so skipping column pruning",
node=node.structure_text(),
node=node.structure_text,
required_select_columns=required_select_columns,
)
)
Expand Down

0 comments on commit 05a9c99

Please sign in to comment.