Skip to content
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

Determine future of FROM DML statements #1669

Open
1 task
johnedquinn opened this issue Dec 2, 2024 · 0 comments
Open
1 task

Determine future of FROM DML statements #1669

johnedquinn opened this issue Dec 2, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@johnedquinn
Copy link
Member

johnedquinn commented Dec 2, 2024

Relevant Issue/Bug

Requested Solution/Feature

  • In Adds DML (INSERT, DELETE, UPDATE, UPSERT, REPLACE) #1666, the FROM DML statements were removed intentionally due to the lack of an RFC and an active customer.
  • There were also issues with parsing with this. We accidentally allowed for FROM to precede any DML statement, such as DELETE, which is not supported by QLDB. The syntactical feature has been removed until its semantics have been formalized by the PartiQL Team.

Additional Context

  • FROM DML is used by the end-of-support QLDB. See the above link for the documentation.

DoD (Definition of Done)

  • Determine whether FROM (INSERT, REMOVE, SET) will be in the official PartiQL Language.
    • Note: If we do add it, we should have dedicated AST nodes to disambiguate syntax from semantics. AKA a FromInsert, FromRemove, FromSet.
@johnedquinn johnedquinn added the enhancement New feature or request label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant