Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update arrow-rs deps #317

Merged
merged 1 commit into from
May 11, 2021
Merged

Update arrow-rs deps #317

merged 1 commit into from
May 11, 2021

Conversation

alamb
Copy link
Contributor

@alamb alamb commented May 11, 2021

What changes are included in this PR?

Update to latest arrow-rs reference

I am actively working on a more regular release schedule for arrow-rs but until then I plan to keep pulling in the changes manually

I ran ./dev/update_arrow_deps.py and checked in the results

nevi-me
nevi-me approved these changes May 11, 2021
View reviewed changes
Copy link
Contributor

@nevi-me nevi-me left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From parquet, this brings in some fixes for the writer.

Do Datafusion or Ballista currently use the writer?

@Dandandan
Copy link
Contributor

Dandandan commented May 11, 2021 •
edited
Loading

From parquet, this brings in some fixes for the writer.

Do Datafusion or Ballista currently use the writer?

Yes ExecutionContext::write_parquet can be used ATM to write query results to parquet.

@Dandandan Dandandan merged commit a1c6898 into apache:master May 11, 2021
@alamb alamb deleted the update_deps branch May 11, 2021 18:02
@houqp houqp added ballista datafusion Changes in the datafusion crate labels Jul 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nevi-me nevi-me nevi-me approved these changes

@Dandandan Dandandan Dandandan approved these changes

Assignees
No one assigned
Labels
datafusion Changes in the datafusion crate
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@alamb @Dandandan @nevi-me @houqp