Uploaded image for project: 'ORC'
  1. ORC
  2. ORC-260

masking data for Decimal might not work as expected because of a bug in DecimalRedactConverter

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

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • None
    • Java
    • None

    Description

      In the DecimalRedactConverter.maskData() function

      else {
              for(int r = start; r < start + length; ++r) {
                target.isNull[r] = source.isNull[r];
                if (target.noNulls || !target.isNull[r]) {
                  target.vector[r].set(source.vector[r]);
                }
      

      should be

      else {
              for(int r = start; r < start + length; ++r) {
                target.isNull[r] = source.isNull[r];
                if (target.noNulls || !target.isNull[r]) {
                  target.vector[r].set(maskDecimal(source.vector[r]));
                }
      

      Attachments

        Activity

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

          People

            smore Sandeep More
            smore Sandeep More
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment