[GitHub] [accumulo] dlmarion opened a new issue, #3308: OnDemand Follow-on: Create a public RowRange object with start/end rows and exclusivity booleans

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

dlmarion opened a new issue, #3308:
URL: https://github.com/apache/accumulo/issues/3308

   Reference comment at: https://github.com/apache/accumulo/pull/3294#discussion_r1167115920. There are a lot of places in the API where `Text start` and `Text end` can be set by the user to specify a Range. This task is about creating a common object that captures the start, end, and inclusivity of both, and then using that in all places in the API where a range is being passed.


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

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

[GitHub] [accumulo] dlmarion commented on issue #3308: OnDemand Follow-on: Create a public RowRange object with start/end rows and exclusivity booleans

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

dlmarion commented on issue #3308:
URL: https://github.com/apache/accumulo/issues/3308#issuecomment-1515324167

   I think it depends on the implementation. I think you might be able to do the following and target `main`:
   
   1. Create new RowRange public API object
   2. Everywhere in the API where Text start/stop row values are being used, deprecate that method and create a new method that uses the new API object.
   
   It's yet to be determined how/where the inclusivity arguments are used in the existing methods.


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

[GitHub] [accumulo] DomGarguilo commented on issue #3308: OnDemand Follow-on: Create a public RowRange object with start/end rows and exclusivity booleans

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

DomGarguilo commented on issue #3308:
URL: https://github.com/apache/accumulo/issues/3308#issuecomment-1515319564

   @dlmarion do you think this change should be targeting the `elasticity` branch directly? Or should this new object be created and used in `main` and then merged into `elasticity`?


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