[
https://issues.apache.org/jira/browse/HBASE-15636?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15238772#comment-15238772
]
Hudson commented on HBASE-15636:
--------------------------------
SUCCESS: Integrated in HBase-1.1-JDK8 #1786 (See
[https://builds.apache.org/job/HBase-1.1-JDK8/1786/])
HBASE-15636 hard coded wait time out value in (syuanjiangdev: rev
b6a5a5164db6b0716107351c2a51154a034edcf2)
* hbase-server/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java
> hard coded wait time out value in
> HBaseTestingUtility#waitUntilAllRegionsAssigned might cause test failure
> ----------------------------------------------------------------------------------------------------------
>
> Key: HBASE-15636
> URL: https://issues.apache.org/jira/browse/HBASE-15636
> Project: HBase
> Issue Type: Bug
> Components: integration tests, test
> Affects Versions: 2.0.0, 1.2.0, 1.3.0, 1.1.4
> Reporter: Stephen Yuan Jiang
> Assignee: Stephen Yuan Jiang
> Priority: Minor
> Fix For: 2.0.0, 1.3.0, 1.4.0, 1.1.5, 1.2.2
>
> Attachments: HBASE-15636.v1-master.patch
>
>
> HBaseTestingUtility#waitUntilAllRegionsAssigned(final TableName tableName)
> hard coded 60 seconds wait for region assignment in a table. This could
> cause flaky tests, as the caller could create table with longs of region
> which takes longs time; or in CM, RS unavailable could slow down assignment;
> or in a slow machine, IT might take long to assign regions.
> In HBaseAdmin.java, we use a configurable value to wait for the table DDLs:
> {{this.syncWaitTimeout =
> this.conf.getInt("hbase.client.sync.wait.timeout.msec", 10 * 60000); //
> 10min}} - we should use the config for the
> HBaseTestingUtility#waitUntilAllRegionsAssigned() too.
> Of course, the caller could use the "public void
> waitUntilAllRegionsAssigned(final TableName tableName, final long timeout)";
> but it requires test change (currently, no test directly calls this
> signature).
> {code}
> /**
> * Wait until all regions for a table in hbase:meta have a non-empty
> * info:server, up to 60 seconds. This means all regions have been deployed,
> * master has been informed and updated hbase:meta with the regions deployed
> * server.
> * @param tableName the table name
> * @throws IOException
> */
> public void waitUntilAllRegionsAssigned(final TableName tableName) throws
> IOException {
> waitUntilAllRegionsAssigned(tableName, 60000);
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
