feat(sqlglot): Vertica dialect#39969
Conversation
Code Review Agent Run #cad16dActionable Suggestions - 0Review Details
Bito Usage GuideCommands Type the following command in the pull request comment and save the comment.
Refer to the documentation for additional commands. Configuration This repository uses Documentation & Help |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #39969 +/- ##
=======================================
Coverage 63.88% 63.88%
=======================================
Files 2583 2584 +1
Lines 136618 136635 +17
Branches 31502 31502
=======================================
+ Hits 87272 87288 +16
- Misses 47830 47831 +1
Partials 1516 1516
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Vitor-Avila
left a comment
There was a problem hiding this comment.
Approved! Thank you so much
|
Bito Automatic Review Skipped – PR Already Merged |
SUMMARY
Vertica currently uses the Postgres dialect, since it's based on its wire protocol. But some functions end up being translated incorrectly when we apply RLS and go through a cycle of parsing/modifying/rendering. For example:
This PR introduces a Vertica dialect, with support for functions that behave differently from Postgres.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION