[GitHub] [accumulo] cshannon commented on pull request #3328: WIP - add RowRange object

["cshannon (via GitHub)" <gi...@apache.org>]

cshannon commented on PR #3328:
URL: https://github.com/apache/accumulo/pull/3328#issuecomment-1518261046

   Having a RowRange object would be useful in 3.1 and not just elasticity. As part of #1327 I need to use a range for fencing off table files. Right now in my prototypes I'm just using the Range object and leaving cq/cf, etc null/empty but really we just need to fence off by rows.
   
   It would need to support some of the original methods though of Range for it to be used. There are some very useful methods in Range like [mergeOverlapping](https://github.com/cshannon/accumulo/blob/9a31c4257b54ed890f02ab9aafe6d99a9638dac6/core/src/main/java/org/apache/accumulo/core/data/Range.java#L418) and [clip](https://github.com/cshannon/accumulo/blob/9a31c4257b54ed890f02ab9aafe6d99a9638dac6/core/src/main/java/org/apache/accumulo/core/data/Range.java#L511) that would need to exist.
   
   After I talked about this with @keith-turner last week I was thinking a one approach would be to do some refactoring and move things just for rows into RowRange and then have Range extend RowRange and add CF, CQ, etc.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org