-
Notifications
You must be signed in to change notification settings - Fork 262
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
Modularized Parser #127
Modularized Parser #127
Conversation
ddl_statement = self.visitChildren(ctx) | ||
return ddl_statement | ||
|
||
def visitDmlStatement(self, ctx: evaql_parser.DdlStatementContext): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the difference between visitDdlStatement on line 54?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicate function, I'll remove one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate functions. I'll remove one
|
||
def visitLogicalExpression( | ||
self, ctx: evaql_parser.LogicalExpressionContext): | ||
if len(ctx.children) < 3: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LOGICAL_NOT only has 2 children
return ExpressionType.LOGICAL_OR | ||
elif op == 'AND': | ||
return ExpressionType.LOGICAL_AND | ||
else: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see we are not supporting NOT. I was thinking we have NOT supported, because https://github.com/georgia-tech-db/eva/blob/master/src/expression/logical_expression.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are not yet. We will add when we need be.
@gaurav274 we need to merge this first. |
…tead of raw values
* Add .circleci/config.yml * Leave out the coverage report * allow sudo to carry user set env var * add pip missing packages * a working version build and test workflow * empty line * Just to trigger new build * integration with coveralls
Modularized Parser
Parser modularized, moved from a single file to multiple modules