-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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 long varchar #5920
Conversation
Please add the issue link in the description. |
ae5aeea
to
3e84bae
Compare
LGTM |
/run-all-tests |
LGTM |
@zhexuany If all the tests passed, please add the all-tests-passed label for the PR. |
/run-all-tests |
@zhexuany Please fix the CI. |
Ran unit-test locally, passed. Reran all tests. /run-all-tests |
merge master's changes and run all tests: /run-all-tests |
@shenli PTAL |
/run-all-tests |
/run-unit-test |
Could we merge this now? |
/run-all-tests |
Its ready to merge. |
This PR supports
long varchar
which is alias ofmediumtext
.