[
https://issues.apache.org/jira/browse/CALCITE-1442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15576924#comment-15576924
]
Julian Hyde commented on CALCITE-1442:
--------------------------------------
I'm not sure. I tinkered with that code a while ago, and came to the conclusion
that while it looks wrong, it's actually correct. Can you provide a test case
that proves there's a bug here?
> SqlIntervalQualifer#getFractionalSecondPrecisionPreservingDefault() returns
> the wrong field
> -------------------------------------------------------------------------------------------
>
> Key: CALCITE-1442
> URL: https://issues.apache.org/jira/browse/CALCITE-1442
> Project: Calcite
> Issue Type: Bug
> Reporter: Laurent Goujon
> Assignee: Julian Hyde
> Priority: Minor
>
> Unless I'm wrong, I believe
> {{SqlIntervalQualifer#getFractionalSecondPrecisionPreservingDefault()}}
> returns the wrong field:
> {code:java}
> public int getFractionalSecondPrecision(RelDataTypeSystem typeSystem) {
> if (fractionalSecondPrecision == RelDataType.PRECISION_NOT_SPECIFIED) {
> return typeName().getDefaultScale();
> } else {
> return fractionalSecondPrecision;
> }
> }
> public int getFractionalSecondPrecisionPreservingDefault() {
> if (useDefaultFractionalSecondPrecision()) {
> return RelDataType.PRECISION_NOT_SPECIFIED;
> } else {
> return startPrecision;
> }
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
