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

Support Parsing ALTER TRIGGER in PostgreSQL #20611

Merged
merged 1 commit into from Aug 28, 2022

Conversation

everly-gif
Copy link
Contributor

Ref #17848
Sub-issue of #14015

Changes proposed in this pull request:

  • Proofread ALTER TRIGGER grammar
  • Support Parsing ALTER TRIGGER in PostgreSQL
  • Adds tests

✅ Builds locally

@codecov-commenter
Copy link

Codecov Report

Merging #20611 (fa131e8) into master (6692919) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master   #20611      +/-   ##
============================================
- Coverage     61.06%   61.06%   -0.01%     
  Complexity     2414     2414              
============================================
  Files          3951     3952       +1     
  Lines         54878    54880       +2     
  Branches       9318     9318              
============================================
+ Hits          33509    33510       +1     
- Misses        18522    18523       +1     
  Partials       2847     2847              
Impacted Files Coverage Δ
...ostgresql/ddl/PostgreSQLAlterTriggerStatement.java 0.00% <0.00%> (ø)
...atement/impl/PostgreSQLDDLStatementSQLVisitor.java 85.57% <100.00%> (+0.03%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@strongduanmu strongduanmu merged commit 6846b2c into apache:master Aug 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants