kevinjmh opened a new pull request #3649: [CARBONDATA-3730] Avoid data conversion and remove duplicate codes in BlockIndexerStorage URL: https://github.com/apache/carbondata/pull/3649 ### Why is this PR needed? The initial purpose of this PR was to remove the conversion between `byte[][]` and `List` when applying RLE on datapage in `BlockIndexerStorageForNoInvertedIndexForShort`, especially case when rle won't give any benefit. After checking other subclass of `BlockIndexerStorage` whether has similar problem, we found many duplicate codes and clean them up also. ### What changes were proposed in this PR? - avoid some conversion between `byte[][]` and `List` - refactor classes of `BlockIndexerStorage` to remove duplicate codes ### Does this PR introduce any user interface change? - No - Yes. (please explain the change and update document) ### Is any new testcase added? - No - Yes
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
