-
Notifications
You must be signed in to change notification settings - Fork 15
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
Longtext fix #70
base: develop
Are you sure you want to change the base?
Longtext fix #70
Conversation
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.
I actually don't think this change is required. We obviously want it to be greater than 333 but I think that was the maximum that mysql engine would allow at the time. Regardless, I think SI has already extended the max term length so this is likely not needed
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 one is actually required for large queries
6352584
to
d047fec
Compare
947fe13
to
c926f70
Compare
3b84396
to
198f2cd
Compare
This pull request provides an enhancement for longer text length support. These changes were required to support the existing SCTAU distribution format