Skip to content

Commit

Permalink
Fix some log messages.
Browse files Browse the repository at this point in the history
  • Loading branch information
plypaul committed Dec 12, 2024
1 parent 8ece79f commit 584e9dc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ def __init__(self, left_branch_node: DataflowPlanNode) -> None: # noqa: D107
self._current_left_node: DataflowPlanNode = left_branch_node

def _log_visit_node_type(self, node: DataflowPlanNode) -> None:
logger.debug(lambda: f"Visiting {node.node_id}")
logger.debug(LazyFormat(lambda: f"Visiting {node.node_id}"))

def _log_combine_failure(
self,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ def visit_combine_aggregated_outputs_node( # noqa: D102
for branch_combination_result in combination_results
]

logger.debug(lambda: f"Got {len(combined_parent_branches)} branches after combination")
logger.debug(LazyFormat(lambda: f"Got {len(combined_parent_branches)} branches after combination"))
assert len(combined_parent_branches) > 0

# If we were able to reduce the parent branches of the CombineAggregatedOutputsNode into a single one, there's
Expand Down

0 comments on commit 584e9dc

Please sign in to comment.