-
Notifications
You must be signed in to change notification settings - Fork 62
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
[v1] Replace sprout-generated AST with hand-written AST #1610
Labels
V1
This is related to the V1 release of PartiQL.
Comments
This was referenced Oct 8, 2024
This was referenced Oct 17, 2024
This was referenced Oct 25, 2024
This was referenced Nov 2, 2024
This was referenced Dec 6, 2024
This was referenced Dec 18, 2024
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Tracking issue for replacing the sprout-generated AST with the hand-written AST. The following will need to be completed to mark this issue as completed:
SqlDialect
- [v1] Port SqlDialect to v1 AST #1638v1
from AST package path #1639.valueOf
and.values
methods for AST enums - [v1] Add .valueOf and .values methods for enums #1628.name
method to AST enum interface - [v1] Add name method to AstEnum #1630.codes() -> int[]
,.names() -> strings[]
,code(string) -> int
,name(int) -> string
methods forAstEnum
schildren
function to different signaturepublic List<AstNode> getChildren()
- [v1] Refactor AstNode getChildren() method; change tag to be mutable int #1674AstNode
'sgetChildren
. E.g. inRelBase
--partiql-lang-kotlin/partiql-plan/src/main/java/org/partiql/plan/rel/RelBase.java
Lines 50 to 57 in 158b814
UNKNOWN
AST enum variant - [v1] Remove UNKNOWN AST enum variant #1688Out of scope for this issue (will be addressed in subsequent PRs)
PartiQLValue
[V1] Remove PartiQLValue from the AST #1589The text was updated successfully, but these errors were encountered: