[HBASE-10450] Fix the Thrift test cases

Author: rshroff

Summary:
The TestThriftServer is the base class for testing all HBase operations
from thrift via different thrift servers. Hence a failure in TestThriftServer
fails multiple test cases. Also TestThriftServer takes the Hbase.Iface as
an input to perform all the operations. The Hbase.Iface can either be the standard
HBase thrift client or the ThriftServerRunner.HBaseHandler (which acts as an
intermediate between the client and the hbase client). The bug is that
the TestThriftServer now assumes the Hbase.Iface to be always of type HBaseHandler,
which causes the problem. Hence, removing it.

Test Plan: run unit tests.

Reviewers: manukranthk, daviddeng

Reviewed By: daviddeng

CC: hbase-eng@

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

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1572205 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-7099] [89-fb]Exception handling during compaction hook execution in Store

Author: adela

Summary:
Compaction failed completely on tsh58 due to exceptions in CompactionHook implementation
https://phabricator.fb.com/P6308922

Test Plan: updated TestLowerToUpperCompaction hook unit test to cover the case

Reviewers: rshroff, aaiyer, manukranthk, liyintang

Reviewed By: rshroff

CC: hbase-eng@

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

Task ID: 2095744

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1572206 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-10538] Fix testcase TestBlocksRead

Author: daviddeng

Summary: In `ScanQueryMatchr.moreRowsMayExistAfter`, a line is added espcially for get-scan. This is an optimization to reduce a read call and thus survive the testcase.

Test Plan: Run TestBlocksRead

Reviewers: manukranthk, rshroff, liyintang

Reviewed By: rshroff

CC: hbase-eng@

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

Task ID: 3790918

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1572207 13f79535-47bb-0310-9956-ffa450edef68
[HBASE-10538] Fix TestStoreFile and TestCompoundBloomFilter

Author: daviddeng

Summary: A mis-use of `Scan`, add `Bytes.nextOf` to `stopRow`

Test Plan: Run `TestStoreFile and` `TestCompoundBloomFilter`

Reviewers: manukranthk, rshroff, liyintang

Reviewed By: rshroff

CC: hbase-eng@

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

Task ID: 3789093

git-svn-id: https://svn.apache.org/repos/asf/hbase/branches/0.89-fb@1572208 13f79535-47bb-0310-9956-ffa450edef68
[0.89-fb] [HBASE-10291] Fix TestFromClientSide4

Author: arjen

Summary: Several of the tests in TestFromClientSide4 put some assertions on cache behavior and would test these using metrics. These assumptions changed however in D1112816, causing these tests to fail.

Test Plan: TestFromClientSide4 tests pass.

Reviewers: manukranthk, daviddeng

Reviewed By: manukranthk

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

Task ID: 3762230

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