cshannon commented on code in PR #5187:
URL: https://github.com/apache/accumulo/pull/5187#discussion_r1903121181
##########
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:
@keith-turner - The latest update
[adds](https://github.com/apache/accumulo/pull/5187/files#diff-d0c46782a2fa101a9e218b4e37c26a18c066e0745128fb351b08b4e8f1480483R35)
a new class just to wrap the metadata and make it easier to validate and store
the delay + the steady time as a separate value. The intent is that the the
client will still only use and pass
[TabletMergeability](https://github.com/apache/accumulo/pull/5187/files#diff-b7655aae54212259b0644ce511c60e22e39b1f9122701f427a176587477921b3R27)
because only the Manager will be touching `TabletMergeabilityMetadata` as it
just exists in Ample. A user would just set the `TabletMergeability` and if a
delay is set then the Manager will set the SteadyTime value when inserting
`TabletMergeabilityMetadata`.
--
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]
