Skip to content

Update README.md - #2511

Closed
fahadullahshah261 wants to merge 1 commit into
apache:mainfrom
fahadullahshah261:main
Closed

fahadullahshah261 wants to merge 1 commit into
apache:mainfrom
fahadullahshah261:main

Conversation

@fahadullahshah261

Copy link
Copy Markdown

Added RisingWave to the users of sqlparser-rs

Added RisingWave to the users of sqlparser-rs
@LucaCappelletti94

Copy link
Copy Markdown
Contributor

RisingWave doesn't depend on this crate. Its parser is an in-tree fork of an old sqlparser-rs (https://github.com/risingwavelabs/risingwave/blob/main/src/sqlparser/README.md)

sqlparser only appears in its Cargo.lock transitively via DataFusion, which is already on the list. Closing. If RisingWave adds a direct dependency, reopen with a link to the Cargo.toml.

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.

2 participants