Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-1098 IllegalStateException: Cannot set a region to be closed it it was not already marked as closing; debugging patch 

git-svn-id: https://svn.apache.org/repos/asf/hadoop/hbase/trunk@730232 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Michael Stack 2008-12-30 20:35:20 +00:00
parent e4db939ffd
commit 127491e583
2 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/java/org/apache/hadoop/hbase/master/HMaster.java
View File

@ -420,7 +420,7 @@ public class HMaster extends Thread implements HConstants, HMasterInterface,
try {
if (LOG.isDebugEnabled()) {
LOG.debug("Main processing loop: " + op.toString());
LOG.debug("Processing todo: " + op.toString());
}
// perform the operation.

21
src/java/org/apache/hadoop/hbase/master/RegionManager.java
View File

@ -1073,8 +1073,8 @@ class RegionManager implements HConstants {
synchronized void setAssigned(String serverName) {
if (!this.unassigned) {
throw new IllegalStateException(
"Cannot assign a region that is not currently unassigned. Region: " +
Bytes.toString(regionName));
"Cannot assign a region that is not currently unassigned. " +
"State: " + toString());
}
this.unassigned = false;
this.assigned = true;
@ -1090,8 +1090,8 @@ class RegionManager implements HConstants {
synchronized void setPending() {
if (!assigned) {
throw new IllegalStateException(
"Cannot set a region as pending if it has not been assigned. Region: " +
Bytes.toString(regionName));
"Cannot set a region as pending if it has not been assigned. " +
"State: " + toString());
}
this.unassigned = false;
this.assigned = false;
@ -1120,8 +1120,9 @@ class RegionManager implements HConstants {
if (!closing) {
throw new IllegalStateException(
"Cannot set a region to be closed if it was not already marked as" +
" closing. Region: " + Bytes.toString(regionName));
" closing. State: " + toString());
}
this.closing = false;
this.closed = true;
}
@ -1131,10 +1132,10 @@ class RegionManager implements HConstants {
@Override
public synchronized String toString() {
return "region name: " + Bytes.toString(this.regionName) +
", isUnassigned: " + this.unassigned + ", isAssigned: " +
this.assigned + ", isPending: " + this.pending + ", isClosing: " +
this.closing + ", isClosed: " + this.closed + ", isOfflined: " +
return "name=" + Bytes.toString(this.regionName) +
", isUnassigned=" + this.unassigned + ", isAssigned=" +
this.assigned + ", isPending=" + this.pending + ", isClosing=" +
this.closing + ", isClosed=" + this.closed + ", isOfflined=" +
this.offlined;
}
@ -1156,4 +1157,4 @@ class RegionManager implements HConstants {
return Bytes.compareTo(this.regionName, o.getRegionName());
}
}
}
}