[GitHub] [accumulo] keith-turner opened a new issue, #3520: Use new requireSame conditional mutation call in more places

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner opened a new issue, #3520:
URL: https://github.com/apache/accumulo/issues/3520

   In #3513 a new conditional mutation check was added called requireSame.  This new method allows one to easily check if something like the set of files on a tablet is the same as what you read earlier before making an update.  This new call could be used to simplify other conditional mutation checks for split, bulk import and location update.  Its most appropriate for operations that do not change the tablets prev end row (like some of the split code) or create a new tablet (like table creation and split).
   
   Using this new method may render some of the existing require methods for conditional mutations unused, if so then those should be removed.


-- 
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] keith-turner closed issue #3520: Use new requireSame conditional mutation call in more places

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner closed issue #3520: Use new requireSame conditional mutation call in more places
URL: https://github.com/apache/accumulo/issues/3520


-- 
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] keith-turner commented on issue #3520: Use new requireSame conditional mutation call in more places

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner commented on issue #3520:
URL: https://github.com/apache/accumulo/issues/3520#issuecomment-1605067358

   Examined the rest of the code and did not find any places where it made sense to use this new call.  In #3513 it was already used in a few places outside the compaction code and those seems to be it.  The location updates seems like a good candidate, however they did not have  TabletMetadata object handy.


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