Skip to content

Commit

Permalink
Include fatal errors in trace events
Browse files Browse the repository at this point in the history
patch by jbellis; reviewed by aleksey for CASSANDRA-5447
  • Loading branch information
jbellis committed Apr 9, 2013
1 parent fd588f6 commit 87b350f
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
4 changes: 4 additions & 0 deletions CHANGES.txt
@@ -1,3 +1,7 @@
1.2.5
* Include fatal errors in trace events (CASSANDRA-5447)


1.2.4
* Ensure that PerRowSecondaryIndex updates see the most recent values
(CASSANDRA-5397)
Expand Down
2 changes: 2 additions & 0 deletions src/java/org/apache/cassandra/service/CassandraDaemon.java
Expand Up @@ -41,6 +41,7 @@
import org.apache.cassandra.io.FSError;
import org.apache.cassandra.io.util.FileUtils;
import org.apache.cassandra.thrift.ThriftServer;
import org.apache.cassandra.tracing.Tracing;
import org.apache.cassandra.utils.CLibrary;
import org.apache.cassandra.utils.Mx4jTool;

Expand Down Expand Up @@ -172,6 +173,7 @@ public void uncaughtException(Thread t, Throwable e)
{
exceptions.incrementAndGet();
logger.error("Exception in thread " + t, e);
Tracing.trace("Exception in thread " + t, e);
for (Throwable e2 = e; e2 != null; e2 = e2.getCause())
{
// some code, like FileChannel.map, will wrap an OutOfMemoryError in another exception
Expand Down

0 comments on commit 87b350f

Please sign in to comment.