-
Notifications
You must be signed in to change notification settings - Fork 755
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
feat: short sql length setting #16502
Conversation
Hi @arkzuse Thanks for the contribution! |
@BohuTANG sorry for the oversight. What can I do to make it review ready? |
74c161f
to
4b58387
Compare
Currently, the settings are only defined. We need to implement them in the Something like You can refer to other settings for guidance on how to Finally, please add a test for this PR. Once all tests pass, it will be ready for review, and the tests can reference the other settings. |
For you reference: |
@arkzuse Hi, could you clarify why the PR was closed? Let me know if you need any help to continue. |
@BohuTANG I didn't close it. I am working on this pr and interested to contribute in future. |
Oh i deleted my local brach so it closed... |
4b58387
to
98d2c5a
Compare
@arkzuse Thanks for your contribution! |
I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/
Summary
New setting for short length sql.
Tests
Type of change
This change is