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

Fix datafusion proto crate json feature #15172

Merged

Conversation

Owen-CH-Leung
Copy link
Contributor

@Owen-CH-Leung Owen-CH-Leung commented Mar 12, 2025

Fixes #15157

This PR fixes the json feature of the datafusion proto crate which is currently broken as main.

Fix datafusion proto crate json feature
@github-actions github-actions bot added the proto Related to proto crate label Mar 12, 2025
@Owen-CH-Leung Owen-CH-Leung marked this pull request as ready for review March 12, 2025 06:22
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you @Owen-CH-Leung 🙏

Can you please merge up from main now that this PR is merged

(per @timsaucer 's comments #15157 (comment))

@alamb
Copy link
Contributor

alamb commented Mar 13, 2025

Thank you @Owen-CH-Leung 🙏

Can you please merge up from main now that this PR is merged

(per @timsaucer 's comments #15157 (comment))

I took the liberty of pushing a commit that did this:
a4441bb

Thanks again @Owen-CH-Leung

@github-actions github-actions bot added the development-process Related to development process of DataFusion label Mar 13, 2025
@alamb alamb changed the title Fix datafusion proto crate json feature Fix datafusion proto crate json feature Mar 13, 2025
@Owen-CH-Leung
Copy link
Contributor Author

Thank you @Owen-CH-Leung 🙏
Can you please merge up from main now that this PR is merged

(per @timsaucer 's comments #15157 (comment))

I took the liberty of pushing a commit that did this: a4441bb

Thanks again @Owen-CH-Leung

Just saw your msg. Thanks for helping to rebase :)

@comphead comphead merged commit d381306 into apache:main Mar 13, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development-process Related to development process of DataFusion proto Related to proto crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

datafusion-proto crate feature json is broken
3 participants