keith-turner commented on code in PR #5187:
URL: https://github.com/apache/accumulo/pull/5187#discussion_r1894421928


##########
core/src/main/java/org/apache/accumulo/core/metadata/schema/Ample.java:
##########
@@ -392,6 +393,8 @@ interface TabletUpdates<T> {
 
     T putCloned();
 
+    T putTabletMergeability(TabletMergeability tabletMergeability);

Review Comment:
   > My original plan was to persist everything as one value by adding the 
current manager time to the value. 
   
   Ok I had thought this would store the exact duration specified by the user.  
Summing the steady time and duration from the user would work.  There may be a 
slight advantage for keeping them separate for debugging purposes as mentioned. 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to