Skip to content

Conversation

@iajoiner
Copy link
Contributor

Closes #1729

We use the serde feature of sqlparser in SXT Proof of SQL and want to ensure no_std support for some of our use cases. Hence we filed this PR.

Thanks to @JayWhite2357 for providing the fix.

Copy link
Contributor

@iffyio iffyio left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @iajoiner!
cc @alamb

@iffyio iffyio merged commit 28736da into apache:main Feb 21, 2025
9 checks passed
@iajoiner iajoiner mentioned this pull request Feb 23, 2025
@iajoiner iajoiner deleted the fix/serde-std branch April 4, 2025 15:17
ayman-sigma pushed a commit to sigmacomputing/sqlparser-rs that referenced this pull request Apr 10, 2025
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.

Make the serde feature of sqlparser no_std

2 participants