dlmarion commented on code in PR #4421:
URL: https://github.com/apache/accumulo/pull/4421#discussion_r1554287799
##########
server/base/src/main/java/org/apache/accumulo/server/metadata/ServerAmpleImpl.java:
##########
@@ -76,9 +77,9 @@ public ServerAmpleImpl(ServerContext context) {
}
@Override
- public Ample.TabletMutator mutateTablet(KeyExtent extent) {
+ public Ample.TabletMutator mutateTablet(KeyExtent extent, ServiceLock lock) {
Review Comment:
@cshannon's suggestion:
`Looking at this, is it possible to move the passing of the lock to the
construction of the mutator?`
I'm fine with changing the how/where the lock is supplied. The biggest
issue I'm having right now is I can't figure out why BigRootTabletIT is failing.
--
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-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
