Skip to content

Commit 0835c5a

Browse files
committed
add exception handling comment to every catch block
1 parent e944b27 commit 0835c5a

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

README.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,7 @@ try {
173173
// make the call and transport the tag across to server
174174
} catch (Throwable e) {
175175
outgoingRemoteCall.error(e);
176+
// rethrow or add your exception handling
176177
} finally {
177178
outgoingRemoteCall.end();
178179
}
@@ -324,6 +325,7 @@ try {
324325

325326
} catch (Exception e) {
326327
outgoingWebRequestTracer.error(e);
328+
// rethrow or add your exception handling
327329
} finally {
328330
outgoingWebRequestTracer.end();
329331
}
@@ -357,7 +359,7 @@ try {
357359
outgoingMessageTracer.setCorrelationId(toSend.correlationId);
358360
} catch (Exception e) {
359361
outgoingMessageTracer.error(e.getMessage());
360-
Logger.logError(e);
362+
// rethrow or add your exception handling
361363
} finally {
362364
outgoingMessageTracer.end();
363365
}
@@ -395,7 +397,7 @@ while(true) {
395397
}
396398
} catch (Exception e) {
397399
incomingMessageReceiveTracer.error(e.getMessage());
398-
Logger.logError(e);
400+
// rethrow or add your exception handling
399401
} finally {
400402
incomingMessageReceiveTracer.end();
401403
}
@@ -421,7 +423,7 @@ public void onMessage(Message message) {
421423
// do the work ...
422424
} catch (Exception e) {
423425
incomingMessageProcessTracer.error(e.getMessage());
424-
Logger.logError(e);
426+
// rethrow or add your exception handling
425427
} finally {
426428
incomingMessageProcessTracer.end();
427429
}

0 commit comments

Comments
 (0)