SDCISA-18302: Add logging in case unreachable code gets reached.

swisspost · Dec 10, 2024 · 333bea5 · 333bea5
1 parent 447c105
commit 333bea5
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/main/java/org/swisspush/redisques/performance/UpperBoundParallel.java b/src/main/java/org/swisspush/redisques/performance/UpperBoundParallel.java
@@ -132,7 +132,11 @@ private <Ctx> void resume(Request<Ctx> req) {
                         // this boolean is just for paranoia, in case mentor tries to call back too often.
                         final AtomicBoolean isCalled = new AtomicBoolean();
                         @Override public void accept(Throwable ex, Void ret) {
-                            if (!isCalled.compareAndSet(false, true)) return;
+                            if (!isCalled.compareAndSet(false, true)) {
+                                if (log.isDebugEnabled()) log.warn("This callback MUST NOT be called multiple times!! Make sure caller only calls it ONCE!", new Exception("stacktrace"));
+                                else log.warn("This callback MUST NOT be called multiple times!! Make sure caller only calls it ONCE! (enable debug log to see stack)");
+                                return;
+                            }
                             onOneDone(req, ex);
                         }
                     }, req.ctx);