Repository navigation
Refactoring: Move DCL parsing into src/parser/dcl.rs - #2710
s5dsn-eqee wants to merge 3 commits into
Conversation
|
@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. |
|
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 |
Codecov Report❌ Patch coverage is
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. 🚀 New features to boost your workflow:
|
Refactoring: Move DCL parsing into
src/parser/dcl.rsPure 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.