You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Now, the method of extending sqlparser in ceresdb is too rough. We copy tons of codes from sqlparser and it's a kind of hard-fork in fact. So as I think, forking sqlparser and moving the extending part to it may be better.
Proposal
Moving 'Statement' used in ceresdb to sqlparser.
Add related keywords.
Add related branch in parse_statement().
Additional context
The text was updated successfully, but these errors were encountered:
Yep... The burden of maintaining the parser part still exists.
The functions(such as parse_columns, parse_optional_column_option, etc) are customized in ceresdb and conflicts with those in sqlparser. It seems not so suitable to move them to sqlparser.
Description
Now, the method of extending sqlparser in ceresdb is too rough. We copy tons of codes from sqlparser and it's a kind of hard-fork in fact. So as I think, forking sqlparser and moving the extending part to it may be better.
Proposal
Additional context
The text was updated successfully, but these errors were encountered: