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

Incorrect fallthrough in Java PartialRow.setMin for DECIMAL times

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 1.7.0
    • Fix Version/s: 1.4.1, 1.5.1, 1.6.1, 1.8.0, 1.7.1
    • Component/s: java
    • Labels:
      None
    • Target Version/s:

      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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: