-
Notifications
You must be signed in to change notification settings - Fork 122
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
Use sink cloud versions of ipc #705
Comments
@josevalim FYI I briefly spoke with Ritchie on the Polars Discord today and he merged the |
First of all: awesome work there on Polars, @Qqwy !! 🎉 Just to let you know that I'm working on this to bring the functionality to Explorer :) |
Hey @Qqwy, I had some issues trying to make the Sink IPC work, so I opened an issue upstream: pola-rs/polars#13614 I tried to find a fix for it, but I couldn't understand where the "block_on" is happening (twice). Would you mind to take a look? |
Hi there! Thanks for tagging me, I'll dig into this problem. Just to double-check: Is this only a problem with sink_ipc_cloud and not with sink_parquet_cloud? |
As far as I know, only ipc. @philss will be able to confirm though! |
@Qqwy yeah, it's only with the PS: sorry, I didn't see the ping before 😬 |
@philss I have created a PR with a fix on the Polars repo: pola-rs/polars#14262 |
With this, we can enable the `to_parquet/2` again, and add the `to_ipc/2` cloud. Closes #705
We need a new Explorer version:
The text was updated successfully, but these errors were encountered: