-
Notifications
You must be signed in to change notification settings - Fork 38
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
Backport to branch(3.13) : Add table metadata service #2504
Conversation
Co-authored-by: Peckstadt Yves <[email protected]>
OUT_OF_RANGE_COLUMN_VALUE_FOR_DATE( | ||
Category.USER_ERROR, | ||
"0158", | ||
"This DATE column value is out of the valid range. It must be between 1000-01-01 and 9999-12-12. Value: %s", | ||
"", | ||
""), | ||
SUBMICROSECOND_PRECISION_NOT_SUPPORTED_FOR_TIME( | ||
Category.USER_ERROR, | ||
"0159", | ||
"This TIME column value precision cannot be shorter than one microsecond. Value: %s", | ||
"", | ||
""), | ||
OUT_OF_RANGE_COLUMN_VALUE_FOR_TIMESTAMP( | ||
Category.USER_ERROR, | ||
"0160", | ||
"This TIMESTAMP column value is out of the valid range. It must be between 1000-01-01T00:00:00.000 and 9999-12-31T23:59:59.999. Value: %s", | ||
"", | ||
""), | ||
SUBMILLISECOND_PRECISION_NOT_SUPPORTED_FOR_TIMESTAMP( | ||
Category.USER_ERROR, | ||
"0161", | ||
"This TIMESTAMP column value precision cannot be shorter than one millisecond. Value: %s", | ||
"", | ||
""), | ||
OUT_OF_RANGE_COLUMN_VALUE_FOR_TIMESTAMPTZ( | ||
Category.USER_ERROR, | ||
"0162", | ||
"This TIMESTAMPTZ column value is out of the valid range. It must be between 1000-01-01T00:00:00.000Z to 9999-12-31T23:59:59.999Z. Value: %s", | ||
"", | ||
""), | ||
SUBMILLISECOND_PRECISION_NOT_SUPPORTED_FOR_TIMESTAMPTZ( | ||
Category.USER_ERROR, | ||
"0163", | ||
"This TIMESTAMPTZ column value precision cannot be shorter than one millisecond. Value: %s", | ||
"", | ||
""), | ||
JDBC_IMPORT_DATA_TYPE_OVERRIDE_NOT_SUPPORTED( | ||
Category.USER_ERROR, | ||
"0164", | ||
"The underlying-storage data type %s is not supported as the ScalarDB %s data type: %s", | ||
"", | ||
""), |
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.
3.13
doesn't have the time-related types. Please remove these messages.
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.
@brfrn169 san,
Thank you.
I have removed the unwanted error messages.
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! Thank you!
This is an automated request for a manual backport of the following:
Thank you!