keith-turner commented on code in PR #6009:
URL: https://github.com/apache/accumulo/pull/6009#discussion_r2607547321
##########
server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java:
##########
@@ -487,14 +487,14 @@ protected void
updateCompactionState(TExternalCompactionJob job, TCompactionStat
* @param exception cause of failure
* @throws RetriesExceededException thrown when retries have been exceeded
*/
- protected void updateCompactionFailed(TExternalCompactionJob job, Throwable
exception)
+ protected void updateCompactionFailed(TExternalCompactionJob job, String
exception)
Review Comment:
Now that this accepts string could change the other place that calls it and
passes null to instead pass a message. Maybe something like the following.
This would make the log message in the coordinator a bit more informative.
```java
updateCompactionFailed(job, "cancelled");
```
--
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]
