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.

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

chore: datafusion 44 upgrade #867

Merged
merged 2 commits into from
Jan 3, 2025
Merged

Conversation

gruuya
Copy link
Contributor

@gruuya gruuya commented Jan 3, 2025

Pick up the latest datafusion version.

@Xuanwo
Copy link
Member

Xuanwo commented Jan 3, 2025

This should also fix

# `instant` is unmaintained
#
# Introduced by datafusion, will be fixed in datafusion 44.
"RUSTSEC-2024-0384",

Would you like to help update it?

@gruuya
Copy link
Contributor Author

gruuya commented Jan 3, 2025

Would you like to help update it?

Certainly, done!

Copy link
Contributor

@liurenjie1024 liurenjie1024 left a comment

Choose a reason for hiding this comment

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

Thanks @gruuya for this pr!

@liurenjie1024
Copy link
Contributor

Wait a moment to see if @Xuanwo have other concerns.

@Xuanwo
Copy link
Member

Xuanwo commented Jan 3, 2025

Let's good to me, let's move!

@Xuanwo Xuanwo merged commit 974d9de into apache:main Jan 3, 2025
17 checks passed
@gruuya gruuya deleted the datafusion-44-upgrade branch January 3, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants