-
Notifications
You must be signed in to change notification settings - Fork 81
Refactor SplitPart expression #1415
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
Conversation
ericvergnaud
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR needs to move to remorph-community-transpiler
bishwajit-db
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@ericvergnaud Eventually, it will, I think, given the interdependency between reconcile depending on custom dialects, we will have to keep this operational within this repo. |
sundarshankar89
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@gueniai Any thoughts? |
|
moved to RCT |
Pull request was closed
Removed
local_expression.SplitPartand refactored to sqlglotexpression.SplitPart.Closes #1411