[HBASE-10502] Simply ParallelScanner

Author: daviddeng

Summary: HTableClientScanner now fetches data on the background, `ParallelScanner` can be simplified.

Test Plan: `TestParallelScanner`

Reviewers: liyintang, manukranthk, elliott

Reviewed By: elliott

CC: hbase-eng@, elliott

Differential Revision: https://phabricator.fb.com/D1312051

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594425 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Fail the client creation if it fails for more than rpcTimeout

Author: manukranthk

Summary: While creating the thrift client, the get() on the future waits indefinitely. Instead, waiting with a timeout equal to the RPC timeout makes more sense. Ideally, this should be done internally in thrift.

Test Plan: Test on calypsohbase001-frc1

Reviewers: rshroff, adela, daviddeng

Reviewed By: daviddeng

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1314459

Task ID: 4253558

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594426 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Set caching in TableServers.testTableOnlineState to the value set in the scan.

Author: manukranthk

Summary: TableServers.testTableOnlineState used to loop around looking for 1 row at a time which is very sub optimal.

Test Plan: Simple change. Tested on calypso.

Reviewers: adela, daviddeng, rshroff

Reviewed By: rshroff

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1315129

Task ID: 4253558

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594427 13f79535-47bb-0310-9956-ffa450edef68
[master] Support startRow and stopRow in endpoint.

Author: daviddeng

Summary:
Checking `startRow` and `stopRow` before calling.
Add new case for this.

Test Plan: `TestLongAggregator`

Reviewers: adela

Reviewed By: adela

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1308056

Task ID: 4223215

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594428 13f79535-47bb-0310-9956-ffa450edef68
[master] Move TestLogRolling to JUnit4 so that surefire will respect unstable tags.

Author: elliott

Summary: Moved TestLogRolling to JUnit4

Test Plan: mvn clean test -Dtest=TestLogRolling doesn't run any test.

Reviewers: daviddeng

Reviewed By: daviddeng

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1316330

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594429 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-11138][0.89-fb] Check for an empty split key before creating a table

Author: gauravm

Summary: HBaseAdmin#checkSplitKeys() doesn't check for empty split keys, which can cause multiple regions with the same start key (i.e., ""). This diff fixes that.

Test Plan: Added a unit test

Reviewers: rshroff, manukranthk, aaiyer, daviddeng

Reviewed By: daviddeng

CC: hbase-eng@, daviddeng

Differential Revision: https://phabricator.fb.com/D1316709

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594430 13f79535-47bb-0310-9956-ffa450edef68
[master] Fix TestHTableMultiplexer.testCachedOfHTable

Author: daviddeng

Summary:
`TestHTableMultiplexer.testCachedOfHTable` assumes a table to be existing named `testCounters`, this is a bad and wrong assumption.
Change the name of the table to a unique one, create that table.

Test Plan: `TestHTableMultiplexer`

Reviewers: elliott, manukranthk, gauravm, adela

Reviewed By: adela

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1322620

Task ID: 4311486

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594431 13f79535-47bb-0310-9956-ffa450edef68
[master] Add log for analyzing why sometimes table cannot be created

Author: daviddeng

Summary: We can know how long as a whole does it take for checking table success or not.

Test Plan: None.

Reviewers: manukranthk, elliott, adela, gauravm

Reviewed By: adela

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1322997

Task ID: 3296253

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594432 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Create a copy of the conf in ClientLocalScanner since we are modifying it

Author: manukranthk

Summary: In ClientLocalScanner.openStoresOnClient, we modify the conf to set the fs.default.name to be the value obtained from the hbase cluster the htable is create for. But, doing so will modify the conf passed in. In case the user intends to use the config later, it might have unknown effects.

Test Plan: Trivial change. Hot fix for search005

Reviewers: daviddeng

Reviewed By: daviddeng

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1325499

Task ID: 4317015

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594433 13f79535-47bb-0310-9956-ffa450edef68
[master] Back compatibilty with old servers

Author: daviddeng

Summary:
In servers before Altair, if we make a scan with equal non empty start/stop rows, it returns the row instead of returnning nothing. So we need to check on the client side.

This has been fixed in Bellatrix.

Test Plan: `TestHTableClientScanner`

Reviewers: manukranthk

Reviewed By: manukranthk

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D1325711

Task ID: 4324015

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1594434 13f79535-47bb-0310-9956-ffa450edef68