Skip to content

Commit a2c74e4

Browse files
timsaucerclaude
andcommitted
style: freeze the resolved-tables carrier
Same reasoning as PhysicalOptimizerRules gaining frozen on the base branch: nothing mutates it between resolve and commit, so there is no reason to pay for the runtime borrow flag a mutable pyclass carries. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent bb62d46 commit a2c74e4

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

crates/core/src/context.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1894,7 +1894,8 @@ impl PySessionContext {
18941894
/// [`PyPhysicalOptimizerRules`]. Each entry is a provider that has already been
18951895
/// imported and a schema that has already been looked up, so committing is an
18961896
/// insert into a resolved destination rather than a fresh name resolution.
1897-
#[pyclass(name = "ResolvedTables", module = "datafusion._internal")]
1897+
/// `frozen` for the same reason as its sibling: the commit only reads.
1898+
#[pyclass(frozen, name = "ResolvedTables", module = "datafusion._internal")]
18981899
pub struct PyResolvedTables {
18991900
tables: Vec<ResolvedTable>,
19001901
}

0 commit comments

Comments
 (0)