dlmarion commented on code in PR #5169:
URL: https://github.com/apache/accumulo/pull/5169#discussion_r1882045395
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/MinorCompactor.java:
##########
@@ -93,7 +94,18 @@ public CompactionStats call() {
try {
do {
try {
- CompactionStats ret = super.call();
+ CompactionStats ret = null;
+ try {
+ ret = super.call();
+ } catch (Exception e) {
+ if (tabletServer.getLock() == null ||
!tabletServer.getLock().verifyLockAtSource()) {
+ log.error("Minor compaction of {} has failed and TabletServer
lock does not exist."
+ + " Halting...", getExtent(), e);
+ Halt.halt("TabletServer lock does not exist", -1);
+ } else {
+ throw e;
+ }
+ }
// log.debug(String.format("MinC %,d recs in | %,d recs out | %,d
recs/sec | %6.3f secs |
Review Comment:
Curious why these log messages are commented out.
--
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]
