[
https://issues.apache.org/jira/browse/HBASE-2691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876851#action_12876851
]
HBase Review Board commented on HBASE-2691:
-------------------------------------------
Message from: "Jean-Daniel Cryans" <[email protected]>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/164/
-----------------------------------------------------------
Review request for hbase.
Summary
-------
Patch that first adds a new testing facility to send exceptions to region
servers when they report in.
Then I added YouAreDeadException and PleaseHoldException that replaces the
ambiguous lease still held exception, and both are treated differently.
Finally, I added a very simple test that makes sure that telling a RS that its
dead will actually end up in killing it.
This addresses bug HBASE-2691.
Diffs
-----
/trunk/src/main/java/org/apache/hadoop/hbase/PleaseHoldException.java
PRE-CREATION
/trunk/src/main/java/org/apache/hadoop/hbase/YouAreDeadException.java
PRE-CREATION
/trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 952836
/trunk/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 952836
/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
952836
/trunk/src/test/java/org/apache/hadoop/hbase/MiniHBaseCluster.java 952836
/trunk/src/test/java/org/apache/hadoop/hbase/TestYouAreDead.java PRE-CREATION
Diff: http://review.hbase.org/r/164/diff
Testing
-------
Thanks,
Jean-Daniel
> LeaseStillHeldException totally ignored by RS, wrongly named
> ------------------------------------------------------------
>
> Key: HBASE-2691
> URL: https://issues.apache.org/jira/browse/HBASE-2691
> Project: HBase
> Issue Type: Bug
> Reporter: Jean-Daniel Cryans
> Assignee: Jean-Daniel Cryans
> Fix For: 0.20.6, 0.21.0
>
>
> Currently region servers don't handle
> org.apache.hadoop.hbase.Leases$LeaseStillHeldException in any way that's
> useful so what happens right now is that it tries to report to the master and
> this happens:
> {code}
> 2010-06-07 17:20:54,368 WARN [RegionServer:0]
> regionserver.HRegionServer(553): Attempt=1
> org.apache.hadoop.hbase.Leases$LeaseStillHeldException
> at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native
> Method)
> at
> sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39)
> at
> sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
> at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
> at
> org.apache.hadoop.hbase.RemoteExceptionHandler.decodeRemoteException(RemoteExceptionHandler.java:94)
> at
> org.apache.hadoop.hbase.RemoteExceptionHandler.checkThrowable(RemoteExceptionHandler.java:48)
> at
> org.apache.hadoop.hbase.RemoteExceptionHandler.checkIOException(RemoteExceptionHandler.java:66)
> at
> org.apache.hadoop.hbase.regionserver.HRegionServer.run(HRegionServer.java:541)
> at
> org.apache.hadoop.hbase.MiniHBaseCluster$MiniHBaseClusterRegionServer.run(MiniHBaseCluster.java:173)
> at java.lang.Thread.run(Thread.java:637)
> {code}
> Then it will retry until the watch is triggered telling it that the session's
> expired! Instead, we should be a lot more proactive initiate abort procedure.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
