Skip to content

Commit 3909faf

Browse files
timsaucerclaude
andcommitted
docs: drop redundant comment in SessionContext constructor
Remove the explanatory comment about FFI bridge availability; the same information already lives on PhysicalOptimizerRuleExportable. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 108cd13 commit 3909faf

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

crates/core/src/context.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -397,11 +397,6 @@ impl PySessionContext {
397397
.with_config(config)
398398
.with_runtime_env(runtime)
399399
.with_default_features();
400-
// DataFusion exposes no FFI bridge for the logical optimizer or
401-
// analyzer, so only physical optimizer rules can be supplied from
402-
// another library. They are appended after the default rules at
403-
// construction time; there is no upstream API to add them to a live
404-
// `SessionContext`.
405400
for rule in physical_optimizer_rules.unwrap_or_default() {
406401
let rule = crate::physical_optimizer::physical_optimizer_rule_from_pyobject(&rule)?;
407402
state_builder = state_builder.with_physical_optimizer_rule(rule);

0 commit comments

Comments
 (0)