[master] Use mockito

Author: daviddeng

Summary: Using mockito, `TestMemStoreFlusher` no need to define a mock class implementing some interfaces.

Test Plan: `TestMemStoreFlusher1

Reviewers: adela, manukranthk, gauravm, fan

Reviewed By: adela

CC: hbase-eng@

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

Task ID: 4114747

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589224 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Return Empty string in case when the conf property is non existent on the server

Author: manukranthk

Summary: Returning empty string on the server side in case of non existent properties on the server side.

Test Plan: Run TestServerConfigFromClient

Reviewers: liyintang, daviddeng

Reviewed By: daviddeng

CC: hbase-eng@

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

Task ID: 3922005

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589225 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Fix the namenode port to fix FixTestHLog

Author: manukranthk

Summary: When I changed this to builder pattern last time, I set the namenode port to 0 instead of the given namenode port. Fixing this to run the unit tests successfully on the dev server.

Test Plan: Run TestHLog

Reviewers: daviddeng, liyintang

Reviewed By: daviddeng

CC: hbase-eng@

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

Task ID: 4183494

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589226 13f79535-47bb-0310-9956-ffa450edef68
[master] fix TabpleInputFormatBase$reverseDNS.

Author: everyoung

Summary: Fix reverseDNS function which causes errors in TestHFileOutputFormat

Test Plan: run tests

Reviewers: manukranthk, daviddeng

Reviewed By: manukranthk

CC: aaiyer, hbase-dev@

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

Task ID: 3762458

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589227 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-11046] Add new scanner API definition

Author: daviddeng

Summary: Add a new definition of scanner api for reducing unnecessary RPC calls.

Test Plan: This diff doesn't contain functional change.

Reviewers: liyintang, manukranthk, aaiyer

Reviewed By: manukranthk

CC: hbase-eng@, andrewcox

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

Task ID: 4184037

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589228 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-11021] HBaseFsck Fix for Table Hole Reporting

Author: achayes

Summary:
HBaseFsck would not report when multiple regions went missing. Instead, it
would report the first missing region it found in the meta table and exit.
The new implementation now lists all ranges of [startkey, endkey] that are
missing from the meta table, which should allow for easier debugging.

Test Plan:
TestHBaseFsck is a short test file written to test this functionality. It
sets up a minicluster, creates a table, and then deletes a bunch of rows
from the table and runs the checker. Since the checker does not throw errors
on failure, the output logs have to be checked manually to see if there was
any error. The result looks like this:

https://www.facebook.com/pxlcld/l7xF

Reviewers: manukranthk

Reviewed by: manukranthk

CC:

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

Task ID: #3675218

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589229 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-2000] Fix TestHeapSize after adding coprocessorHost in HRegion and remove all warnings from TestHeapSize.java

Author: adela

Summary: increase the number of class references in HRegion.FIXED_OVERHEAD

Test Plan: ran the test after the change and it passes

Reviewers: daviddeng, manukranthk, elliott

Reviewed By: elliott

CC: hbase-eng@

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

Task ID: 4173054

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589230 13f79535-47bb-0310-9956-ffa450edef68
[Master] Make TestHTableMultiplexer more stable

Author: liyintang

Summary: Make TestHTableMultiplexer more stable

Test Plan: Run through this unit test 10 times

Reviewers: adela, daviddeng, elliott

Reviewed By: elliott

CC: hbase-dev@, elliott

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

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589231 13f79535-47bb-0310-9956-ffa450edef68
[master] Load/reload endpoints in the region server.

Author: daviddeng

Summary:
The endpoint server can load/reload endpoints according to configuration.

In this diff we still use normal class loader to load classes.

Test Plan:
`TestEndpoint`
`TestEndpointReload`

Reviewers: adela, gauravm

Reviewed By: adela

CC: hbase-eng@, andrewcox

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

Task ID: 4157832

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589232 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-9704] Fix TestRegionFavoredNodes by setting dfs.permissions to false.

Author: manukranthk

Summary: We are not concerned about setting permissions for the unit tests. So, we can do the namenode operation here by just turning off the permissions.

Test Plan: Run TestRegionFavoredNodes

Reviewers: liyintang, weiyan, daviddeng

Reviewed By: daviddeng

CC: hbase-eng@, elliott

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

Task ID: 4183494

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1589233 13f79535-47bb-0310-9956-ffa450edef68
[master] Customized Executor for HTableClientScanner

Author: daviddeng

Summary:
`HTableClientScanner` defaultly use `HTable.multiActionThreadPool`
A `Builder` is defined for `HTableClientScanner`
The user can specify an alternative `Executor` in `Builder`

Test Plan: `TestHTableClientScanner`

Reviewers: manukranthk, liyintang, fan

Reviewed By: manukranthk

CC: hbase-eng@

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

Task ID: 4171349, 3546998

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