[ https://issues.apache.org/jira/browse/HBASE-4014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13065612#comment-13065612 ]
jirapos...@reviews.apache.org commented on HBASE-4014: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/969/ ----------------------------------------------------------- (Updated 2011-07-14 23:39:07.065843) Review request for hbase, Gary Helmling and Mingjie Lai. Changes ------- Addresses Mingjie Lai's and Ted Yu's comments. Summary ------- https://issues.apache.org/jira/browse/HBASE-4014 Coprocessors: Flag the presence of coprocessors in logged exceptions The general gist here is to wrap each of {Master,RegionServer}CoprocessorHost's coprocessor call inside a "try { ... } catch (Throwable e) { handleCoprocessorThrowable(e) }" block. handleCoprocessorThrowable() is responsible for either passing 'e' along to the client (if 'e' is an IOException) or, otherwise, aborting the service (Regionserver or Master). The abort message contains a list of the loaded coprocessors for crash analysis. This addresses bug HBASE-4014. https://issues.apache.org/jira/browse/HBASE-4014 Diffs (updated) ----- src/main/java/org/apache/hadoop/hbase/master/MasterCoprocessorHost.java aa930f5 src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorEnvironment.java 54ccd6f src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java 18ba6e7 src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java c2b3558 src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerServices.java 8ffa086 src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWatcher.java 4fa82c0 src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java 6af0ecf src/test/java/org/apache/hadoop/hbase/coprocessor/BuggyRegionObserver.java PRE-CREATION src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterCoprocessorException.java PRE-CREATION src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionServerCoprocessorException.java PRE-CREATION src/test/java/org/apache/hadoop/hbase/master/TestActiveMasterManager.java 7f19c72 src/test/java/org/apache/hadoop/hbase/master/TestCatalogJanitor.java 78e7d62 src/test/java/org/apache/hadoop/hbase/master/TestClockSkewDetection.java c571227 src/test/java/org/apache/hadoop/hbase/master/TestLogsCleaner.java fc05e47 src/test/java/org/apache/hadoop/hbase/regionserver/handler/TestOpenRegionHandler.java aa48c22 src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java dd5dc3e Diff: https://reviews.apache.org/r/969/diff Testing ------- patch includes two tests: TestMasterCoprocessorException.java TestRegionServerCoprocessorException.java both tests pass in my build environment. Thanks, Eugene > Coprocessors: Flag the presence of coprocessors in logged exceptions > -------------------------------------------------------------------- > > Key: HBASE-4014 > URL: https://issues.apache.org/jira/browse/HBASE-4014 > Project: HBase > Issue Type: Improvement > Components: coprocessors > Reporter: Andrew Purtell > Assignee: Eugene Koontz > Fix For: 0.92.0 > > Attachments: HBASE-4014.patch, HBASE-4014.patch, HBASE-4014.patch, > HBASE-4014.patch > > > For some initial triage of bug reports for core versus for deployments with > loaded coprocessors, we need something like the Linux kernel's taint flag, > and list of linked in modules that show up in the output of every OOPS, to > appear above or below exceptions that appear in the logs. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira