-
Notifications
You must be signed in to change notification settings - Fork 490
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
parser: support secondary_load
and secondary_unload
#496
Conversation
Codecov Report
@@ Coverage Diff @@
## master #496 +/- ##
==========================================
+ Coverage 71.48% 71.49% +0.01%
==========================================
Files 32 32
Lines 7672 7676 +4
==========================================
+ Hits 5484 5488 +4
Misses 1667 1667
Partials 521 521
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #496 +/- ##
==========================================
+ Coverage 71.49% 71.49% +<.01%
==========================================
Files 32 32
Lines 7675 7676 +1
==========================================
+ Hits 5487 5488 +1
Misses 1667 1667
Partials 521 521
Continue to review full report at Codecov.
|
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...
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
@spongedu resolve conflicts please |
What problem does this PR solve?
Fix #444 , support
SECONDARY_LOAD
andSECONDARY_UNLOAD
option inALTER TABLE
syntaxCheck List
Tests
Code changes
N / A
Side effects
N / A
Related changes
N / A