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
[CALCITE-1684] Set the precision for VARCHAR and VARBINARY to the max… #394
Conversation
…imum value when not specified
… 1 to "unspecified" (Kevin Liew) If you concatenate two strings and the sum of their precisions is greater than the maximum precision (65,536) the result precision will be unspecified. Close apache#394
… 1 to "unspecified" (Kevin Liew) If you concatenate two strings and the sum of their precisions is greater than the maximum precision (65,536) the result precision will be unspecified. Close apache#394
… 1 to "unspecified" (Kevin Liew) If you concatenate two strings and the sum of their precisions is greater than the maximum precision (65,536) the result precision will be unspecified. Close apache#394
… 1 to "unspecified" (Kevin Liew) If you concatenate two strings and the sum of their precisions is greater than the maximum precision (65,536) the result precision will be unspecified. Close apache#394
… 1 to "unspecified" (Kevin Liew) If you concatenate two strings and the sum of their precisions is greater than the maximum precision (65,536 by default on Calcite, but can be overriden in RelDataTypeSystem) the result precision will be unspecified. When casting from unspecified precision to a particular precision, may need to truncate values. Close apache#394
@julianhyde @kliewkliew: this change is causing failures in DruidAdapterIT. |
@nishantmonu51 the tests passed when I ran |
The DruidAdapterIT tests don't get run unless you have a https://github.com/vlsi/calcite-test-dataset VM running. |
We don't require committers to run integration tests for every commit. Usually we're OK - but in this case we're not. On the server where I run tests, the mere fact that the VM is running seems to make it more likely that a test run will seg-fault, so I've been less inclined to run it recently. |
… 1 to "unspecified" (Kevin Liew) If you concatenate two strings and the sum of their precisions is greater than the maximum precision (65,536 by default on Calcite, but can be overriden in RelDataTypeSystem) the result precision will be unspecified. When casting from unspecified precision to a particular precision, may need to truncate values. Some unrelated USING tests, and fix typos in method names. Close apache/calcite#394
…imum value when not specified