DomGarguilo commented on PR #4740:
URL: https://github.com/apache/accumulo/pull/4740#issuecomment-2233667619
Since the scope of this PR has changed slightly since it has been created,
here is an updated description of changes in this PR:
* `AbstractServer.updateIdleStatus()` was changed from having
`Supplier<Boolean>` as its input to just a `boolean`. This was done to simplify
things since the method was just using it as a boolean anyways
* the method was also renamed from `idleProcessCheck()` to
`updateIdleStatus()`
* The idle check in `Compactor` was simplified to just set the idle status
to true or false during its operation. This removes duplicate logic that checks
it has been idle for a period of time (already done in
`AbstractServer.idleProcessCheck()`)
* Adds test cases to ensure that the idle metric for Compactor and
ScanServer behave as expected. Tserver was omitted since it is not expected to
be idle in 2.1.
--
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]
