You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/09/07 13:12:47 UTC

[GitHub] [accumulo] cshannon commented on pull request #2799: Adding support for setting multiple properties at once atomically

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

   @keith-turner - The retry was removed as it was discussed and determined to not be necessary or wanted and should just fail instead. I forget when but I believe it was an offline conversation with @ctubbsii and others.
   
   A separate PR to update the shell would be a good idea if we want to do that to keep the scope of this PR smaller as it's already a decent sized change. I could add that as a follow on once his is merged.


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