Uploaded image for project: 'Parquet'
  1. Parquet
  2. PARQUET-543

Remove BoundedInt encodings

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.8.1
    • 1.9.0
    • parquet-mr
    • None

    Description

      The classes in org.apache.parquet.column.values.boundedint aren't used. It looks like this was intended to be the "right" way to use the RLE/BitPacking hybrid, but callers ended up instantiating the RLE encoder or writer directly.

      The ZeroIntegerValuesReader and DevNullValuesWriter are used, but should be relocated. The ZeroIntegerValuesReader is only used when the encoding is RLE (in Encoding.java) and the DevNullValuesWriter actually writes BIT_PACKED values. It would be better to relocate those classes in the rle and bitpacking packages.

      Attachments

        Activity

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

          People

            rdblue Ryan Blue Assign to me
            rdblue Ryan Blue
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment