You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Laurent Goujon (JIRA)" <ji...@apache.org> on 2016/10/14 23:10:20 UTC
[jira] [Commented] (CALCITE-1442)
SqlIntervalQualifer#getFractionalSecondPrecisionPreservingDefault() returns
the wrong field
[ https://issues.apache.org/jira/browse/CALCITE-1442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15576833#comment-15576833 ]
Laurent Goujon commented on CALCITE-1442:
-----------------------------------------
Proposed change: https://github.com/apache/calcite/pull/309
> 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)