Github user navsan commented on the issue:
Please address the comments: mostly to do with performance improvements.
Note that the objective here is not just to make SplitRowStore faster than it
is today, but to get it to the point where we can drop PackedRowStore. We're
not close to that right now, because we're doing far too much work in the inner
- Consider getting rid of occupancy_bitmap_
- Consider moving the null_bitmap into the header as a single bitmap for
the entire tuple storage subblock.
- Consider getting rid of TypedValue
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket