keith-turner commented on issue #4454: URL: https://github.com/apache/accumulo/issues/4454#issuecomment-2067765994
One possible way to handle time ito use existing [steady time]( https://github.com/apache/accumulo/blob/cb25f26ab3be77dbfce54e9d1e7234e5974510ec/server/manager/src/main/java/org/apache/accumulo/manager/Manager.java#L1633) in the manager which is suitable for persisting and using across multiple manager processes. One potential drawback to this approach is that the TabletManagment iterator would need access to the current steady time to pass to the compaction job generator so it can make decisions. One way to handle this would be to add a steadytime parameter to [this class](https://github.com/apache/accumulo/blob/cb25f26ab3be77dbfce54e9d1e7234e5974510ec/server/base/src/main/java/org/apache/accumulo/server/manager/state/TabletManagementParameters.java#L79) that is used to pass information from the manager to the tablet mgmt iterator. Trying this could be explored in its own PR. If it were done, thinking a good prerequisite change would be to introduce a SteadyTime type to replace the long currently returned by the method. If SteadyTime is more widely used a more narrow type in the code would be good for correctness. -- 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]
