Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-2416

Incorrect fallthrough in Java PartialRow.setMin for DECIMAL times

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Critical
    • Resolution: Fixed
    • 1.7.0
    • 1.4.1, 1.5.1, 1.6.1, 1.8.0, 1.7.1
    • java
    • None

    Description

      There's a missing 'break' statement in the following code:

            case DECIMAL:
              ColumnTypeAttributes typeAttributes = column.getTypeAttributes();
              addDecimal(index,
                  DecimalUtil.minValue(typeAttributes.getPrecision(), typeAttributes.getScale()));
            case STRING:
              addStringUtf8(index, AsyncKuduClient.EMPTY_ARRAY);
              break;
      

      which I think could cause incorrect results for range partition pruning on decimal columns.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            granthenke Grant Henke
            tlipcon Todd Lipcon
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment