-
Notifications
You must be signed in to change notification settings - Fork 416
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deltalake.DeltaTable constructor hangs in Jupyter #1093
Comments
I am seeing this issue as well. |
@xbrianh - can you try delta-rs v0.6.4? If I recall correctly, this was a 0.6.0 bug that was fixed. Let me know if that helps! |
@MrPowers Same issue:
|
This was never resolved for me in #915. I can still repro in |
@MrPowers I compiled a develop version and ran a backtrace. Looks like it's hanging writing to stdio for some reason: delta-rs/rust/src/action/parquet_read/mod.rs Line 468 in aecccba
|
This can be worked around by setting the following configuration value: c.IPKernelApp.capture_fd_output = False. It is related to this github issue: jupyterlab/jupyterlab#12845. That said can anyone tell me why this much output is being generated? (This is a VERY short summary of what it's trying to print)
|
Looks like this is a known issue maybe the fact that this will hang a default jupyter lab installation ups the priority? #928 |
@houqp many thanks! Any idea what the side effect of not respecting that log field is? I'm new to all this delta table stuff but to a beginner it seems like we will be getting rows back that should be deleted. |
# Description logging this error per row is too verbose when reading newer unsupported versions of delta log # Related Issue(s) closes #1093
@jabbera you will end up reading deleted rows if the deletion vector is set but ignored, so we should probably add support for this as soon we could to avoid data issue. |
@jabbera @houqp If the deletion vector has a value, then the reader protocol version should be 3, and we will therefore error (since the max we support is 1). So this should never happen. |
@wjones127 any idea when we are getting a new release of the python bindings with this fix? |
Yeah I will probably cut a release next weekend. |
# Description logging this error per row is too verbose when reading newer unsupported versions of delta log # Related Issue(s) closes delta-io#1093
Environment
Delta-rs version:0.6.0
Binding: python 3.10
Environment:
Bug
What happened: DeltaTable constructor hangs in Jupyter
What you expected to happen: Not hang
How to reproduce it: Attempt to create an azure storage backed DeltaTable in Jupyter
More details: This was lightly discussed in #915
The text was updated successfully, but these errors were encountered: