[ https://issues.apache.org/jira/browse/HBASE-8706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13681794#comment-13681794 ]
Hudson commented on HBASE-8706: ------------------------------- Integrated in hbase-0.95-on-hadoop2 #131 (See [https://builds.apache.org/job/hbase-0.95-on-hadoop2/131/]) HBASE-8706 Some improvement in snapshot (binlijin) (Revision 1492399) Result = FAILURE mbertozzi : Files : * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ProcedureCoordinator.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ProcedureMember.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/Subprocedure.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/snapshot/RegionServerSnapshotManager.java * /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/procedure/TestProcedureCoordinator.java * /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/procedure/TestProcedureMember.java * /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/procedure/TestZKProcedure.java > Some improvement in snapshot > ---------------------------- > > Key: HBASE-8706 > URL: https://issues.apache.org/jira/browse/HBASE-8706 > Project: HBase > Issue Type: Bug > Components: snapshots > Affects Versions: 0.94.8, 0.95.0 > Reporter: binlijin > Assignee: Matteo Bertozzi > Fix For: 0.98.0, 0.95.2 > > Attachments: HBASE-8706-2.patch, HBASE-8706-3.patch, > HBASE-8706.patch, HBASE-8706-v4.patch, HBASE-8706-v4.patch > > > (1)timeout for Procedure can not be configured. > {code} > Procedure's timeout > ProcedureCoordinator > final static long TIMEOUT_MILLIS_DEFAULT = 60000; > createProcedure(ForeignExceptionDispatcher fed, String procName, byte[] > procArgs, > List<String> expectedMembers) { > // build the procedure > return new Procedure(this, fed, WAKE_MILLIS_DEFAULT, > TIMEOUT_MILLIS_DEFAULT, > procName, procArgs, expectedMembers); > } > RegionServerSnapshotManager: > /** Conf key for max time to keep threads in snapshot request pool waiting > */ > public static final String SNAPSHOT_TIMEOUT_MILLIS_KEY = > "hbase.snapshot.region.timeout"; > /** Keep threads alive in request pool for max of 60 seconds */ > public static final long SNAPSHOT_TIMEOUT_MILLIS_DEFAULT = 60000; > public Subprocedure buildSubprocedure(SnapshotDescription snapshot) { > long timeoutMillis = conf.getLong(SNAPSHOT_TIMEOUT_MILLIS_KEY, > SNAPSHOT_TIMEOUT_MILLIS_DEFAULT); > case FLUSH: > SnapshotSubprocedurePool taskManager = > new SnapshotSubprocedurePool(rss.getServerName().toString(), conf); > } > {code} > (2)TakeSnapshotHandler > after snapshotRegions we should call monitor.rethrowException(); to check if > there is exception and if there is we can skip the verifySnapshot > (3)too much error message when error happened in some place. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira