accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Drob" <md...@mdrob.com>
Subject Review Request 18365: ACCUMULO-2390 InvocationTargetEx in TraceProxy
Date Fri, 21 Feb 2014 19:11:12 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18365/
-----------------------------------------------------------

Review request for accumulo.


Bugs: ACCUMULO-2390
    https://issues.apache.org/jira/browse/ACCUMULO-2390


Repository: accumulo


Description
-------

ACCUMULO-2390 InvocationTargetEx in TraceProxy

Handle InvocationTargetException specifically in TraceProxy, instead
of letting it get propogated up the call stack. In some cases this is
very bad as it turned into an UndeclaredThrowableException and made
debugging more difficult than necessary.

Added unit test to verify behaviour.


Diffs
-----

  src/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/TraceProxy.java 67c4463429fdacdb4aae1d07d5256a5a4f077552

  src/trace/src/test/java/org/apache/accumulo/cloudtrace/instrument/TracerTest.java 30b62f9f7d4c0a047a33fd0244b3acd57733b5b1


Diff: https://reviews.apache.org/r/18365/diff/


Testing
-------

Unit tests run.


Thanks,

Mike Drob


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message