[Python] Add python_scan_all_frames to opt-in to scanning all frames (< 1.1 behavior)
#13896
Conversation
…frames in a replacement scan
|
Thanks! |
|
Howdy @Tishj! Would it be possible to set Thanks! |
I don't think that's desired behavior in all cases, it feels much more sane to follow python's variable lookup logic here than to automatically dig through all frames. |
Makes sense! I agree, isolating in the same way that Python isolates is the best way. I think the JupySQL issue might be that they were stretching the bounds of what is recommended in some way. Thanks! |
Improve error on enums (duckdb/duckdb#13885) [Python] Add `python_scan_all_frames` to opt-in to scanning all frames (< 1.1 behavior) (duckdb/duckdb#13896)
Improve error on enums (duckdb/duckdb#13885) [Python] Add `python_scan_all_frames` to opt-in to scanning all frames (< 1.1 behavior) (duckdb/duckdb#13896) Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
This PR fixes #13836
Using
con.execute("set python_scan_all_frames=true")we will scan all frames again when performing a replacement scan.Meaning that if
dfis not defined in the current scope, but it is defined in the previous scope (or deeper), then it will still be found if this setting is flipped to true.