[jira] [HBASE-6882] [89-fb] Thrift IOError should include exception class
ClosedPublic

Press ? to show keyboard shortcuts.
Author
mbautin
Reviewers
Liyin
Karthik
aaiyer
chip
JIRA
Lint
Lint Warnings
Excusegenerated code
src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java
Warning(TXT3) Line Too Long at line 40
Warning(TXT3) Line Too Long at line 126
Warning(TXT3) Line Too Long at line 127
Warning(TXT3) Line Too Long at line 382
Warning(TXT3) Line Too Long at line 387
Show Full Lint Results (5 Details)
Unit
Unit Tests Skipped
Commits
Unknown Object (Diffusion Commit)
Unknown Object (Diffusion Commit)
Branch
ioerror_class_name
Apply Patch
arc patch D5679
Arcanist Project
Restricted Arcanist Project
Subscribers
None
Projects
None
Summary

Return exception class as part of IOError thrown from the Thrift proxy or the embedded Thrift server in the regionserver.

Test Plan

Unit tests
Test through C++ HBase client

Liyin accepted this revision.Via LegacySep 25 2012, 10:34 PM

LGTM !

mbautin closed this revision.Via LegacySep 27 2012, 9:14 PM

Revision Update History

DiffIDBaseDescriptionCreatedLintUnit
BaseBase
Diff 1186691389842Sep 25 2012, 8:38 PM
Diff 2188431391220Commit rHBASEEIGHTNINEFBBRANCH1391221Sep 27 2012, 9:07 PM

Local Commits

CommitTreeParentsAuthorSummaryDate
c3688b03fd6f197b5a794ee30bc24e8a8355mbautin
IOError should include class name
Sep 25 2012, 8:31 PM

Diff 18843

src/main/java/org/apache/hadoop/hbase/RegionException.java

Loading...

src/main/java/org/apache/hadoop/hbase/regionserver/HRegionThriftServer.java

Loading...

src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java

Loading...

src/main/java/org/apache/hadoop/hbase/thrift/generated/IOError.java

Loading...

src/main/resources/org/apache/hadoop/hbase/thrift/Hbase.thrift

Loading...

Add Comment