Skip to content

Refactoring: Move DCL parsing into src/parser/dcl.rs - #2710

Open
s5dsn-eqee wants to merge 3 commits into
apache:mainfrom
s5dsn-eqee:parser-split-dcl
Open

s5dsn-eqee wants to merge 3 commits into
apache:mainfrom
s5dsn-eqee:parser-split-dcl

Conversation

@s5dsn-eqee

Copy link
Copy Markdown
Contributor

Refactoring: Move DCL parsing into src/parser/dcl.rs

Pure move of the GRANT/DENY/REVOKE, CREATE USER, CREATE ROLE, SET ROLE,
USE, CREATE POLICY and DROP POLICY parsers out of src/parser/mod.rs.
Three functions called from mod.rs become pub(super). Part of #2591,
layout per #2650.

@LucaCappelletti94

Copy link
Copy Markdown
Contributor

@s5dsn-eqee FYI if you were wondering at the CI issues, basically the whole Apache org is currently sortof maxed out and all CIs are stuck. Infra has already on the case, but it may not be an immediate fix.

@github-actions

Copy link
Copy Markdown

No dialect label was applied because the title does not name a SQL dialect.

If this change targets one or more dialects, edit the title to <Dialect>[, <Dialect>]: <description>, add a Dialects: <Dialect>[, <Dialect>] line to the description, or reply with one. Reply Dialects: none if the change is dialect-agnostic.

@codecov-commenter

codecov-commenter commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.55828% with 134 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.83%. Comparing base (6a8b8bb) to head (8789743).

Files with missing lines Patch % Lines
src/parser/dcl.rs 83.55% 55 Missing and 79 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2710   +/-   ##
=======================================
  Coverage   81.83%   81.83%           
=======================================
  Files          42       43    +1     
  Lines       34651    34654    +3     
  Branches    34651    34654    +3     
=======================================
+ Hits        28357    28360    +3     
  Misses       2753     2753           
  Partials     3541     3541           

☔ 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants