[Master] Fix TestRegionServerMetrics

Author: adela

Summary:
disable & drop table after unit.
testMultipleRegions is relying on number of regions - which means if that unit is not executed first number of regions will increase and test will fail.
Dropping table created in each unit to clear that out

Test Plan:
reshuffle test order execution

  @Test
  public void test() throws Exception {
    testGetNextSize();
    testCompactionWriteMetric();
    testMultipleRegions();
    testEncodingInCache();
  }

Reviewers: manukranthk, daviddeng, elliott

Reviewed By: manukranthk

CC: hbase-eng@

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

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1590089 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] RowLockID being positive is not guarenteed in the code; Removing the assert.

Author: manukranthk

Summary: TestSimpleOperations#testRowLock checks id the rowlock id is positive which seems like an invalid check to me. Removing it.

Test Plan: Run TestSimpleOperations#testRowLock

Reviewers: rshroff, liyintang, adela, elliott, daviddeng

Reviewed By: adela

CC: hbase-eng@

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

Task ID: 4202192

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1590090 13f79535-47bb-0310-9956-ffa450edef68
[master] make TestLogSplitOnMasterFailover more stable

Author: daviddeng

Summary:
useAssignmentPlan
Increase the timeout

Test Plan: `TestLogSplitOnMasterFailover`

Reviewers: manukranthk, liyintang, gauravm

Reviewed By: manukranthk

CC: hbase-eng@

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

Task ID: 3376780

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1590091 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Revert the part increment table id in https://phabricator.fb.com/D1291992

Author: manukranthk

Summary: That change might cause unintended side effects as noted by adela in https://phabricator.fb.com/D1294255. So, reverting the same.

Test Plan: Run TestSimpleOperations

Reviewers: adela

Reviewed By: adela

CC: hbase-eng@

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

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