-
Notifications
You must be signed in to change notification settings - Fork 38
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
Implement sink stream for LazyFrame #343
Conversation
4c70ae6
to
dade261
Compare
@Sicheng-Pan thankyou for looking into this. I guess this PR is missing #323 to allow all optimization args. I have just updated #323 and will merge asap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect by adding opt-toggles the warning in unittest should go away. At least if explicitly choosing no CSE. Then PR is good.
#260
Implements
sink_ipc
andsink_parquet
forLazyFrame