Skip to content

PostgreSQL: parse ALTER TRIGGER RENAME TO and DEPENDS ON EXTENSION - #2430

Open
LucaCappelletti94 wants to merge 3 commits into
apache:mainfrom
LucaCappelletti94:pg-alter-trigger
Open

LucaCappelletti94 wants to merge 3 commits into
apache:mainfrom
LucaCappelletti94:pg-alter-trigger

Conversation

@LucaCappelletti94

Copy link
Copy Markdown
Contributor

ALTER TRIGGER tr ON t RENAME TO tq did not parse in any dialect, because TRIGGER was not among the object types parse_alter accepted.

PostgreSQL's grammar has exactly two forms, so AlterTriggerOperation covers RENAME TO and [ NO ] DEPENDS ON EXTENSION, the second reusing the shape AlterFunctionOperation::DependsOnExtension already provides for that same clause.

@LucaCappelletti94
LucaCappelletti94 marked this pull request as ready for review August 5, 2026 15:37
LucaCappelletti94 added a commit to LucaCappelletti94/sqlparser-rs that referenced this pull request Oct 1, 2026
@codecov-commenter

codecov-commenter commented Oct 16, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.55%. Comparing base (424378c) to head (eb54972).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2430      +/-   ##
==========================================
+ Coverage   81.52%   81.55%   +0.03%     
==========================================
  Files          42       42              
  Lines       34205    34250      +45     
  Branches    34205    34250      +45     
==========================================
+ Hits        27885    27934      +49     
+ Misses       2770     2767       -3     
+ Partials     3550     3549       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

3 participants