[GitHub] [accumulo] keith-turner opened a new issue, #3356: Always set condition to check prev end row when updating tablets using conditional mutations

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

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

   In the Ample interface for conditionally updating tablets there is a method to require a certain prev end row as a condition of the update.  This condition should always be set when updating tablets.
   
   One possible way to do this is to remove [this method](https://github.com/apache/accumulo/blob/1fd1284afc270366d4dfadb79ee285fab5b060c9/core/src/main/java/org/apache/accumulo/core/metadata/schema/Ample.java#L399) and in the implementation of [this method](https://github.com/apache/accumulo/blob/1fd1284afc270366d4dfadb79ee285fab5b060c9/core/src/main/java/org/apache/accumulo/core/metadata/schema/Ample.java#L256) always set the condition to check the prev end row.
   
   If cases ever arise where a conditional update is needed that does not want to check the prev endrow then an method could be added to the Ample fluent interface to disable adding the condition.  However that could be added when and if its needed.


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

Re: [I] Always set condition to check prev end row when updating tablets using conditional mutations [accumulo]

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

dlmarion closed issue #3356: Always set condition to check prev end row when updating tablets using conditional mutations
URL: https://github.com/apache/accumulo/issues/3356


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