subject:"\[GitHub\] \[hbase\] Apache\-HBase commented on issue #477\: HBASE\-22834 Removed deprecated methods from HBaseTestingUtility"

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-09-01 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-527003835
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 45 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 366 | master passed |
   | +1 | compile | 58 | master passed |
   | +1 | checkstyle | 87 | master passed |
   | +1 | shadedjars | 279 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 241 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 239 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 327 | the patch passed |
   | +1 | compile | 60 | the patch passed |
   | +1 | javac | 60 | the patch passed |
   | +1 | checkstyle | 85 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 266 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 925 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 34 | the patch passed |
   | +1 | findbugs | 252 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18697 | hbase-server in the patch failed. |
   | -1 | asflicense | 33 | The patch generated 18 ASF License warnings. |
   | | | 22173 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestAcidGuaranteesWithEagerPolicy |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureBackoff |
   |   | hadoop.hbase.regionserver.TestRegionServerReportForDuty |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.client.TestAsyncDecommissionAdminApi |
   |   | hadoop.hbase.master.TestWarmupRegion |
   |   | hadoop.hbase.client.TestAvoidCellReferencesIntoShippedBlocks |
   |   | hadoop.hbase.snapshot.TestRestoreSnapshotHelper |
   |   | hadoop.hbase.master.TestCatalogJanitorInMemoryStates |
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.client.TestMultiRespectsLimits |
   |   | hadoop.hbase.ipc.TestNettyRpcServer |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasScan |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.replication.regionserver.TestReplicator |
   |   | hadoop.hbase.client.TestAsyncTableBatchRetryImmediately |
   |   | hadoop.hbase.TestJMXListener |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicasWithoutZKCoordinated
 |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocator |
   |   | hadoop.hbase.security.visibility.TestWithDisabledAuthorization |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.client.TestResultSizeEstimation |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | hadoop.hbase.master.TestMasterOperationsForRegionReplicas |
   |   | hadoop.hbase.util.TestFSUtils |
   |   | hadoop.hbase.mob.TestMobWithByteBuffAllocator |
   |   | hadoop.hbase.client.TestAppendFromClientSide |
   |   | hadoop.hbase.client.TestReplicasClient |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | hadoop.hbase.security.visibility.TestEnforcingScanLabelGenerator |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.master.TestMasterHandlerFullWhenTransitRegion |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.TestRegionRebalancing |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocatorConcurrenyLimit |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.client.TestMetaWithReplicas |
   |   | hadoop.hbase.client.TestFromClientSide |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.quotas.TestQuotaObserverChoreRegionReports |
   |   | 
hadoop.hbase.master.procedure.TestTableDescriptorModificationFromClient |
   |   | hadoop.hbase.regionserver.TestRegionServerAbort |
   |   | hadoop.hbase.filter.TestFuzzyRowFilterEndToEnd

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-09-01 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-526948555
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 82 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 367 | master passed |
   | +1 | compile | 65 | master passed |
   | +1 | checkstyle | 101 | master passed |
   | +1 | shadedjars | 324 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 40 | master passed |
   | 0 | spotbugs | 288 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 286 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 338 | the patch passed |
   | +1 | compile | 62 | the patch passed |
   | +1 | javac | 62 | the patch passed |
   | +1 | checkstyle | 99 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 310 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1216 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 45 | the patch passed |
   | +1 | findbugs | 357 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20227 | hbase-server in the patch failed. |
   | -1 | asflicense | 26 | The patch generated 33 ASF License warnings. |
   | | | 24440 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.regionserver.TestRegionServerReportForDuty |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.security.visibility.TestEnforcingScanLabelGenerator |
   |   | hadoop.hbase.io.asyncfs.TestFanOutOneBlockAsyncDFSOutput |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.security.access.TestScanEarlyTermination |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.quotas.TestQuotaAdmin |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi2 |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleWAL
 |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelReplicationWithExpAsString |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.regionserver.TestHRegionOnCluster |
   |   | hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureBackoff |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | hadoop.hbase.client.TestResultScannerCursor |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpointNoMaster
 |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.client.TestDropTimeoutRequest |
   |   | hadoop.hbase.replication.TestReplicationProcedureRetry |
   |   | hadoop.hbase.coprocessor.TestRegionObserverBypass |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.regionserver.TestHdfsSnapshotHRegion |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.client.TestRawAsyncTableScan |
   |   | hadoop.hbase.regionserver.TestOpenSeqNumUnexpectedIncrease |
   |   | hadoop.hbase.regionserver.TestRowTooBig |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.regionserver.TestCompactionState |
   |   | hadoop.hbase.master.procedure.TestEnableTableProcedure |
   |   | hadoop.hbase.regionserver.TestScannerWithBulkload |
   |   | hadoop.hbase.quotas.TestSnapshotQuotaObserverChore |
   |   | hadoop.hbase.master.TestMergeTableRegionsWhileRSCrash |
   |   | hadoop.hbase.regionserver.TestJoinedScanners |
   |   | hadoop.hbase.master.cleaner.TestHFileCleaner |
   |   | hadoop.hbase.regionserver.wal.TestSecureAsyncWALReplay |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | 
hadoop.hbase.master.replication.TestTransitPeerSyncReplicationStateProcedureBackoff
 |
   |   | hadoop.hbase.regionserver.TestPerColumnFamilyFlush |
   |   | hadoop.hbase.regionserver.TestRemoveRegionMetrics |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-31 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-526884410
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 41 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 408 | master passed |
   | +1 | compile | 77 | master passed |
   | +1 | checkstyle | 112 | master passed |
   | +1 | shadedjars | 310 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 260 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 258 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 323 | the patch passed |
   | +1 | compile | 59 | the patch passed |
   | +1 | javac | 59 | the patch passed |
   | +1 | checkstyle | 84 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 268 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 961 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 37 | the patch passed |
   | +1 | findbugs | 253 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20292 | hbase-server in the patch failed. |
   | -1 | asflicense | 34 | The patch generated 18 ASF License warnings. |
   | | | 23956 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting |
   |   | hadoop.hbase.client.TestAdmin1 |
   |   | hadoop.hbase.wal.TestWALSplitBoundedLogWriterCreation |
   |   | hadoop.hbase.regionserver.throttle.TestFlushWithThroughputController |
   |   | 
hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterSplittingRegions |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.regionserver.wal.TestProtobufLog |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.TestPartialResultsFromClientSide |
   |   | hadoop.hbase.client.TestCIPutOperationTimeout |
   |   | hadoop.hbase.client.TestAlwaysSetScannerId |
   |   | hadoop.hbase.quotas.TestSpaceQuotaRemoval |
   |   | hadoop.hbase.client.TestBlockEvictionFromClient |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.replication.TestSyncReplicationStandBy |
   |   | hadoop.hbase.master.TestMetaShutdownHandler |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.client.TestAvoidCellReferencesIntoShippedBlocks |
   |   | hadoop.hbase.quotas.TestSpaceQuotaOnBulkLoad |
   |   | hadoop.hbase.coprocessor.TestMetaTableMetrics |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.TestAcidGuaranteesWithNoInMemCompaction |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.master.TestShutdownWithNoRegionServer |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.mob.TestExpiredMobFileCleaner |
   |   | hadoop.hbase.client.TestAsyncClientPauseForCallQueueTooBig |
   |   | hadoop.hbase.master.TestSplitWALManager |
   |   | hadoop.hbase.regionserver.TestRegionServerCrashDisableWAL |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.replication.TestSyncReplicationActive |
   |   | hadoop.hbase.regionserver.TestHRegionFileSystem |
   |   | hadoop.hbase.TestInfoServers |
   |   | hadoop.hbase.client.TestAsyncTableRSCrashPublish |
   |   | hadoop.hbase.TestClientClusterStatus |
   |   | hadoop.hbase.security.access.TestWithDisabledAuthorization |
   |   | hadoop.hbase.client.TestHbck |
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.coprocessor.TestWALObserver |
   |   | hadoop.hbase.master.TestMaster |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi2 |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-31 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-526863179
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 87 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 366 | master passed |
   | +1 | compile | 60 | master passed |
   | +1 | checkstyle | 97 | master passed |
   | +1 | shadedjars | 301 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 43 | master passed |
   | 0 | spotbugs | 278 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 276 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 338 | the patch passed |
   | +1 | compile | 61 | the patch passed |
   | +1 | javac | 61 | the patch passed |
   | +1 | checkstyle | 97 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 303 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1073 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 38 | the patch passed |
   | +1 | findbugs | 282 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20655 | hbase-server in the patch failed. |
   | -1 | asflicense | 26 | The patch generated 25 ASF License warnings. |
   | | | 24483 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | hadoop.hbase.client.TestScannersFromClientSide2 |
   |   | hadoop.hbase.master.procedure.TestSCPWithMeta |
   |   | hadoop.hbase.replication.TestRemoveFromSerialReplicationPeer |
   |   | hadoop.hbase.master.procedure.TestFastFailOnProcedureNotRegistered |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.master.TestDLSAsyncFSWAL |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.master.procedure.TestSafemodeBringsDownMaster |
   |   | hadoop.hbase.master.procedure.TestCloneSnapshotProcedure |
   |   | 
hadoop.hbase.replication.TestReplicationStatusBothNormalAndRecoveryLagging |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.client.TestCIIncrementRpcTimeout |
   |   | hadoop.hbase.TestClusterBootOrder |
   |   | 
hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureInfiniteLoop |
   |   | hadoop.hbase.client.TestSnapshotFromClient |
   |   | hadoop.hbase.replication.master.TestRecoverStandbyProcedure |
   |   | hadoop.hbase.filter.TestFilterListOrOperatorWithBlkCnt |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.util.TestHBaseFsckEncryption |
   |   | hadoop.hbase.client.TestFromClientSideScanExcpetion |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.regionserver.wal.TestLogRollAbort |
   |   | hadoop.hbase.master.cleaner.TestReplicationBarrierCleaner |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.master.snapshot.TestSnapshotFileCache |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithAbort |
   |   | hadoop.hbase.coprocessor.TestRegionObserverScannerOpenHook |
   |   | hadoop.hbase.quotas.TestSpaceQuotaOnBulkLoad |
   |   | 
hadoop.hbase.coprocessor.TestRegionServerCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.replication.TestSyncReplicationActive |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSource |
   |   | hadoop.hbase.client.TestSplitOrMergeStatus |
   |   | hadoop.hbase.quotas.TestQuotaAdmin |
   |   | hadoop.hbase.regionserver.wal.TestLogRollingNoCluster |
   |   | hadoop.hbase.client.TestAsyncTableScanException |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.regionserver.TestHRegionServerBulkLoad |
   |   | hadoop.hbase.master.normalizer.TestSimpleRegionNormalizerOnCluster |
   |   | hadoop.hbase.wal.TestFSHLogCorruptionDueToDanglingByteBuffer |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-29 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-526426546
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 89 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 370 | master passed |
   | +1 | compile | 60 | master passed |
   | +1 | checkstyle | 91 | master passed |
   | +1 | shadedjars | 286 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 38 | master passed |
   | 0 | spotbugs | 279 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 277 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 309 | the patch passed |
   | +1 | compile | 63 | the patch passed |
   | +1 | javac | 63 | the patch passed |
   | +1 | checkstyle | 91 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 284 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 968 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 49 | the patch passed |
   | +1 | findbugs | 289 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18413 | hbase-server in the patch failed. |
   | -1 | asflicense | 31 | The patch generated 30 ASF License warnings. |
   | | | 22106 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | hadoop.hbase.tool.TestCanaryTool |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.quotas.TestSpaceQuotaIncrease |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | hadoop.hbase.client.TestAvoidCellReferencesIntoShippedBlocks |
   |   | hadoop.hbase.mob.TestMobWithByteBuffAllocator |
   |   | hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureBackoff |
   |   | hadoop.hbase.util.TestMiniClusterLoadEncoded |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.master.TestServerCrashProcedureStuck |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.snapshot.TestRestoreSnapshotHelper |
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.master.procedure.TestTruncateTableProcedure |
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.master.procedure.TestRestoreSnapshotProcedure |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.regionserver.wal.TestLogRolling |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.client.TestAsyncTableBatch |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.regionserver.TestSettingTimeoutOnBlockingPoint |
   |   | hadoop.hbase.client.TestGetScanColumnsWithNewVersionBehavior |
   |   | hadoop.hbase.master.TestMetaShutdownHandler |
   |   | hadoop.hbase.coprocessor.TestWALObserver |
   |   | hadoop.hbase.client.TestAlwaysSetScannerId |
   |   | hadoop.hbase.client.TestMultipleTimestamps |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.client.TestScannersFromClientSide2 |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.regionserver.TestScannerWithCorruptHFile |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.regionserver.TestHRegionOnCluster |
   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-28 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-525684148
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 323 | master passed |
   | +1 | compile | 56 | master passed |
   | +1 | checkstyle | 89 | master passed |
   | +1 | shadedjars | 276 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 39 | master passed |
   | 0 | spotbugs | 256 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 252 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 306 | the patch passed |
   | +1 | compile | 57 | the patch passed |
   | +1 | javac | 57 | the patch passed |
   | +1 | checkstyle | 86 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 273 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 951 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 40 | the patch passed |
   | +1 | findbugs | 254 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18480 | hbase-server in the patch failed. |
   | -1 | asflicense | 33 | The patch generated 18 ASF License warnings. |
   | | | 21948 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.constraint.TestConstraint |
   |   | hadoop.hbase.TestAcidGuaranteesWithEagerPolicy |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.regionserver.TestRegionServerReportForDuty |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.client.TestAsyncDecommissionAdminApi |
   |   | hadoop.hbase.snapshot.TestRestoreSnapshotHelper |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi |
   |   | hadoop.hbase.master.TestCatalogJanitorInMemoryStates |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasScan |
   |   | hadoop.hbase.wal.TestFSHLogCorruptionDueToDanglingByteBuffer |
   |   | hadoop.hbase.replication.regionserver.TestReplicator |
   |   | hadoop.hbase.client.TestAsyncTableBatchRetryImmediately |
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.TestJMXListener |
   |   | hadoop.hbase.security.visibility.TestWithDisabledAuthorization |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.client.TestResultSizeEstimation |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | hadoop.hbase.regionserver.TestCleanupCompactedFileAfterFailover |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.regionserver.TestHRegionServerBulkLoad |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi |
   |   | hadoop.hbase.util.TestFSUtils |
   |   | hadoop.hbase.mob.TestMobWithByteBuffAllocator |
   |   | hadoop.hbase.client.TestReplicasClient |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | hadoop.hbase.snapshot.TestMobFlushSnapshotFromClient |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleWAL
 |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.master.TestGetReplicationLoad |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.master.TestMasterHandlerFullWhenTransitRegion |
   |   | hadoop.hbase.regionserver.wal.TestFSHLog |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.procedure.TestFailedProcCleanup |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.master.TestClusterRestartSplitWithoutZk |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocatorConcurrenyLimit |
   |   | hadoop.hbase.client.TestMetaWithReplicas |
   |   | hadoop.hbase.replication.TestMasterReplication |
   |   | hadoop.hbase.wal.TestWALFiltering |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.quotas.TestQuotaObserverChoreRegionReports |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-27 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-525500084
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 35 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 345 | master passed |
   | +1 | compile | 59 | master passed |
   | +1 | checkstyle | 96 | master passed |
   | +1 | shadedjars | 288 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 40 | master passed |
   | 0 | spotbugs | 243 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 240 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 317 | the patch passed |
   | +1 | compile | 59 | the patch passed |
   | +1 | javac | 59 | the patch passed |
   | +1 | checkstyle | 94 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 269 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1035 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 39 | the patch passed |
   | +1 | findbugs | 270 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19012 | hbase-server in the patch failed. |
   | -1 | asflicense | 33 | The patch generated 19 ASF License warnings. |
   | | | 22661 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.coprocessor.TestRegionObserverScannerOpenHook |
   |   | hadoop.hbase.regionserver.throttle.TestFlushWithThroughputController |
   |   | hadoop.hbase.security.visibility.TestVisibilityLablesWithGroups |
   |   | 
hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterSplittingRegions |
   |   | hadoop.hbase.master.procedure.TestModifyNamespaceProcedure |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.client.TestRegionLocator |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.tool.TestBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.master.TestRegionPlacement2 |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.master.procedure.TestSCPWithMeta |
   |   | hadoop.hbase.TestPartialResultsFromClientSide |
   |   | hadoop.hbase.client.TestUpdateConfiguration |
   |   | hadoop.hbase.TestMultiVersions |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.client.TestBlockEvictionFromClient |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.client.TestGetProcedureResult |
   |   | hadoop.hbase.replication.TestSyncReplicationStandBy |
   |   | hadoop.hbase.client.TestAsyncTableBatch |
   |   | hadoop.hbase.regionserver.wal.TestSecureWALReplay |
   |   | hadoop.hbase.client.TestAvoidCellReferencesIntoShippedBlocks |
   |   | hadoop.hbase.quotas.TestSpaceQuotaOnBulkLoad |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.coprocessor.TestMetaTableMetrics |
   |   | hadoop.hbase.TestAcidGuaranteesWithNoInMemCompaction |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.mob.compactions.TestPartitionedMobCompactor |
   |   | hadoop.hbase.regionserver.TestClearRegionBlockCache |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.security.access.TestAccessController3 |
   |   | hadoop.hbase.io.asyncfs.TestFanOutOneBlockAsyncDFSOutput |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.regionserver.TestRegionServerCrashDisableWAL |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.client.TestRawAsyncTablePartialScan |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-26 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-525130047
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 35 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 324 | master passed |
   | +1 | compile | 77 | master passed |
   | +1 | checkstyle | 89 | master passed |
   | +1 | shadedjars | 272 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 38 | master passed |
   | 0 | spotbugs | 236 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 234 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 296 | the patch passed |
   | +1 | compile | 56 | the patch passed |
   | +1 | javac | 56 | the patch passed |
   | +1 | checkstyle | 88 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 273 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 921 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 236 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18759 | hbase-server in the patch failed. |
   | -1 | asflicense | 26 | The patch generated 54 ASF License warnings. |
   | | | 22136 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.coprocessor.TestRegionServerCoprocessorExceptionWithAbort 
|
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.TestClientClusterStatus |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.client.TestCIDeleteOperationTimeout |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.quotas.TestSpaceQuotaOnBulkLoad |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSource |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.master.TestDLSAsyncFSWAL |
   |   | hadoop.hbase.replication.TestMasterReplication |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.TestMultiVersions |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.client.TestAppendFromClientSide |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestart |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationEndpoint |
   |   | hadoop.hbase.security.access.TestCellACLs |
   |   | hadoop.hbase.master.procedure.TestProcedureAdmin |
   |   | hadoop.hbase.ipc.TestNettyRpcServer |
   |   | hadoop.hbase.master.procedure.TestMasterObserverPostCalls |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.snapshot.TestRegionSnapshotTask |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | hadoop.hbase.master.assignment.TestOpenRegionProcedureHang |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRS |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | hadoop.hbase.wal.TestWALSplit |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.ipc.TestMasterFifoRpcScheduler |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi3 |
   |   | hadoop.hbase.master.procedure.TestSafemodeBringsDownMaster |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.wal.TestWALFactory |
   |   | hadoop.hbase.regionserver.wal.TestDurability |
   |   | hadoop.hbase.coprocessor.TestCoprocessorSharedConnection |
   |   | hadoop.hbase.client.TestAsyncBufferMutator |
   |   | hadoop.hbase.master.TestMetaFixer |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.client.TestResultSizeEstimation |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSimple |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-25 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524701418
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 36 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 339 | master passed |
   | +1 | compile | 57 | master passed |
   | +1 | checkstyle | 92 | master passed |
   | +1 | shadedjars | 285 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 40 | master passed |
   | 0 | spotbugs | 298 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 295 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 378 | the patch passed |
   | +1 | compile | 70 | the patch passed |
   | +1 | javac | 70 | the patch passed |
   | +1 | checkstyle | 94 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 321 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1197 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 40 | the patch passed |
   | +1 | findbugs | 306 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20240 | hbase-server in the patch failed. |
   | -1 | asflicense | 28 | The patch generated 28 ASF License warnings. |
   | | | 24284 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting |
   |   | 
hadoop.hbase.master.assignment.TestReportRegionStateTransitionFromDeadServer |
   |   | hadoop.hbase.master.balancer.TestFavoredStochasticBalancerPickers |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.coprocessor.TestRegionObserverScannerOpenHook |
   |   | hadoop.hbase.regionserver.throttle.TestFlushWithThroughputController |
   |   | hadoop.hbase.security.visibility.TestVisibilityLablesWithGroups |
   |   | hadoop.hbase.master.procedure.TestModifyNamespaceProcedure |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.master.TestMasterFileSystemWithWALDir |
   |   | hadoop.hbase.regionserver.wal.TestProtobufLog |
   |   | hadoop.hbase.coprocessor.TestAppendTimeRange |
   |   | hadoop.hbase.client.TestRegionLocator |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.tool.TestBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.master.TestRegionPlacement2 |
   |   | hadoop.hbase.regionserver.TestRowTooBig |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.master.procedure.TestSCPWithMeta |
   |   | hadoop.hbase.client.TestAlwaysSetScannerId |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.quotas.TestSpaceQuotaDropTable |
   |   | hadoop.hbase.client.TestBlockEvictionFromClient |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestGetProcedureResult |
   |   | hadoop.hbase.replication.TestSyncReplicationStandBy |
   |   | hadoop.hbase.master.TestTableStateManager |
   |   | hadoop.hbase.client.TestAsyncTableBatch |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.client.TestAvoidCellReferencesIntoShippedBlocks |
   |   | hadoop.hbase.regionserver.TestRegionServerNoMaster |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasGet |
   |   | hadoop.hbase.coprocessor.TestMetaTableMetrics |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.master.TestShutdownWithNoRegionServer |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.security.access.TestCellACLs |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-25 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524629387
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 71 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 360 | master passed |
   | +1 | compile | 58 | master passed |
   | +1 | checkstyle | 99 | master passed |
   | +1 | shadedjars | 298 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 264 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 262 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 327 | the patch passed |
   | +1 | compile | 58 | the patch passed |
   | +1 | javac | 58 | the patch passed |
   | +1 | checkstyle | 94 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 300 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1032 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 37 | the patch passed |
   | +1 | findbugs | 271 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19821 | hbase-server in the patch failed. |
   | -1 | asflicense | 31 | The patch generated 18 ASF License warnings. |
   | | | 23572 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi2 |
   |   | hadoop.hbase.master.TestGetLastFlushedSequenceId |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.security.visibility.TestVisibilityWithCheckAuths |
   |   | hadoop.hbase.quotas.TestSnapshotQuotaObserverChore |
   |   | hadoop.hbase.master.TestMetaFixer |
   |   | 
hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureInfiniteLoop |
   |   | hadoop.hbase.master.TestAssignmentManagerMetrics |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelReplicationWithExpAsString |
   |   | hadoop.hbase.replication.master.TestRecoverStandbyProcedure |
   |   | hadoop.hbase.master.TestCloseAnOpeningRegion |
   |   | hadoop.hbase.TestInfoServers |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi3 |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.regionserver.TestRegionServerReportForDuty |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.wal.TestBoundedRegionGroupingStrategy |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationSyncUpToolWithMultipleAsyncWAL 
|
   |   | hadoop.hbase.client.TestReplicasClient |
   |   | hadoop.hbase.filter.TestColumnRangeFilter |
   |   | hadoop.hbase.master.procedure.TestCreateNamespaceProcedure |
   |   | hadoop.hbase.regionserver.TestPerColumnFamilyFlush |
   |   | hadoop.hbase.client.TestSplitOrMergeStatus |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.client.TestAsyncAdminMasterSwitch |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.client.TestFromClientSideWithCoprocessor |
   |   | hadoop.hbase.master.cleaner.TestHFileCleaner |
   |   | hadoop.hbase.filter.TestMultiRowRangeFilter |
   |   | hadoop.hbase.regionserver.TestSecureBulkLoadManager |
   |   | hadoop.hbase.regionserver.TestShutdownWhileWALBroken |
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.master.TestNewStartedRegionServerVersion |
   |   | hadoop.hbase.TestZooKeeper |
   |   | hadoop.hbase.master.procedure.TestSCP |
   |   | hadoop.hbase.regionserver.TestEndToEndSplitTransaction |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-24 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524558428
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 81 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 365 | master passed |
   | +1 | compile | 63 | master passed |
   | +1 | checkstyle | 97 | master passed |
   | +1 | shadedjars | 314 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 46 | master passed |
   | 0 | spotbugs | 283 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 281 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 342 | the patch passed |
   | +1 | compile | 61 | the patch passed |
   | +1 | javac | 61 | the patch passed |
   | +1 | checkstyle | 96 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 305 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1046 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 45 | the patch passed |
   | +1 | findbugs | 293 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19286 | hbase-server in the patch failed. |
   | -1 | asflicense | 31 | The patch generated 18 ASF License warnings. |
   | | | 23164 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.security.visibility.TestEnforcingScanLabelGenerator |
   |   | hadoop.hbase.io.asyncfs.TestFanOutOneBlockAsyncDFSOutput |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.security.access.TestScanEarlyTermination |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.regionserver.compactions.TestFIFOCompactionPolicy |
   |   | hadoop.hbase.client.TestLimitedScanWithFilter |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleWAL
 |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelReplicationWithExpAsString |
   |   | hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureBackoff |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpointNoMaster
 |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.master.cleaner.TestReplicationBarrierCleaner |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplay |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.client.TestDropTimeoutRequest |
   |   | hadoop.hbase.replication.TestReplicationProcedureRetry |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestart |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.regionserver.TestRowTooBig |
   |   | hadoop.hbase.snapshot.TestRegionSnapshotTask |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.regionserver.TestCompactionState |
   |   | hadoop.hbase.master.procedure.TestEnableTableProcedure |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.master.TestMergeTableRegionsWhileRSCrash |
   |   | hadoop.hbase.regionserver.wal.TestSecureAsyncWALReplay |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi3 |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | 
hadoop.hbase.master.replication.TestTransitPeerSyncReplicationStateProcedureBackoff
 |
   |   | hadoop.hbase.regionserver.TestSplitWalDataLoss |
   |   | hadoop.hbase.regionserver.TestPerColumnFamilyFlush |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | 
hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureInfiniteLoop |
   |   | hadoop.hbase.regionserver.wal.TestSecureWALReplay |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-24 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524525258
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 118 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 442 | master passed |
   | +1 | compile | 62 | master passed |
   | +1 | checkstyle | 84 | master passed |
   | +1 | shadedjars | 316 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 45 | master passed |
   | 0 | spotbugs | 269 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 267 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 340 | the patch passed |
   | +1 | compile | 63 | the patch passed |
   | +1 | javac | 63 | the patch passed |
   | +1 | checkstyle | 86 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 300 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1159 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 41 | the patch passed |
   | +1 | findbugs | 326 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 17227 | hbase-server in the patch failed. |
   | -1 | asflicense | 30 | The patch generated 18 ASF License warnings. |
   | | | 21349 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestCloneSnapshotFromClientNormal 
|
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.wal.TestFSHLogProvider |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.master.TestMasterMetricsWrapper |
   |   | hadoop.hbase.client.TestAsyncBufferMutator |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi3 |
   |   | hadoop.hbase.regionserver.TestHRegionReplayEvents |
   |   | hadoop.hbase.coprocessor.TestRegionObserverScannerOpenHook |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestSplitOrMergeStatus |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.client.TestScannersFromClientSide2 |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.client.TestSmallReversedScanner |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.filter.TestFilterListOnMini |
   |   | hadoop.hbase.master.TestMasterHandlerFullWhenTransitRegion |
   |   | hadoop.hbase.client.TestTableOperationException |
   |   | hadoop.hbase.io.asyncfs.TestSaslFanOutOneBlockAsyncDFSOutput |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.coprocessor.TestAppendTimeRange |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientAfterSplittingRegion |
   |   | hadoop.hbase.client.TestCIPutRpcTimeout |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.client.TestAsyncDecommissionAdminApi |
   |   | hadoop.hbase.client.TestFromClientSideNoCodec |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.regionserver.TestRegionOpen |
   |   | hadoop.hbase.security.access.TestCellACLs |
   |   | hadoop.hbase.quotas.TestQuotaObserverChoreWithMiniCluster |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.client.TestAsyncAdminBuilder |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.client.TestMetaWithReplicas |
   |   | hadoop.hbase.regionserver.TestRegionServerReportForDuty |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas |
   |   | hadoop.hbase.client.TestBufferedMutator |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMeta |
   |   | hadoop.hbase.regionserver.TestJoinedScanners |
   |   | hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.filter.TestMultiRowRangeFilter |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-23 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524516987
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 76 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 484 | master passed |
   | +1 | compile | 78 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 371 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 48 | master passed |
   | 0 | spotbugs | 312 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 309 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 421 | the patch passed |
   | +1 | compile | 73 | the patch passed |
   | +1 | javac | 73 | the patch passed |
   | +1 | checkstyle | 107 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 369 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1398 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 47 | the patch passed |
   | +1 | findbugs | 327 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18738 | hbase-server in the patch failed. |
   | -1 | asflicense | 43 | The patch generated 35 ASF License warnings. |
   | | | 23553 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.assignment.TestRegionReplicaSplit 
|
   |   | hadoop.hbase.regionserver.wal.TestLogRollPeriod |
   |   | hadoop.hbase.util.TestHBaseFsckEncryption |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicas |
   |   | hadoop.hbase.client.TestScanWithoutFetchingData |
   |   | hadoop.hbase.client.TestTableOperationException |
   |   | hadoop.hbase.coprocessor.TestRegionObserverBypass |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOnNewVersionBehaviorTable |
   |   | hadoop.hbase.wal.TestWALSplitBoundedLogWriterCreation |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.regionserver.TestFSErrorsExposed |
   |   | hadoop.hbase.snapshot.TestConcurrentFlushSnapshotFromClient |
   |   | hadoop.hbase.client.TestAsyncTableNoncedRetry |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayCompressed |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.client.TestScannersFromClientSide |
   |   | hadoop.hbase.master.procedure.TestMasterProcedureEvents |
   |   | hadoop.hbase.master.TestMasterRepairMode |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.coprocessor.TestWALObserver |
   |   | hadoop.hbase.client.TestFromClientSideScanExcpetion |
   |   | hadoop.hbase.master.assignment.TestRegionBypass |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.master.TestWarmupRegion |
   |   | hadoop.hbase.regionserver.TestRowTooBig |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMeta |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.replication.TestReplicationStatusAfterLagging |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.master.cleaner.TestLogsCleaner |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.replication.TestSyncReplicationActive |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.regionserver.TestRegionFavoredNodes |
   |   | hadoop.hbase.master.assignment.TestSCPGetRegionsRace |
   |   | hadoop.hbase.TestClusterBootOrder |
   |   | hadoop.hbase.master.TestCatalogJanitorCluster |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.master.TestGetReplicationLoad |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.client.TestAdmin |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-23 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524508321
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 124 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 386 | master passed |
   | +1 | compile | 64 | master passed |
   | +1 | checkstyle | 104 | master passed |
   | +1 | shadedjars | 305 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 274 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 272 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 332 | the patch passed |
   | +1 | compile | 60 | the patch passed |
   | +1 | javac | 60 | the patch passed |
   | +1 | checkstyle | 94 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 302 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1041 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 35 | the patch passed |
   | +1 | findbugs | 277 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19622 | hbase-server in the patch failed. |
   | -1 | asflicense | 33 | The patch generated 52 ASF License warnings. |
   | | | 23520 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.filter.TestFilterWithScanLimits |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRS |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.master.assignment.TestAssignmentOnRSCrash |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | hadoop.hbase.client.TestAsyncAdminWithRegionReplicas |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.regionserver.wal.TestWALReplay |
   |   | hadoop.hbase.client.TestCIPutRpcTimeout |
   |   | hadoop.hbase.client.TestRawAsyncTablePartialScan |
   |   | hadoop.hbase.filter.TestFuzzyRowFilterEndToEnd |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestart |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.util.TestFSUtils |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi2 |
   |   | hadoop.hbase.master.procedure.TestMasterObserverPostCalls |
   |   | hadoop.hbase.regionserver.TestScannerWithCorruptHFile |
   |   | hadoop.hbase.master.procedure.TestModifyNamespaceProcedure |
   |   | hadoop.hbase.util.TestCoprocessorScanPolicy |
   |   | hadoop.hbase.master.assignment.TestRegionStateStore |
   |   | hadoop.hbase.security.visibility.TestDefaultScanLabelGeneratorStack |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestartSplitWithoutZk |
   |   | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.security.access.TestNamespaceCommands |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationEndpoint |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.replication.TestRemoveFromSerialReplicationPeer |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.util.compaction.TestMajorCompactor |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.snapshot.TestSnapshotClientRetries |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasScan |
   |   | hadoop.hbase.master.procedure.TestCreateNamespaceProcedure |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.replication.TestSyncReplicationNewRSJoinBetweenRefreshes |
   |   | hadoop.hbase.quotas.TestQuotaObserverChoreWithMiniCluster |
   |   | hadoop.hbase.client.TestMultiParallel |
   |   | hadoop.hbase.client.TestAsyncTableNoncedRetry |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.client.TestAvoidCellReferencesIntoShippedBlocks |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-23 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524497800
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 40 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 329 | master passed |
   | +1 | compile | 53 | master passed |
   | +1 | checkstyle | 77 | master passed |
   | +1 | shadedjars | 276 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 247 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 246 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 295 | the patch passed |
   | +1 | compile | 54 | the patch passed |
   | +1 | javac | 54 | the patch passed |
   | +1 | checkstyle | 76 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 269 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 911 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 35 | the patch passed |
   | +1 | findbugs | 255 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19496 | hbase-server in the patch failed. |
   | -1 | asflicense | 24 | The patch generated 18 ASF License warnings. |
   | | | 22846 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.TestNamespace |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientAfterSplittingRegion 
|
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.master.locking.TestLockManager |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.backup.TestHFileArchiving |
   |   | hadoop.hbase.master.procedure.TestCreateNamespaceProcedure |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.TestMovedRegionsCleaner |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.client.TestResultFromCoprocessor |
   |   | hadoop.hbase.master.TestGetInfoPort |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.coprocessor.TestIncrementTimeRange |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi |
   |   | hadoop.hbase.master.TestDLSFSHLog |
   |   | hadoop.hbase.master.TestMasterAbortAndRSGotKilled |
   |   | hadoop.hbase.client.TestSnapshotWithAclAsyncAdmin |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.coprocessor.TestCoprocessorSharedConnection |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestLimitedScanWithFilter |
   |   | hadoop.hbase.constraint.TestConstraint |
   |   | hadoop.hbase.master.TestClusterRestart |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-22 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524148440
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 41 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 352 | master passed |
   | +1 | compile | 54 | master passed |
   | +1 | checkstyle | 86 | master passed |
   | +1 | shadedjars | 292 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 34 | master passed |
   | 0 | spotbugs | 257 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 255 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 335 | the patch passed |
   | +1 | compile | 59 | the patch passed |
   | +1 | javac | 59 | the patch passed |
   | +1 | checkstyle | 83 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 289 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 974 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 249 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 17771 | hbase-server in the patch failed. |
   | -1 | asflicense | 25 | The patch generated 30 ASF License warnings. |
   | | | 21302 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.quotas.TestQuotaObserverChoreRegionReports |
   |   | hadoop.hbase.client.TestTableFavoredNodes |
   |   | hadoop.hbase.client.TestAsyncProcedureAdminApi |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationSyncUpToolWithMultipleAsyncWAL 
|
   |   | hadoop.hbase.regionserver.TestFSErrorsExposed |
   |   | hadoop.hbase.regionserver.TestIsDeleteFailure |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.client.TestAsyncTableLocateRegionForDeletedTable |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.master.TestMasterRepairMode |
   |   | hadoop.hbase.security.visibility.TestEnforcingScanLabelGenerator |
   |   | hadoop.hbase.client.TestAsyncAdminMasterSwitch |
   |   | hadoop.hbase.replication.TestReplicationDroppedTables |
   |   | hadoop.hbase.TestNamespace |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | hadoop.hbase.client.TestIncrementFromClientSideWithCoprocessor |
   |   | hadoop.hbase.regionserver.TestDeleteMobTable |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.client.TestGetProcedureResult |
   |   | hadoop.hbase.master.TestCatalogJanitorInMemoryStates |
   |   | hadoop.hbase.client.TestRawAsyncTableLimitedScanWithFilter |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.master.balancer.TestBalancerStatusTagInJMXMetrics |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.TestJMXListener |
   |   | hadoop.hbase.regionserver.TestRemoveRegionMetrics |
   |   | hadoop.hbase.client.TestIncrementsFromClientSide |
   |   | hadoop.hbase.quotas.TestQuotaAdmin |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNoOps |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestart |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.client.TestFromClientSide |
   |   | hadoop.hbase.master.procedure.TestSCPWithMeta |
   |   | hadoop.hbase.mob.TestDefaultMobStoreFlusher |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRS |
   |   | hadoop.hbase.master.assignment.TestReportRegionStateTransitionRetry |
   |   | hadoop.hbase.master.TestMigrateNamespaceTable |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicas |
   |   | hadoop.hbase.client.TestScannersFromClientSide |
   |   | hadoop.hbase.client.TestMobSnapshotFromClient |
   |   | hadoop.hbase.master.TestServerCrashProcedureCarryingMetaStuck |
   |   | hadoop.hbase.wal.TestAsyncFSWALCorruptionDueToDanglingByteBuffer |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting |
   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-22 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524117722
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 113 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 358 | master passed |
   | +1 | compile | 54 | master passed |
   | +1 | checkstyle | 81 | master passed |
   | +1 | shadedjars | 281 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 249 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 245 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 317 | the patch passed |
   | +1 | compile | 55 | the patch passed |
   | +1 | javac | 55 | the patch passed |
   | +1 | checkstyle | 78 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 279 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 972 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 34 | the patch passed |
   | +1 | findbugs | 271 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19517 | hbase-server in the patch failed. |
   | -1 | asflicense | 32 | The patch generated 51 ASF License warnings. |
   | | | 23121 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.master.TestServerCrashProcedureStuck |
   |   | hadoop.hbase.client.TestSplitOrMergeAtTableLevel |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.master.assignment.TestWakeUpUnexpectedProcedure |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.replication.TestSyncReplicationStandBy |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.regionserver.TestMobStoreScanner |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.replication.TestReplicationMetricsforUI |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | hadoop.hbase.client.TestTableFavoredNodes |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationSyncUpToolWithMultipleWAL |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting |
   |   | hadoop.hbase.quotas.TestSuperUserQuotaPermissions |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.io.asyncfs.TestSaslFanOutOneBlockAsyncDFSOutput |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayBoundedLogWriterCreation |
   |   | hadoop.hbase.master.TestClusterRestartSplitWithoutZk |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi2 |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.wal.TestWALSplitCompressed |
   |   | hadoop.hbase.TestInfoServers |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.master.replication.TestModifyPeerProcedureRetryBackoff |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutZKCoordinated |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.master.TestRollingRestart |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.TestClusterRestartFailoverSplitWithoutZk |
   |   | hadoop.hbase.TestHBaseOnOtherDfsCluster |
   |   | hadoop.hbase.master.assignment.TestMergeTableRegionsProcedure |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.replication.TestReplicationEmptyWALRecovery |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   |   | hadoop.hbase.TestGlobalMemStoreSize |
   |   | hadoop.hbase.util.TestRegionSplitter |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.TestHBaseTestingUtility |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientGetCompactionState 
|
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-22 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-524041051
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 59 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 423 | master passed |
   | +1 | compile | 77 | master passed |
   | +1 | checkstyle | 119 | master passed |
   | +1 | shadedjars | 372 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 44 | master passed |
   | 0 | spotbugs | 287 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 284 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 403 | the patch passed |
   | +1 | compile | 81 | the patch passed |
   | +1 | javac | 81 | the patch passed |
   | +1 | checkstyle | 104 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 367 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1267 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 44 | the patch passed |
   | +1 | findbugs | 304 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19879 | hbase-server in the patch failed. |
   | -1 | asflicense | 34 | The patch generated 23 ASF License warnings. |
   | | | 24356 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestIncrementsFromClientSide |
   |   | hadoop.hbase.client.TestTableFavoredNodes |
   |   | hadoop.hbase.master.assignment.TestMergeTableRegionsProcedure |
   |   | hadoop.hbase.client.TestSnapshotWithAclAsyncAdmin |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientAfterSplittingRegion |
   |   | hadoop.hbase.master.TestClusterRestartSplitWithoutZk |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.filter.TestColumnRangeFilter |
   |   | hadoop.hbase.client.TestAsyncClientPushback |
   |   | hadoop.hbase.master.assignment.TestAssignmentManagerUtil |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocatorConcurrenyLimit |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestartSplitWithoutZk |
   |   | hadoop.hbase.client.TestCIDeleteOperationTimeout |
   |   | hadoop.hbase.master.TestCloseAnOpeningRegion |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.replication.TestReplicationMetricsforUI |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasScan |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicas |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestAsyncProcedureAdminApi |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.client.TestAsyncBufferMutator |
   |   | hadoop.hbase.master.cleaner.TestHFileCleaner |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.master.TestMasterRestartAfterDisablingTable |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.regionserver.wal.TestCombinedAsyncWriter |
   |   | hadoop.hbase.mob.compactions.TestPartitionedMobCompactor |
   |   | hadoop.hbase.client.TestRawAsyncTableScan |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.constraint.TestConstraint |
   |   | hadoop.hbase.master.procedure.TestSCPWithMeta |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestSyncReplicationNewRSJoinBetweenRefreshes |
   |   | hadoop.hbase.master.assignment.TestRegionBypass |
   |   | hadoop.hbase.master.assignment.TestCloseRegionWhileRSCrash |
   |   | hadoop.hbase.snapshot.TestSnapshotClientRetries |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRS |
   |   | hadoop.hbase.client.TestAsyncTableScan |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-22 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523946642
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 95 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 483 | master passed |
   | +1 | compile | 67 | master passed |
   | +1 | checkstyle | 101 | master passed |
   | +1 | shadedjars | 393 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 45 | master passed |
   | 0 | spotbugs | 312 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 309 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 426 | the patch passed |
   | +1 | compile | 71 | the patch passed |
   | +1 | javac | 71 | the patch passed |
   | +1 | checkstyle | 102 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 377 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1301 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 39 | the patch passed |
   | +1 | findbugs | 307 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18978 | hbase-server in the patch failed. |
   | -1 | asflicense | 74 | The patch generated 30 ASF License warnings. |
   | | | 23728 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.regionserver.TestLogRoller |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.client.TestZKAsyncRegistry |
   |   | hadoop.hbase.master.TestServerCrashProcedureStuck |
   |   | hadoop.hbase.master.TestServerCrashProcedureCarryingMetaStuck |
   |   | hadoop.hbase.client.TestRawAsyncTableLimitedScanWithFilter |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.quotas.TestQuotaTableUtil |
   |   | hadoop.hbase.client.TestResultScannerCursor |
   |   | hadoop.hbase.TestAcidGuaranteesWithAdaptivePolicy |
   |   | hadoop.hbase.replication.TestSyncReplicationNewRSJoinBetweenRefreshes |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.client.TestGetProcedureResult |
   |   | hadoop.hbase.security.access.TestNamespaceCommands |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocator |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.coprocessor.TestNegativeMemStoreSizeWithSlowCoprocessor |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.util.TestHBaseFsckCleanReplicationBarriers |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.master.TestGetLastFlushedSequenceId |
   |   | hadoop.hbase.client.TestSmallReversedScanner |
   |   | hadoop.hbase.TestClientClusterStatus |
   |   | hadoop.hbase.replication.master.TestRecoverStandbyProcedure |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.master.TestRollingRestart |
   |   | hadoop.hbase.client.TestAsyncTableScanAll |
   |   | hadoop.hbase.client.TestAsyncTableBatchRetryImmediately |
   |   | hadoop.hbase.TestHColumnDescriptorDefaultVersions |
   |   | hadoop.hbase.procedure.TestFailedProcCleanup |
   |   | hadoop.hbase.quotas.TestSnapshotQuotaObserverChore |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | hadoop.hbase.master.assignment.TestCloseRegionWhileRSCrash |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.client.TestAsyncProcedureAdminApi |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.regionserver.TestSplitWalDataLoss |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNoOps |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.client.TestLimitedScanWithFilter |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.regionserver.wal.TestLogRolling |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-22 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523844416
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 95 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 355 | master passed |
   | +1 | compile | 59 | master passed |
   | +1 | checkstyle | 92 | master passed |
   | +1 | shadedjars | 294 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 39 | master passed |
   | 0 | spotbugs | 280 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 278 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 311 | the patch passed |
   | +1 | compile | 59 | the patch passed |
   | +1 | javac | 59 | the patch passed |
   | +1 | checkstyle | 90 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 292 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 967 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 35 | the patch passed |
   | +1 | findbugs | 273 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20478 | hbase-server in the patch failed. |
   | -1 | asflicense | 46 | The patch generated 30 ASF License warnings. |
   | | | 24158 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | hadoop.hbase.tool.TestCanaryTool |
   |   | hadoop.hbase.master.procedure.TestProcedureAdmin |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureBackoff |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.replication.regionserver.TestGlobalReplicationThrottler |
   |   | hadoop.hbase.master.TestServerCrashProcedureStuck |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.client.TestGetProcedureResult |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.regionserver.TestCleanupCompactedFileOnRegionClose |
   |   | hadoop.hbase.master.balancer.TestRegionLocationFinder |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.regionserver.TestOpenSeqNumUnexpectedIncrease |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.regionserver.wal.TestLogRolling |
   |   | hadoop.hbase.master.TestDLSAsyncFSWAL |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.client.TestAsyncTableBatch |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.regionserver.TestSettingTimeoutOnBlockingPoint |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.master.TestShutdownWithNoRegionServer |
   |   | hadoop.hbase.client.TestAlwaysSetScannerId |
   |   | hadoop.hbase.client.TestMultipleTimestamps |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.client.TestScannersFromClientSide2 |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestAsyncTableNoncedRetry |
   |   | hadoop.hbase.client.TestCIIncrementRpcTimeout |
   |   | hadoop.hbase.replication.TestReplicationEmptyWALRecovery |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-21 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523678990
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 71 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 478 | master passed |
   | +1 | compile | 78 | master passed |
   | +1 | checkstyle | 117 | master passed |
   | +1 | shadedjars | 400 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 51 | master passed |
   | 0 | spotbugs | 328 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 325 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 440 | the patch passed |
   | +1 | compile | 75 | the patch passed |
   | +1 | javac | 75 | the patch passed |
   | +1 | checkstyle | 110 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 401 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1399 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 47 | the patch passed |
   | +1 | findbugs | 333 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19761 | hbase-server in the patch failed. |
   | -1 | asflicense | 42 | The patch generated 30 ASF License warnings. |
   | | | 24677 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.snapshot.TestRestoreFlushSnapshotFromClient |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.client.TestSmallReversedScanner |
   |   | 
hadoop.hbase.replication.TestReplicationStatusBothNormalAndRecoveryLagging |
   |   | hadoop.hbase.util.TestHBaseFsckEncryption |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationEndpoint |
   |   | hadoop.hbase.replication.regionserver.TestWALEntryStream |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayBoundedLogWriterCreation |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.tool.TestCanaryTool |
   |   | hadoop.hbase.client.TestAdmin3 |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.regionserver.TestFSErrorsExposed |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.master.procedure.TestTruncateTableProcedure |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithACL |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestartSplitWithoutZk |
   |   | hadoop.hbase.regionserver.TestRegionReplicasAreDistributed |
   |   | hadoop.hbase.client.TestSnapshotFromClient |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.client.TestUpdateConfiguration |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpointNoMaster
 |
   |   | hadoop.hbase.client.TestAsyncReplicationAdminApi |
   |   | 
hadoop.hbase.master.procedure.TestDeleteColumnFamilyProcedureFromClient |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSource |
   |   | hadoop.hbase.replication.TestVerifyCellsReplicationEndpoint |
   |   | hadoop.hbase.TestClusterBootOrder |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.client.TestAsyncTableBatchRetryImmediately |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.backup.TestHFileArchiving |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.wal.TestWALSplit |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.master.TestClusterRestartFailover |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-21 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523658299
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 80 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 349 | master passed |
   | +1 | compile | 59 | master passed |
   | +1 | checkstyle | 96 | master passed |
   | +1 | shadedjars | 287 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 38 | master passed |
   | 0 | spotbugs | 273 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 271 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 341 | the patch passed |
   | +1 | compile | 73 | the patch passed |
   | +1 | javac | 73 | the patch passed |
   | +1 | checkstyle | 112 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 346 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1194 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 44 | the patch passed |
   | +1 | findbugs | 272 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18920 | hbase-server in the patch failed. |
   | -1 | asflicense | 34 | The patch generated 21 ASF License warnings. |
   | | | 22999 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.coprocessor.TestIncrementTimeRange |
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | hadoop.hbase.tool.TestCanaryTool |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.replication.regionserver.TestGlobalReplicationThrottler |
   |   | hadoop.hbase.master.TestServerCrashProcedureStuck |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.snapshot.TestRestoreSnapshotHelper |
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.regionserver.TestOpenSeqNumUnexpectedIncrease |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.master.procedure.TestTruncateTableProcedure |
   |   | hadoop.hbase.regionserver.TestCompactionFileNotFound |
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.master.procedure.TestRestoreSnapshotProcedure |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | hadoop.hbase.regionserver.wal.TestLogRolling |
   |   | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.client.TestAsyncTableBatch |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.regionserver.TestSettingTimeoutOnBlockingPoint |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.coprocessor.TestWALObserver |
   |   | hadoop.hbase.master.TestGetLastFlushedSequenceId |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.client.TestScannersFromClientSide2 |
   |   | hadoop.hbase.replication.TestReplicationSmallTestsSync |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.regionserver.TestHRegionOnCluster |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestAsyncTableNoncedRetry |
   |   | hadoop.hbase.client.TestCIIncrementRpcTimeout |
   |   | hadoop.hbase.client.TestSnapshotFromClient |
   |   | hadoop.hbase.replication.TestReplicationEmptyWALRecovery |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.util.compaction.TestMajorCompactor |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-21 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523537567
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 46 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 356 | master passed |
   | +1 | compile | 55 | master passed |
   | +1 | checkstyle | 83 | master passed |
   | +1 | shadedjars | 282 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 262 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 260 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 315 | the patch passed |
   | +1 | compile | 58 | the patch passed |
   | +1 | javac | 58 | the patch passed |
   | +1 | checkstyle | 81 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 306 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1012 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 278 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19173 | hbase-server in the patch failed. |
   | -1 | asflicense | 23 | The patch generated 30 ASF License warnings. |
   | | | 22794 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestTableOperationException |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOnNewVersionBehaviorTable |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelReplicationWithExpAsString |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.master.procedure.TestEnableTableProcedure |
   |   | hadoop.hbase.client.TestMobSnapshotFromClient |
   |   | hadoop.hbase.master.TestWarmupRegion |
   |   | hadoop.hbase.client.TestAsyncTableLocateRegionForDeletedTable |
   |   | hadoop.hbase.namespace.TestNamespaceAuditor |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.security.token.TestGenerateDelegationToken |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.regionserver.TestScannerHeartbeatMessages |
   |   | hadoop.hbase.client.TestAsyncSnapshotAdminApi |
   |   | hadoop.hbase.security.access.TestAccessControlFilter |
   |   | hadoop.hbase.regionserver.wal.TestDurability |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestRawAsyncTableScan |
   |   | hadoop.hbase.replication.regionserver.TestGlobalReplicationThrottler |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.master.assignment.TestAssignmentOnRSCrash |
   |   | hadoop.hbase.client.TestAsyncNamespaceAdminApi |
   |   | hadoop.hbase.client.TestIncrementsFromClientSide |
   |   | hadoop.hbase.master.procedure.TestTruncateTableProcedure |
   |   | 
hadoop.hbase.master.assignment.TestReportRegionStateTransitionFromDeadServer |
   |   | hadoop.hbase.security.access.TestRpcAccessChecks |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.client.TestAsyncProcedureAdminApi |
   |   | hadoop.hbase.client.TestAsyncDecommissionAdminApi |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.wal.TestAsyncFSWALCorruptionDueToDanglingByteBuffer |
   |   | 
hadoop.hbase.master.assignment.TestRegionAssignedToMultipleRegionServers |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationEndpointWithMultipleWAL |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.master.TestRollingRestart |
   |   | hadoop.hbase.coprocessor.TestRegionObserverScannerOpenHook |
   |   | hadoop.hbase.TestAcidGuaranteesWithAdaptivePolicy |
   |   | hadoop.hbase.security.visibility.TestDefaultScanLabelGeneratorStack |
   |   | hadoop.hbase.replication.TestVerifyCellsReplicationEndpoint |
   |   | 
hadoop.hbase.master.replication.TestTransitPeerSyncReplicationStateProcedureBackoff
 |
   |   | 
hadoop.hbase.master.procedure.TestTableDescriptorModificationFromClient |
   |   | hadoop.hbase.client.TestMultiParallel |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-21 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523501782
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 328 | master passed |
   | +1 | compile | 56 | master passed |
   | +1 | checkstyle | 81 | master passed |
   | +1 | shadedjars | 276 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 255 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 253 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 307 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 54 | the patch passed |
   | +1 | checkstyle | 84 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 277 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 986 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 272 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20895 | hbase-server in the patch failed. |
   | -1 | asflicense | 23 | The patch generated 19 ASF License warnings. |
   | | | 24383 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestHColumnDescriptorDefaultVersions |
   |   | hadoop.hbase.filter.TestScanRowPrefix |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.client.TestAsyncTableScanException |
   |   | hadoop.hbase.client.TestCIPutOperationTimeout |
   |   | hadoop.hbase.snapshot.TestMobRestoreFlushSnapshotFromClient |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.regionserver.TestNotCleanupCompactedFileWhenRegionWarmup |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRS |
   |   | hadoop.hbase.coprocessor.TestRegionObserverScannerOpenHook |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.regionserver.TestCleanupCompactedFileAfterFailover |
   |   | hadoop.hbase.mob.TestDefaultMobStoreFlusher |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedureMuitipleRegions |
   |   | hadoop.hbase.client.TestClientOperationInterrupt |
   |   | hadoop.hbase.master.balancer.TestFavoredNodeTableImport |
   |   | hadoop.hbase.client.TestDropTimeoutRequest |
   |   | hadoop.hbase.replication.TestSyncReplicationStandBy |
   |   | hadoop.hbase.client.TestCIPutRpcTimeout |
   |   | hadoop.hbase.client.TestMetaCache |
   |   | hadoop.hbase.master.procedure.TestDeleteNamespaceProcedure |
   |   | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.coprocessor.TestNegativeMemStoreSizeWithSlowCoprocessor |
   |   | hadoop.hbase.coprocessor.TestRegionServerCoprocessorExceptionWithAbort 
|
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.client.TestAsyncTableLocateRegionForDeletedTable |
   |   | hadoop.hbase.master.assignment.TestAssignmentManagerUtil |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.master.balancer.TestRegionLocationFinder |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestart |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | hadoop.hbase.replication.regionserver.TestReplicator |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.TestServerSideScanMetricsFromClientSide |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-21 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523352745
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 121 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 485 | master passed |
   | +1 | compile | 69 | master passed |
   | +1 | checkstyle | 102 | master passed |
   | +1 | shadedjars | 323 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 41 | master passed |
   | 0 | spotbugs | 233 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 229 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 334 | the patch passed |
   | +1 | compile | 65 | the patch passed |
   | +1 | javac | 65 | the patch passed |
   | +1 | checkstyle | 100 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 314 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1095 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 38 | the patch passed |
   | +1 | findbugs | 292 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20743 | hbase-server in the patch failed. |
   | -1 | asflicense | 27 | The patch generated 19 ASF License warnings. |
   | | | 24820 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestCIIncrementRpcTimeout |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.coprocessor.TestIncrementTimeRange |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillMaster |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | 
hadoop.hbase.coprocessor.TestRegionServerCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.util.TestHBaseFsckReplication |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.replication.TestReplicationStatusAfterLagging |
   |   | hadoop.hbase.master.procedure.TestProcedurePriority |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRSWithSeparateOldWALs 
|
   |   | hadoop.hbase.regionserver.wal.TestSecureAsyncWALReplay |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.snapshot.TestSnapshotClientRetries |
   |   | hadoop.hbase.regionserver.TestRemoveRegionMetrics |
   |   | hadoop.hbase.coprocessor.TestRegionObserverBypass |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.master.TestServerCrashProcedureCarryingMetaStuck |
   |   | hadoop.hbase.client.TestZKAsyncRegistry |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.master.TestCatalogJanitorInMemoryStates |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestMetaCache |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.client.TestBlockEvictionFromClient |
   |   | hadoop.hbase.regionserver.throttle.TestFlushWithThroughputController |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.fs.TestBlockReorder |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | 
hadoop.hbase.master.procedure.TestDeleteColumnFamilyProcedureFromClient |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | hadoop.hbase.wal.TestWrongMetaWALFileName |
   |   | 
hadoop.hbase.replication.TestReplicationStatusBothNormalAndRecoveryLagging |
   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-20 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-523243123
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 137 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 357 | master passed |
   | +1 | compile | 55 | master passed |
   | +1 | checkstyle | 100 | master passed |
   | +1 | shadedjars | 299 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 38 | master passed |
   | 0 | spotbugs | 274 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 271 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 330 | the patch passed |
   | +1 | compile | 56 | the patch passed |
   | +1 | javac | 56 | the patch passed |
   | +1 | checkstyle | 95 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 307 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1036 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 35 | the patch passed |
   | +1 | findbugs | 276 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20531 | hbase-server in the patch failed. |
   | -1 | asflicense | 39 | The patch generated 29 ASF License warnings. |
   | | | 24378 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.filter.TestFilterWithScanLimits |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRS |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.master.TestGetReplicationLoad |
   |   | hadoop.hbase.master.assignment.TestAssignmentOnRSCrash |
   |   | hadoop.hbase.client.TestAsyncAdminWithRegionReplicas |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.regionserver.wal.TestWALReplay |
   |   | hadoop.hbase.client.TestCIPutRpcTimeout |
   |   | hadoop.hbase.client.TestRawAsyncTablePartialScan |
   |   | hadoop.hbase.filter.TestFuzzyRowFilterEndToEnd |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.util.TestFSUtils |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi2 |
   |   | hadoop.hbase.master.procedure.TestMasterObserverPostCalls |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.master.assignment.TestRegionStateStore |
   |   | hadoop.hbase.security.visibility.TestDefaultScanLabelGeneratorStack |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestartSplitWithoutZk |
   |   | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.regionserver.TestIsDeleteFailure |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.regionserver.TestCompactionArchiveIOException |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationEndpoint |
   |   | hadoop.hbase.TestRegionLoad |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.replication.TestRemoveFromSerialReplicationPeer |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.snapshot.TestSnapshotClientRetries |
   |   | hadoop.hbase.regionserver.TestCompactionFileNotFound |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.filter.TestColumnRangeFilter |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasScan |
   |   | hadoop.hbase.master.procedure.TestCreateNamespaceProcedure |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.replication.TestSyncReplicationNewRSJoinBetweenRefreshes |
   |   | hadoop.hbase.quotas.TestQuotaObserverChoreWithMiniCluster |
   |   | hadoop.hbase.client.TestMultiParallel |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.regionserver.TestSettingTimeoutOnBlockingPoint |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-20 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522968039
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 98 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 348 | master passed |
   | +1 | compile | 56 | master passed |
   | +1 | checkstyle | 99 | master passed |
   | +1 | shadedjars | 300 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 38 | master passed |
   | 0 | spotbugs | 277 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 275 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 326 | the patch passed |
   | +1 | compile | 56 | the patch passed |
   | +1 | javac | 56 | the patch passed |
   | +1 | checkstyle | 96 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 294 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1025 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 297 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18417 | hbase-server in the patch failed. |
   | -1 | asflicense | 32 | The patch generated 19 ASF License warnings. |
   | | | 22206 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestInfoServers |
   |   | hadoop.hbase.master.TestMasterAbortAndRSGotKilled |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi2 |
   |   | hadoop.hbase.replication.regionserver.TestWALEntryStream |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.regionserver.TestMobStoreScanner |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.regionserver.wal.TestWALReplay |
   |   | hadoop.hbase.security.access.TestAccessController3 |
   |   | hadoop.hbase.master.snapshot.TestSnapshotFileCache |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNoOps |
   |   | hadoop.hbase.client.TestAsyncTableRSCrashPublish |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.replication.TestRemoveFromSerialReplicationPeer |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.replication.regionserver.TestGlobalReplicationThrottler |
   |   | hadoop.hbase.client.TestCIPutOperationTimeout |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSource |
   |   | hadoop.hbase.client.TestAsyncAdminWithRegionReplicas |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.security.visibility.TestVisibilityLablesWithGroups |
   |   | hadoop.hbase.master.TestRetainAssignmentOnRestartSplitWithoutZk |
   |   | hadoop.hbase.client.TestMultipleTimestamps |
   |   | hadoop.hbase.master.TestMasterMetrics |
   |   | hadoop.hbase.master.TestMasterMetricsWrapper |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.client.TestScannersFromClientSide |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.regionserver.TestDeleteMobTable |
   |   | hadoop.hbase.client.TestTableOperationException |
   |   | hadoop.hbase.regionserver.TestRegionServerReportForDuty |
   |   | hadoop.hbase.backup.TestHFileArchiving |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | hadoop.hbase.regionserver.TestRegionMove |
   |   | hadoop.hbase.regionserver.TestHRegion |
   |   | hadoop.hbase.security.access.TestScanEarlyTermination |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.client.TestFlushFromClient |
   |   | hadoop.hbase.master.procedure.TestIgnoreUnknownFamily |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-19 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522850640
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 121 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 501 | master passed |
   | +1 | compile | 73 | master passed |
   | +1 | checkstyle | 109 | master passed |
   | +1 | shadedjars | 357 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 49 | master passed |
   | 0 | spotbugs | 310 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 307 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 460 | the patch passed |
   | +1 | compile | 75 | the patch passed |
   | +1 | javac | 75 | the patch passed |
   | +1 | checkstyle | 108 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 376 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1394 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 47 | the patch passed |
   | +1 | findbugs | 333 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18380 | hbase-server in the patch failed. |
   | -1 | asflicense | 42 | The patch generated 18 ASF License warnings. |
   | | | 23269 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestCIIncrementRpcTimeout |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillMaster |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | 
hadoop.hbase.coprocessor.TestRegionServerCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.replication.TestReplicationStatusAfterLagging |
   |   | hadoop.hbase.master.procedure.TestProcedurePriority |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRSWithSeparateOldWALs 
|
   |   | hadoop.hbase.regionserver.TestRemoveRegionMetrics |
   |   | hadoop.hbase.master.TestServerCrashProcedureCarryingMetaStuck |
   |   | hadoop.hbase.client.TestMultiParallel |
   |   | hadoop.hbase.client.TestZKAsyncRegistry |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestMetaCache |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.client.TestBlockEvictionFromClient |
   |   | hadoop.hbase.regionserver.throttle.TestFlushWithThroughputController |
   |   | hadoop.hbase.replication.TestReplicationProcedureRetry |
   |   | hadoop.hbase.master.TestMasterFailover |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.master.procedure.TestDeleteNamespaceProcedure |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSink |
   |   | hadoop.hbase.fs.TestBlockReorder |
   |   | 
hadoop.hbase.master.procedure.TestDeleteColumnFamilyProcedureFromClient |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSimple |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | 
hadoop.hbase.replication.TestReplicationStatusBothNormalAndRecoveryLagging |
   |   | hadoop.hbase.master.procedure.TestEnableTableProcedure |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.master.locking.TestLockProcedure |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-19 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522824156
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 310 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 77 | master passed |
   | +1 | shadedjars | 265 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 254 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 253 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 290 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 53 | the patch passed |
   | +1 | checkstyle | 74 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 261 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 898 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 32 | the patch passed |
   | +1 | findbugs | 252 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19269 | hbase-server in the patch failed. |
   | -1 | asflicense | 26 | The patch generated 19 ASF License warnings. |
   | | | 22531 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   |   | hadoop.hbase.security.visibility.TestDefaultScanLabelGeneratorStack |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.regionserver.TestIsDeleteFailure |
   |   | hadoop.hbase.regionserver.TestScannerWithBulkload |
   |   | hadoop.hbase.master.TestMetaShutdownHandler |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.TestMultiVersions |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithACL |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.wal.TestFSHLogCorruptionDueToDanglingByteBuffer |
   |   | hadoop.hbase.fs.TestBlockReorderBlockLocation |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasGet |
   |   | hadoop.hbase.TestClusterPortAssignment |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.regionserver.TestHdfsSnapshotHRegion |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocator |
   |   | hadoop.hbase.client.TestAsyncSnapshotAdminApi |
   |   | hadoop.hbase.replication.TestSerialSyncReplication |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-19 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522787635
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 33 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 350 | master passed |
   | +1 | compile | 60 | master passed |
   | +1 | checkstyle | 98 | master passed |
   | +1 | shadedjars | 328 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 39 | master passed |
   | 0 | spotbugs | 298 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 295 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 354 | the patch passed |
   | +1 | compile | 61 | the patch passed |
   | +1 | javac | 61 | the patch passed |
   | +1 | checkstyle | 97 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 305 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1078 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 38 | the patch passed |
   | +1 | findbugs | 301 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19892 | hbase-server in the patch failed. |
   | -1 | asflicense | 33 | The patch generated 19 ASF License warnings. |
   | | | 23844 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.regionserver.TestRegionServerAbort |
   |   | hadoop.hbase.util.compaction.TestMajorCompactor |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.master.TestMasterHandlerFullWhenTransitRegion |
   |   | hadoop.hbase.master.assignment.TestWakeUpUnexpectedProcedure |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.TestServerSideScanMetricsFromClientSide |
   |   | hadoop.hbase.tool.TestCanaryTool |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.replication.TestMasterReplication |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedureMuitipleRegions |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.master.procedure.TestMasterFailoverWithProcedures |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.client.TestResultSizeEstimation |
   |   | hadoop.hbase.master.TestClusterRestartFailover |
   |   | hadoop.hbase.client.TestDropTimeoutRequest |
   |   | hadoop.hbase.client.TestEnableTable |
   |   | hadoop.hbase.coprocessor.TestMetaTableMetrics |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicas |
   |   | hadoop.hbase.regionserver.TestRegionOpen |
   |   | 
hadoop.hbase.master.procedure.TestTableDescriptorModificationFromClient |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientGetCompactionState 
|
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.replication.TestReplicationMetricsforUI |
   |   | hadoop.hbase.client.TestCheckAndMutate |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.regionserver.TestNotCleanupCompactedFileWhenRegionWarmup |
   |   | hadoop.hbase.client.TestResultFromCoprocessor |
   |   | hadoop.hbase.quotas.TestFileArchiverNotifierImpl |
   |   | hadoop.hbase.master.balancer.TestBalancerStatusTagInJMXMetrics |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationSyncUpToolWithMultipleWAL |
   |   | hadoop.hbase.client.TestRegionLocator |
   |   | hadoop.hbase.TestClientClusterStatus |
   |   | hadoop.hbase.coprocessor.TestMasterObserverToModifyTableSchema |
   |   | hadoop.hbase.regionserver.wal.TestAsyncProtobufLog |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-18 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522299970
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 39 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 323 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 85 | master passed |
   | +1 | shadedjars | 277 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 243 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 242 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 302 | the patch passed |
   | +1 | compile | 54 | the patch passed |
   | +1 | javac | 54 | the patch passed |
   | +1 | checkstyle | 80 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 277 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 942 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 31 | the patch passed |
   | +1 | findbugs | 246 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18106 | hbase-server in the patch failed. |
   | -1 | asflicense | 21 | The patch generated 18 ASF License warnings. |
   | | | 21485 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.TestNamespace |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | 
hadoop.hbase.coprocessor.TestRegionServerCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.regionserver.TestNotCleanupCompactedFileWhenRegionWarmup |
   |   | hadoop.hbase.quotas.TestLowLatencySpaceQuotas |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientAfterSplittingRegion 
|
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.regionserver.TestRowTooBig |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | 
hadoop.hbase.master.assignment.TestRegionAssignedToMultipleRegionServers |
   |   | hadoop.hbase.coprocessor.TestOpenTableInCoprocessor |
   |   | hadoop.hbase.filter.TestFilterListOnMini |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.coprocessor.TestIncrementAndAppendWithNullResult |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.TestMovedRegionsCleaner |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-17 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522280250
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 39 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 319 | master passed |
   | +1 | compile | 54 | master passed |
   | +1 | checkstyle | 82 | master passed |
   | +1 | shadedjars | 282 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 243 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 241 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 309 | the patch passed |
   | +1 | compile | 52 | the patch passed |
   | +1 | javac | 52 | the patch passed |
   | +1 | checkstyle | 81 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 274 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 934 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 238 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 17607 | hbase-server in the patch failed. |
   | -1 | asflicense | 23 | The patch generated 22 ASF License warnings. |
   | | | 20974 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.regionserver.TestRegionServerAbortTimeout |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.quotas.TestLowLatencySpaceQuotas |
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.ipc.TestNettyRpcServer |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestAsyncRegionLocator |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.coprocessor.TestOpenTableInCoprocessor |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.client.TestAppendFromClientSide |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.regionserver.TestOpenSeqNumUnexpectedIncrease |
   |   | hadoop.hbase.TestMovedRegionsCleaner |
   |   | hadoop.hbase.client.TestAsyncReplicationAdminApi |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.ipc.TestRpcClientLeaks |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | hadoop.hbase.regionserver.TestClearRegionBlockCache |
   |   | hadoop.hbase.client.TestResultFromCoprocessor |
   |   | hadoop.hbase.master.TestGetInfoPort |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-17 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522272297
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 41 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 323 | master passed |
   | +1 | compile | 53 | master passed |
   | +1 | checkstyle | 82 | master passed |
   | +1 | shadedjars | 284 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 243 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 241 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 297 | the patch passed |
   | +1 | compile | 54 | the patch passed |
   | +1 | javac | 54 | the patch passed |
   | +1 | checkstyle | 78 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 273 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 957 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 253 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 17618 | hbase-server in the patch failed. |
   | -1 | asflicense | 25 | The patch generated 19 ASF License warnings. |
   | | | 21029 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.regionserver.wal.TestLogRollPeriod |
   |   | hadoop.hbase.regionserver.wal.TestAsyncFSWAL |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithAbort |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.master.TestMasterFailoverBalancerPersistence |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.snapshot.TestRestoreSnapshotHelper |
   |   | hadoop.hbase.fs.TestBlockReorderBlockLocation |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.regionserver.TestClearRegionBlockCache |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | hadoop.hbase.regionserver.wal.TestAsyncLogRollPeriod |
   |   | hadoop.hbase.security.token.TestGenerateDelegationToken |
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.client.TestIncrementFromClientSideWithCoprocessor |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocator |
   |   | hadoop.hbase.client.TestAsyncSnapshotAdminApi |
   |   | hadoop.hbase.replication.TestSerialSyncReplication |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-17 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522256950
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 39 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 325 | master passed |
   | +1 | compile | 51 | master passed |
   | +1 | checkstyle | 81 | master passed |
   | +1 | shadedjars | 281 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 244 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 243 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 300 | the patch passed |
   | +1 | compile | 52 | the patch passed |
   | +1 | javac | 52 | the patch passed |
   | +1 | checkstyle | 80 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 273 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 928 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 249 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19543 | hbase-server in the patch failed. |
   | -1 | asflicense | 25 | The patch generated 19 ASF License warnings. |
   | | | 22912 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.security.visibility.TestVisibilityWithCheckAuths |
   |   | hadoop.hbase.quotas.TestLowLatencySpaceQuotas |
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.ipc.TestNettyRpcServer |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.client.TestAsyncRegionLocator |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.filter.TestFilterListOnMini |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.coprocessor.TestIncrementAndAppendWithNullResult |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.regionserver.wal.TestAsyncFSWAL |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | hadoop.hbase.master.TestClusterRestartFailover |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.client.TestAsyncReplicationAdminApi |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.ipc.TestRpcClientLeaks |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-17 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522215480
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 159 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 396 | master passed |
   | +1 | compile | 54 | master passed |
   | +1 | checkstyle | 82 | master passed |
   | +1 | shadedjars | 280 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 240 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 237 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 306 | the patch passed |
   | +1 | compile | 55 | the patch passed |
   | +1 | javac | 55 | the patch passed |
   | +1 | checkstyle | 83 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 277 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 942 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 245 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19226 | hbase-server in the patch failed. |
   | -1 | asflicense | 54 | The patch generated 33 ASF License warnings. |
   | | | 22859 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.filter.TestFilterListOnMini |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.security.access.TestAccessControlFilter |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.mob.TestMobDataBlockEncoding |
   |   | hadoop.hbase.client.TestCheckAndMutate |
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNoOps |
   |   | hadoop.hbase.tool.TestBulkLoadHFilesSplitRecovery |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithACL |
   |   | hadoop.hbase.master.procedure.TestMasterObserverPostCalls |
   |   | hadoop.hbase.master.TestMasterFailover |
   |   | 
hadoop.hbase.master.assignment.TestReportRegionStateTransitionFromDeadServer |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.client.TestAsyncRegionLocator |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.procedure.TestFailedProcCleanup |
   |   | hadoop.hbase.wal.TestWALSplitBoundedLogWriterCreation |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.wal.TestSyncReplicationWALProvider |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationEndpoint |
   |   | hadoop.hbase.wal.TestFSHLogCorruptionDueToDanglingByteBuffer |
   |   | hadoop.hbase.security.visibility.TestDefaultScanLabelGeneratorStack |
   |   | hadoop.hbase.master.TestMigrateNamespaceTable |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocatorConcurrenyLimit |
   |   | hadoop.hbase.filter.TestColumnRangeFilter |
   |   | hadoop.hbase.master.assignment.TestAssignmentManager |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.client.TestScannersFromClientSide |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.replication.regionserver.TestGlobalReplicationThrottler |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.replication.TestReplicationMetricsforUI |
   |   | hadoop.hbase.TestRegionLoad |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSink |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-16 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522072792
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 85 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 396 | master passed |
   | +1 | compile | 69 | master passed |
   | +1 | checkstyle | 101 | master passed |
   | +1 | shadedjars | 305 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 266 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 264 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 331 | the patch passed |
   | +1 | compile | 57 | the patch passed |
   | +1 | javac | 57 | the patch passed |
   | +1 | checkstyle | 96 | the patch passed |
   | +1 | whitespace | 1 | The patch has no whitespace issues. |
   | +1 | shadedjars | 297 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1025 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 35 | the patch passed |
   | +1 | findbugs | 275 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19874 | hbase-server in the patch failed. |
   | -1 | asflicense | 30 | The patch generated 18 ASF License warnings. |
   | | | 23680 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestUpdateConfiguration |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationEndpointWithMultipleWAL |
   |   | hadoop.hbase.client.TestScanWithoutFetchingData |
   |   | hadoop.hbase.client.TestTableOperationException |
   |   | hadoop.hbase.coprocessor.TestRegionObserverBypass |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOnNewVersionBehaviorTable |
   |   | hadoop.hbase.wal.TestWALSplitBoundedLogWriterCreation |
   |   | hadoop.hbase.io.asyncfs.TestFanOutOneBlockAsyncDFSOutput |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.regionserver.wal.TestWALReplayCompressed |
   |   | hadoop.hbase.client.TestScannersFromClientSide |
   |   | hadoop.hbase.master.procedure.TestMasterProcedureEvents |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.coprocessor.TestWALObserver |
   |   | hadoop.hbase.client.TestFromClientSideScanExcpetion |
   |   | hadoop.hbase.client.TestResultSizeEstimation |
   |   | hadoop.hbase.client.TestSplitOrMergeStatus |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.master.TestWarmupRegion |
   |   | hadoop.hbase.wal.TestFSHLogCorruptionDueToDanglingByteBuffer |
   |   | hadoop.hbase.quotas.TestQuotaTableUtil |
   |   | hadoop.hbase.regionserver.TestRowTooBig |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.master.TestMergeTableRegionsWhileRSCrash |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.wal.TestWALSplitCompressed |
   |   | hadoop.hbase.quotas.TestLowLatencySpaceQuotas |
   |   | hadoop.hbase.client.TestAdmin2 |
   |   | hadoop.hbase.regionserver.TestRegionFavoredNodes |
   |   | hadoop.hbase.master.assignment.TestSCPGetRegionsRace |
   |   | hadoop.hbase.TestClusterBootOrder |
   |   | hadoop.hbase.client.TestSnapshotTemporaryDirectory |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.client.TestAsyncBufferMutator |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi2 |
   |   | hadoop.hbase.quotas.TestQuotaObserverChoreWithMiniCluster |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-16 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-522003967
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 124 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 480 | master passed |
   | +1 | compile | 73 | master passed |
   | +1 | checkstyle | 117 | master passed |
   | +1 | shadedjars | 374 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 52 | master passed |
   | 0 | spotbugs | 300 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 298 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 430 | the patch passed |
   | +1 | compile | 71 | the patch passed |
   | +1 | javac | 71 | the patch passed |
   | +1 | checkstyle | 107 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 362 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1405 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 51 | the patch passed |
   | +1 | findbugs | 316 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18115 | hbase-server in the patch failed. |
   | -1 | asflicense | 29 | The patch generated 25 ASF License warnings. |
   | | | 22944 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.tool.TestBulkLoadHFiles |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi2 |
   |   | hadoop.hbase.client.TestAdmin1 |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.quotas.TestSnapshotQuotaObserverChore |
   |   | 
hadoop.hbase.master.procedure.TestReopenTableRegionsProcedureInfiniteLoop |
   |   | hadoop.hbase.security.access.TestWithDisabledAuthorization |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelReplicationWithExpAsString |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.master.assignment.TestRegionBypass |
   |   | hadoop.hbase.client.TestAsyncMetaRegionLocator |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi3 |
   |   | hadoop.hbase.client.TestMvccConsistentScanner |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.client.TestAsyncTableScanException |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationSyncUpToolWithMultipleAsyncWAL 
|
   |   | hadoop.hbase.client.TestReplicasClient |
   |   | hadoop.hbase.filter.TestColumnRangeFilter |
   |   | hadoop.hbase.master.procedure.TestCreateNamespaceProcedure |
   |   | hadoop.hbase.regionserver.TestPerColumnFamilyFlush |
   |   | hadoop.hbase.client.TestSplitOrMergeStatus |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.client.TestAsyncAdminMasterSwitch |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.master.cleaner.TestHFileCleaner |
   |   | hadoop.hbase.filter.TestMultiRowRangeFilter |
   |   | hadoop.hbase.master.procedure.TestIgnoreUnknownFamily |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | hadoop.hbase.master.procedure.TestSCP |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationEndpointWithMultipleWAL |
   |   | hadoop.hbase.TestAcidGuaranteesWithNoInMemCompaction |
   |   | hadoop.hbase.regionserver.TestEndToEndSplitTransaction |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | hadoop.hbase.client.TestAsyncClientPauseForCallQueueTooBig |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.io.TestFileLink |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.master.TestGetInfoPort |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.master.assignment.TestAssignmentOnRSCrash |
   |   | hadoop.hbase.TestRegionLoad |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-16 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521940248
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 80 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 369 | master passed |
   | +1 | compile | 59 | master passed |
   | +1 | checkstyle | 97 | master passed |
   | +1 | shadedjars | 315 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 44 | master passed |
   | 0 | spotbugs | 312 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 308 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 338 | the patch passed |
   | +1 | compile | 59 | the patch passed |
   | +1 | javac | 59 | the patch passed |
   | +1 | checkstyle | 95 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 307 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1053 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 42 | the patch passed |
   | +1 | findbugs | 319 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20354 | hbase-server in the patch failed. |
   | -1 | asflicense | 29 | The patch generated 18 ASF License warnings. |
   | | | 24283 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.client.TestCIIncrementRpcTimeout |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | hadoop.hbase.coprocessor.TestMasterObserver |
   |   | hadoop.hbase.coprocessor.TestIncrementTimeRange |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillMaster |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | hadoop.hbase.util.TestHBaseFsckReplication |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocatorConcurrenyLimit |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.TestSerialReplication |
   |   | hadoop.hbase.master.procedure.TestProcedurePriority |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRSWithSeparateOldWALs 
|
   |   | hadoop.hbase.regionserver.wal.TestSecureAsyncWALReplay |
   |   | hadoop.hbase.replication.TestMultiSlaveReplication |
   |   | hadoop.hbase.regionserver.TestRemoveRegionMetrics |
   |   | hadoop.hbase.coprocessor.TestRegionObserverBypass |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.master.TestServerCrashProcedureCarryingMetaStuck |
   |   | hadoop.hbase.client.TestZKAsyncRegistry |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.master.TestCatalogJanitorInMemoryStates |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.replication.TestReplicationSyncUpToolWithBulkLoadedData |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestMetaCache |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedWithRecovery
 |
   |   | hadoop.hbase.regionserver.throttle.TestFlushWithThroughputController |
   |   | 
hadoop.hbase.replication.regionserver.TestDrainReplicationQueuesForStandBy |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.master.procedure.TestDeleteNamespaceProcedure |
   |   | hadoop.hbase.replication.regionserver.TestReplicationSink |
   |   | hadoop.hbase.fs.TestBlockReorder |
   |   | 
hadoop.hbase.master.procedure.TestDeleteColumnFamilyProcedureFromClient |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | 
hadoop.hbase.replication.TestReplicationStatusBothNormalAndRecoveryLagging |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-16 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521925700
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 38 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 314 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 75 | master passed |
   | +1 | shadedjars | 260 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 34 | master passed |
   | 0 | spotbugs | 248 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 245 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 284 | the patch passed |
   | +1 | compile | 51 | the patch passed |
   | +1 | javac | 51 | the patch passed |
   | +1 | checkstyle | 75 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 262 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 896 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 244 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 19748 | hbase-server in the patch failed. |
   | -1 | asflicense | 25 | The patch generated 18 ASF License warnings. |
   | | | 22988 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.regionserver.TestRegionFavoredNodes |
   |   | hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush |
   |   | hadoop.hbase.security.visibility.TestDefaultScanLabelGeneratorStack |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.TestServerSideScanMetricsFromClientSide |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | hadoop.hbase.client.TestHbck |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithACL |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.regionserver.TestDeleteMobTable |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.fs.TestBlockReorderBlockLocation |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasGet |
   |   | hadoop.hbase.quotas.TestSpaceQuotasWithSnapshots |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleWAL
 |
   |   | hadoop.hbase.regionserver.wal.TestAsyncLogRollPeriod |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.security.token.TestGenerateDelegationToken |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.client.TestRegionLocator |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.replication.TestAddToSerialReplicationPeer |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocator |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-15 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521868301
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 76 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 353 | master passed |
   | +1 | compile | 55 | master passed |
   | +1 | checkstyle | 83 | master passed |
   | +1 | shadedjars | 278 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 34 | master passed |
   | 0 | spotbugs | 246 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 243 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 290 | the patch passed |
   | +1 | compile | 52 | the patch passed |
   | +1 | javac | 52 | the patch passed |
   | +1 | checkstyle | 77 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 285 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 949 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 30 | the patch passed |
   | +1 | findbugs | 245 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18031 | hbase-server in the patch failed. |
   | -1 | asflicense | 24 | The patch generated 20 ASF License warnings. |
   | | | 21479 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestNamespace |
   |   | hadoop.hbase.filter.TestScanRowPrefix |
   |   | hadoop.hbase.master.assignment.TestRegionReplicaSplit |
   |   | hadoop.hbase.master.procedure.TestSplitWALProcedure |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.security.visibility.TestVisibilityWithCheckAuths |
   |   | hadoop.hbase.quotas.TestLowLatencySpaceQuotas |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientAfterSplittingRegion 
|
   |   | hadoop.hbase.client.TestCIGetRpcTimeout |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.ipc.TestNettyRpcServer |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | 
hadoop.hbase.master.assignment.TestRegionAssignedToMultipleRegionServers |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.regionserver.TestHRegion |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | hadoop.hbase.quotas.TestQuotaTableUtil |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.TestLocalHBaseCluster |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.regionserver.TestScannerWithCorruptHFile |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.master.assignment.TestAssignmentManager |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-15 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521583630
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 70 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 360 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 80 | master passed |
   | +1 | shadedjars | 273 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 260 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 256 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 306 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 53 | the patch passed |
   | +1 | checkstyle | 76 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 261 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 913 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 32 | the patch passed |
   | +1 | findbugs | 245 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18817 | hbase-server in the patch failed. |
   | -1 | asflicense | 65 | The patch generated 17 ASF License warnings. |
   | | | 22285 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.master.TestMasterFailoverBalancerPersistence |
   |   | hadoop.hbase.master.balancer.TestFavoredNodeTableImport |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.client.TestSplitOrMergeAtTableLevel |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.replication.TestSyncReplicationStandBy |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.regionserver.TestMobStoreScanner |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | 
hadoop.hbase.replication.regionserver.TestRaceWhenCreatingReplicationSource |
   |   | hadoop.hbase.replication.TestReplicationMetricsforUI |
   |   | hadoop.hbase.client.TestTableFavoredNodes |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationSyncUpToolWithMultipleWAL |
   |   | hadoop.hbase.quotas.TestSuperUserQuotaPermissions |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | hadoop.hbase.master.assignment.TestSplitTableRegionProcedure |
   |   | hadoop.hbase.io.asyncfs.TestSaslFanOutOneBlockAsyncDFSOutput |
   |   | hadoop.hbase.master.TestClusterRestartSplitWithoutZk |
   |   | hadoop.hbase.regionserver.TestRegionServerMetrics |
   |   | hadoop.hbase.wal.TestWALOpenAfterDNRollingStart |
   |   | hadoop.hbase.replication.TestReplicationSyncUpTool |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | hadoop.hbase.master.replication.TestModifyPeerProcedureRetryBackoff |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestAsyncNonMetaRegionLocatorConcurrenyLimit |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.TestHBaseOnOtherDfsCluster |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.replication.TestReplicationEmptyWALRecovery |
   |   | hadoop.hbase.replication.TestReplicationChangingPeerRegionservers |
   |   | hadoop.hbase.master.procedure.TestDeleteTableProcedure |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.quotas.TestSnapshotQuotaObserverChore |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   |   | hadoop.hbase.regionserver.TestDeleteMobTable |
   |   | hadoop.hbase.fs.TestBlockReorderBlockLocation |
   |   | hadoop.hbase.client.TestMetaWithReplicas |
   |   | hadoop.hbase.TestGlobalMemStoreSize |
   |   | hadoop.hbase.client.TestReplicaWithCluster |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.regionserver.TestIsDeleteFailure |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-15 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521560284
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 40 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 320 | master passed |
   | +1 | compile | 53 | master passed |
   | +1 | checkstyle | 86 | master passed |
   | +1 | shadedjars | 276 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 37 | master passed |
   | 0 | spotbugs | 248 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 246 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 300 | the patch passed |
   | +1 | compile | 51 | the patch passed |
   | +1 | javac | 51 | the patch passed |
   | +1 | checkstyle | 75 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 258 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 917 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 34 | the patch passed |
   | +1 | findbugs | 247 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 18780 | hbase-server in the patch failed. |
   | -1 | asflicense | 24 | The patch generated 35 ASF License warnings. |
   | | | 22117 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.TestNamespace |
   |   | hadoop.hbase.filter.TestScanRowPrefix |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.regionserver.TestNotCleanupCompactedFileWhenRegionWarmup |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientAfterSplittingRegion 
|
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.client.TestMobSnapshotFromClient |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable |
   |   | hadoop.hbase.ipc.TestNettyRpcServer |
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | 
hadoop.hbase.master.assignment.TestRegionAssignedToMultipleRegionServers |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.coprocessor.TestOpenTableInCoprocessor |
   |   | hadoop.hbase.filter.TestFilterListOnMini |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | 
hadoop.hbase.coprocessor.TestRegionObserverForAddingMutationsFromCoprocessors |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.regionserver.wal.TestAsyncFSWAL |
   |   | hadoop.hbase.util.TestFromClientSide3WoUnsafe |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | hadoop.hbase.snapshot.TestFlushSnapshotFromClient |
   |   | hadoop.hbase.quotas.TestQuotaTableUtil |
   |   | hadoop.hbase.client.TestAsyncTableScanMetrics |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.TestMovedRegionsCleaner |
   |   | hadoop.hbase.wal.TestWALSplitBoundedLogWriterCreation |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientClone |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-14 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521445636
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 39 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 371 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 75 | master passed |
   | +1 | shadedjars | 259 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 33 | master passed |
   | 0 | spotbugs | 268 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 266 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 288 | the patch passed |
   | +1 | compile | 51 | the patch passed |
   | +1 | javac | 51 | the patch passed |
   | +1 | checkstyle | 75 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 262 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 887 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 250 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 16222 | hbase-server in the patch failed. |
   | -1 | asflicense | 31 | The patch generated 22 ASF License warnings. |
   | | | 19555 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithAbort |
   |   | hadoop.hbase.client.TestHbck |
   |   | hadoop.hbase.client.TestCIPutRpcTimeout |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.TestMasterFailoverBalancerPersistence |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.regionserver.TestDeleteMobTable |
   |   | hadoop.hbase.client.TestEnableTable |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.regionserver.TestHdfsSnapshotHRegion |
   |   | hadoop.hbase.master.procedure.TestSCPWithMetaWithoutZKCoordinated |
   |   | hadoop.hbase.client.TestGetScanPartialResult |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.replication.TestSerialSyncReplication |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.master.procedure.TestIgnoreUnknownFamily |
   |   | hadoop.hbase.replication.TestReplicationKillSlaveRS |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationChecker |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.regionserver.TestNotCleanupCompactedFileWhenRegionWarmup |
   |   | hadoop.hbase.client.TestAsyncTableRegionLocator |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMeta |
   |   | hadoop.hbase.tool.TestSecureBulkLoadHFiles |
   |   | hadoop.hbase.master.procedure.TestCloneSnapshotProcedure |
   |   | hadoop.hbase.master.snapshot.TestSnapshotWhileRSCrashes |
   |   | hadoop.hbase.client.TestCIGetOperationTimeout |
   |   | hadoop.hbase.util.TestMiniClusterLoadParallel |
   |   | hadoop.hbase.client.TestSnapshotWithAcl |
   |   | 
hadoop.hbase.master.procedure.TestSCPWithMetaWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestIllegalTableDescriptor |
   |   | hadoop.hbase.coprocessor.TestNegativeMemStoreSizeWithSlowCoprocessor |
   |   | hadoop.hbase.client.TestAsyncClientPushback |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-13 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521091207
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 37 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 323 | master passed |
   | +1 | compile | 52 | master passed |
   | +1 | checkstyle | 85 | master passed |
   | +1 | shadedjars | 275 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 245 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 243 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 292 | the patch passed |
   | +1 | compile | 53 | the patch passed |
   | +1 | javac | 53 | the patch passed |
   | +1 | checkstyle | 81 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 263 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 898 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 256 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 14526 | hbase-server in the patch failed. |
   | -1 | asflicense | 24 | The patch generated 25 ASF License warnings. |
   | | | 17849 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.TestNamespace |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.security.visibility.TestVisibilityWithCheckAuths |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientAfterSplittingRegion 
|
   |   | hadoop.hbase.coprocessor.TestCoprocessorMetrics |
   |   | hadoop.hbase.client.TestMobSnapshotFromClient |
   |   | hadoop.hbase.procedure.TestFailedProcCleanup |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.security.token.TestDelegationTokenWithEncryption |
   |   | hadoop.hbase.regionserver.TestRegionServerHostname |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestAsyncRegionLocator |
   |   | hadoop.hbase.master.TestShutdownBackupMaster |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithRestartScenarios |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.filter.TestFilterListOnMini |
   |   | hadoop.hbase.client.TestServerBusyException |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.client.TestAppendFromClientSide |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.coprocessor.TestIncrementAndAppendWithNullResult |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithRemove |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.regionserver.wal.TestAsyncFSWAL |
   |   | hadoop.hbase.master.procedure.TestDisableTableProcedure |
   |   | hadoop.hbase.quotas.TestQuotaTableUtil |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.TestMovedRegionsCleaner |
   |   | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.wal.TestWALSplitBoundedLogWriterCreation |
   |   | hadoop.hbase.client.TestAsyncReplicationAdminApi |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   | hadoop.hbase.coprocessor.TestRegionObserverInterface |
   |   | hadoop.hbase.regionserver.TestClearRegionBlockCache |
   |   | hadoop.hbase.client.TestResultFromCoprocessor |
   |   | hadoop.hbase.client.TestAlwaysSetScannerId |
   |   | hadoop.hbase.master.TestGetInfoPort |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.coprocessor.TestIncrementTimeRange |
   |   | hadoop.hbase.client.TestAsyncRegionAdminApi |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-13 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-521052437
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 38 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 326 | master passed |
   | +1 | compile | 53 | master passed |
   | +1 | checkstyle | 80 | master passed |
   | +1 | shadedjars | 270 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 36 | master passed |
   | 0 | spotbugs | 247 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 245 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 300 | the patch passed |
   | +1 | compile | 52 | the patch passed |
   | +1 | javac | 52 | the patch passed |
   | +1 | checkstyle | 82 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 275 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 915 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 34 | the patch passed |
   | +1 | findbugs | 243 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 16395 | hbase-server in the patch failed. |
   | -1 | asflicense | 25 | The patch generated 19 ASF License warnings. |
   | | | 19740 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | hadoop.hbase.regionserver.TestRegionFavoredNodes |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.regionserver.TestIsDeleteFailure |
   |   | hadoop.hbase.regionserver.wal.TestLogRollPeriod |
   |   | hadoop.hbase.snapshot.TestSnapshotWhenChoreCleaning |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.coprocessor.TestMasterCoprocessorExceptionWithAbort |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | hadoop.hbase.client.TestHbck |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.master.TestMasterFailoverBalancerPersistence |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.security.access.TestAccessController |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.snapshot.TestRestoreSnapshotHelper |
   |   | hadoop.hbase.wal.TestFSHLogCorruptionDueToDanglingByteBuffer |
   |   | hadoop.hbase.fs.TestBlockReorderBlockLocation |
   |   | hadoop.hbase.filter.TestFuzzyRowAndColumnRangeFilter |
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.quotas.TestSpaceQuotasWithSnapshots |
   |   | hadoop.hbase.master.TestMasterFileSystem |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleWAL
 |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.regionserver.wal.TestAsyncLogRollPeriod |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.client.TestIncrementFromClientSideWithCoprocessor |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.client.TestRegionLocator |
   |   | hadoop.hbase.TestJMXConnectorServer |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestSnapshotCloneIndependence |
   |   | hadoop.hbase.replication.TestSerialSyncReplication |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.master.procedure.TestIgnoreUnknownFamily |
   |   | hadoop.hbase.client.TestAppendFromClientSide |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOnNewVersionBehaviorTable |
   |   | hadoop.hbase.replication.TestReplicationKillMasterRSCompressed |
   |   | hadoop.hbase.master.TestMasterAbortAndRSGotKilled |
   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-13 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-520875693
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 75 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 320 | master passed |
   | +1 | compile | 53 | master passed |
   | +1 | checkstyle | 80 | master passed |
   | +1 | shadedjars | 294 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 35 | master passed |
   | 0 | spotbugs | 240 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 238 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 341 | the patch passed |
   | +1 | compile | 57 | the patch passed |
   | +1 | javac | 57 | the patch passed |
   | +1 | checkstyle | 80 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 267 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 894 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 32 | the patch passed |
   | +1 | findbugs | 252 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 14313 | hbase-server in the patch failed. |
   | -1 | asflicense | 40 | The patch generated 19 ASF License warnings. |
   | | | 17778 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationEmptyWALRecovery |
   |   | hadoop.hbase.client.TestBlockEvictionFromClient |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.wal.TestSyncReplicationWALProvider |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.client.TestAsyncTableRSCrashPublish |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi2 |
   |   | hadoop.hbase.TestHColumnDescriptorDefaultVersions |
   |   | hadoop.hbase.master.TestMasterFailoverBalancerPersistence |
   |   | hadoop.hbase.wal.TestWrongMetaWALFileName |
   |   | hadoop.hbase.replication.regionserver.TestSerialReplicationEndpoint |
   |   | 
hadoop.hbase.master.assignment.TestReportRegionStateTransitionFromDeadServer |
   |   | hadoop.hbase.master.assignment.TestModifyTableWhileMerging |
   |   | hadoop.hbase.regionserver.wal.TestAsyncFSWAL |
   |   | hadoop.hbase.tool.TestCanaryTool |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpointNoMaster
 |
   |   | hadoop.hbase.security.access.TestAccessControlFilter |
   |   | hadoop.hbase.client.TestAsyncAdminMasterSwitch |
   |   | hadoop.hbase.security.access.TestCellACLs |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithoutZKCoordinated |
   |   | hadoop.hbase.coprocessor.TestNegativeMemStoreSizeWithSlowCoprocessor |
   |   | hadoop.hbase.coprocessor.TestWALObserver |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationEndpointWithMultipleAsyncWAL |
   |   | hadoop.hbase.client.TestMobSnapshotCloneIndependence |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | hadoop.hbase.master.replication.TestRegisterPeerWorkerWhenRestarting |
   |   | hadoop.hbase.client.TestMetaCache |
   |   | hadoop.hbase.master.TestMigrateNamespaceTable |
   |   | hadoop.hbase.client.TestClientTimeouts |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi |
   |   | hadoop.hbase.replication.regionserver.TestSyncReplicationShipperQuit |
   |   | hadoop.hbase.regionserver.TestPerColumnFamilyFlush |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.util.TestFSUtils |
   |   | hadoop.hbase.client.TestScannersFromClientSide |
   |   | hadoop.hbase.master.procedure.TestSCPWithoutMetaWithReplicas |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.filter.TestMultiRowRangeFilter |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientAfterSplittingRegion |
   |   | hadoop.hbase.client.TestAsyncTableLocatePrefetch |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.master.procedure.TestEnableTableProcedure |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-13 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-520854241
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 319 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 1 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 468 | master passed |
   | +1 | compile | 117 | master passed |
   | +1 | checkstyle | 102 | master passed |
   | +1 | shadedjars | 382 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 47 | master passed |
   | 0 | spotbugs | 372 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 362 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 485 | the patch passed |
   | +1 | compile | 65 | the patch passed |
   | +1 | javac | 65 | the patch passed |
   | +1 | checkstyle | 100 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 345 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 1240 | Patch does not cause any errors with Hadoop 
2.8.5 2.9.2 or 3.1.2. |
   | +1 | javadoc | 49 | the patch passed |
   | +1 | findbugs | 332 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 14073 | hbase-server in the patch failed. |
   | -1 | asflicense | 27 | The patch generated 19 ASF License warnings. |
   | | | 19056 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.master.TestServerCrashProcedureStuck |
   |   | hadoop.hbase.master.TestServerCrashProcedureCarryingMetaStuck |
   |   | hadoop.hbase.util.TestMiniClusterLoadEncoded |
   |   | hadoop.hbase.TestClientOperationTimeout |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientError |
   |   | hadoop.hbase.client.TestResultScannerCursor |
   |   | hadoop.hbase.master.procedure.TestModifyNamespaceProcedure |
   |   | hadoop.hbase.security.visibility.TestWithDisabledAuthorization |
   |   | hadoop.hbase.replication.TestVerifyCellsReplicationEndpoint |
   |   | hadoop.hbase.replication.TestSyncReplicationNewRSJoinBetweenRefreshes |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientAfterTruncate |
   |   | hadoop.hbase.security.access.TestNamespaceCommands |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.client.TestCIBadHostname |
   |   | hadoop.hbase.regionserver.TestRegionServerAbort |
   |   | 
hadoop.hbase.regionserver.throttle.TestCompactionWithThroughputController |
   |   | hadoop.hbase.client.TestFromClientSide3 |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientNormal |
   |   | hadoop.hbase.regionserver.TestEncryptionRandomKeying |
   |   | hadoop.hbase.client.TestAsyncTableRSCrashPublish |
   |   | hadoop.hbase.TestClientClusterStatus |
   |   | hadoop.hbase.replication.master.TestRecoverStandbyProcedure |
   |   | hadoop.hbase.client.TestAsyncResultScannerCursor |
   |   | hadoop.hbase.master.TestRollingRestart |
   |   | hadoop.hbase.client.TestAsyncTableScanAll |
   |   | hadoop.hbase.client.replication.TestReplicationAdminForSyncReplication 
|
   |   | hadoop.hbase.TestHColumnDescriptorDefaultVersions |
   |   | hadoop.hbase.procedure.TestFailedProcCleanup |
   |   | hadoop.hbase.client.TestTimestampsFilter |
   |   | hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable |
   |   | hadoop.hbase.client.TestClientScannerRPCTimeout |
   |   | hadoop.hbase.TestSplitMerge |
   |   | hadoop.hbase.client.TestScannersFromClientSide2 |
   |   | hadoop.hbase.client.TestAsyncQuotaAdminApi |
   |   | hadoop.hbase.master.assignment.TestCloseRegionWhileRSCrash |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.replication.TestSyncReplicationActive |
   |   | hadoop.hbase.client.TestAsyncProcedureAdminApi |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNoOps |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.util.TestConnectionCache |
   |   | hadoop.hbase.io.encoding.TestChangingEncoding |
   |   | hadoop.hbase.client.TestLimitedScanWithFilter |
   |   | 
hadoop.hbase.client.TestMobCloneSnapshotFromClientCloneLinksAfterDelete |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithACL |
   |   | hadoop.hbase.TestNamespace |
   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-13 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-520828715
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 43 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 366 | master passed |
   | +1 | compile | 53 | master passed |
   | +1 | checkstyle | 76 | master passed |
   | +1 | shadedjars | 259 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 32 | master passed |
   | 0 | spotbugs | 246 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 242 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 282 | the patch passed |
   | +1 | compile | 55 | the patch passed |
   | +1 | javac | 55 | the patch passed |
   | +1 | checkstyle | 74 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 261 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 889 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 33 | the patch passed |
   | +1 | findbugs | 247 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 20247 | hbase-server in the patch failed. |
   | -1 | asflicense | 27 | The patch generated 19 ASF License warnings. |
   | | | 23558 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.master.TestSplitRegionWhileRSCrash |
   |   | hadoop.hbase.client.TestSnapshotFromClientWithRegionReplicas |
   |   | hadoop.hbase.client.TestAsyncTable |
   |   | hadoop.hbase.security.access.TestCoprocessorWhitelistMasterObserver |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithCustomVisLabService |
   |   | hadoop.hbase.quotas.TestMasterQuotasObserver |
   |   | hadoop.hbase.security.visibility.TestVisibilityWithCheckAuths |
   |   | hadoop.hbase.quotas.TestLowLatencySpaceQuotas |
   |   | hadoop.hbase.client.TestMobCloneSnapshotFromClientAfterSplittingRegion 
|
   |   | hadoop.hbase.client.TestMobSnapshotFromClient |
   |   | hadoop.hbase.master.TestDeadServer |
   |   | hadoop.hbase.TestIOFencing |
   |   | hadoop.hbase.master.locking.TestLockManager |
   |   | hadoop.hbase.replication.TestSerialReplicationFailover |
   |   | hadoop.hbase.client.TestAsyncRegionLocator |
   |   | hadoop.hbase.master.procedure.TestCreateNamespaceProcedure |
   |   | hadoop.hbase.client.TestSeparateClientZKCluster |
   |   | hadoop.hbase.client.TestAdmin |
   |   | hadoop.hbase.client.TestCloneSnapshotFromClientError |
   |   | hadoop.hbase.TestSequenceIdMonotonicallyIncreasing |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.regionserver.TestMultiLogThreshold |
   |   | 
hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalCopyToRemote |
   |   | 
hadoop.hbase.master.assignment.TestRegionAssignedToMultipleRegionServers |
   |   | hadoop.hbase.replication.TestPerTableCFReplication |
   |   | hadoop.hbase.coprocessor.TestOpenTableInCoprocessor |
   |   | hadoop.hbase.client.TestAsyncTableScanner |
   |   | hadoop.hbase.replication.TestSyncReplicationRemoveRemoteWAL |
   |   | hadoop.hbase.TestFullLogReconstruction |
   |   | hadoop.hbase.replication.TestSyncReplicationStandbyKillRS |
   |   | hadoop.hbase.client.TestAppendFromClientSide |
   |   | hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory |
   |   | hadoop.hbase.TestMetaTableAccessor |
   |   | hadoop.hbase.security.access.TestAccessController2 |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.regionserver.wal.TestAsyncFSWAL |
   |   | hadoop.hbase.snapshot.TestFlushSnapshotFromClient |
   |   | hadoop.hbase.regionserver.wal.TestAsyncWALReplayCompressed |
   |   | hadoop.hbase.master.TestClusterRestartFailover |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleAsyncWAL
 |
   |   | hadoop.hbase.TestMovedRegionsCleaner |
   |   | hadoop.hbase.master.TestMasterFailover |
   |   | hadoop.hbase.client.TestAsyncReplicationAdminApi |
   |   | hadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated |
   |   | hadoop.hbase.TestAcidGuaranteesWithBasicPolicy |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientGetCompactionState |
   |   | hadoop.hbase.client.TestAsyncClusterAdminApi |
   |   | hadoop.hbase.replication.TestNamespaceReplication |
   |   |

[GitHub] [hbase] Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods from HBaseTestingUtility

2019-08-12 Thread GitBox

Apache-HBase commented on issue #477: HBASE-22834 Removed deprecated methods 
from HBaseTestingUtility
URL: https://github.com/apache/hbase/pull/477#issuecomment-520684605
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | 0 | reexec | 38 | Docker mode activated. |
   ||| _ Prechecks _ |
   | +1 | dupname | 0 | No case conflicting files found. |
   | +1 | hbaseanti | 0 |  Patch does not have any anti-patterns. |
   | +1 | @author | 0 | The patch does not contain any @author tags. |
   | +1 | test4tests | 0 | The patch appears to include 46 new or modified test 
files. |
   ||| _ master Compile Tests _ |
   | +1 | mvninstall | 390 | master passed |
   | +1 | compile | 56 | master passed |
   | +1 | checkstyle | 83 | master passed |
   | +1 | shadedjars | 282 | branch has no errors when building our shaded 
downstream artifacts. |
   | +1 | javadoc | 38 | master passed |
   | 0 | spotbugs | 256 | Used deprecated FindBugs config; considering 
switching to SpotBugs. |
   | +1 | findbugs | 254 | master passed |
   ||| _ Patch Compile Tests _ |
   | +1 | mvninstall | 325 | the patch passed |
   | +1 | compile | 57 | the patch passed |
   | +1 | javac | 57 | the patch passed |
   | +1 | checkstyle | 82 | the patch passed |
   | +1 | whitespace | 0 | The patch has no whitespace issues. |
   | +1 | shadedjars | 293 | patch has no errors when building our shaded 
downstream artifacts. |
   | +1 | hadoopcheck | 961 | Patch does not cause any errors with Hadoop 2.8.5 
2.9.2 or 3.1.2. |
   | +1 | javadoc | 36 | the patch passed |
   | +1 | findbugs | 266 | the patch passed |
   ||| _ Other Tests _ |
   | -1 | unit | 21141 | hbase-server in the patch failed. |
   | -1 | asflicense | 20 | The patch generated 46 ASF License warnings. |
   | | | 24714 | |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | 
hadoop.hbase.replication.TestReplicationStatusSourceStartedTargetStoppedNewOp |
   |   | hadoop.hbase.replication.TestReplicationDisableInactivePeer |
   |   | hadoop.hbase.fs.TestBlockReorderMultiBlocks |
   |   | 
hadoop.hbase.replication.regionserver.TestRegionReplicaReplicationEndpoint |
   |   | hadoop.hbase.regionserver.TestRegionFavoredNodes |
   |   | hadoop.hbase.replication.TestReplicationWithTags |
   |   | hadoop.hbase.master.TestMirroringTableStateManager |
   |   | hadoop.hbase.client.TestAsyncSingleRequestRpcRetryingCaller |
   |   | hadoop.hbase.client.TestMobRestoreSnapshotFromClientSchemaChange |
   |   | hadoop.hbase.client.TestAsyncTableAdminApi |
   |   | hadoop.hbase.master.TestMetaShutdownHandler |
   |   | hadoop.hbase.master.assignment.TestTransitRegionStateProcedure |
   |   | hadoop.hbase.quotas.TestFileArchiverNotifierImpl |
   |   | hadoop.hbase.master.TestMetaAssignmentWithStopMaster |
   |   | hadoop.hbase.TestMultiVersions |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsOpWithDifferentUsersNoACL |
   |   | hadoop.hbase.master.procedure.TestCreateTableProcedure |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithACL |
   |   | hadoop.hbase.procedure.TestProcedureManager |
   |   | hadoop.hbase.master.procedure.TestModifyTableProcedure |
   |   | hadoop.hbase.backup.TestHFileArchiving |
   |   | hadoop.hbase.regionserver.TestCompactorMemLeak |
   |   | hadoop.hbase.security.access.TestTablePermissions |
   |   | hadoop.hbase.coprocessor.TestPostIncrementAndAppendBeforeWAL |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithSLGStack |
   |   | hadoop.hbase.fs.TestBlockReorderBlockLocation |
   |   | hadoop.hbase.client.TestRestoreSnapshotFromClientAfterSplittingRegions 
|
   |   | hadoop.hbase.coprocessor.TestCoprocessorStop |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsReplication |
   |   | hadoop.hbase.client.TestAsyncTableRegionReplicasGet |
   |   | hadoop.hbase.client.TestAsyncTableGetMultiThreaded |
   |   | hadoop.hbase.quotas.TestSpaceQuotasWithSnapshots |
   |   | hadoop.hbase.regionserver.TestRegionReplicas |
   |   | hadoop.hbase.client.TestInvalidMutationDurabilityException |
   |   | hadoop.hbase.client.TestScannerTimeout |
   |   | hadoop.hbase.security.access.TestCellACLWithMultipleVersions |
   |   | hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster |
   |   | 
hadoop.hbase.replication.multiwal.TestReplicationKillMasterRSCompressedWithMultipleWAL
 |
   |   | hadoop.hbase.coprocessor.TestPassCustomCellViaRegionObserver |
   |   | hadoop.hbase.client.TestAsyncTableScannerCloseWhileSuspending |
   |   | 
hadoop.hbase.security.visibility.TestVisibilityLabelsWithDefaultVisLabelService 
|
   |   | hadoop.hbase.client.TestRawAsyncScanCursor |
   |   | hadoop.hbase.mob.compactions.TestMobCompactor |
   |   | hadoop.hbase.security.visibility.TestVisibilityLabelsWithDeletes |
   |   | hadoop.hbase.regionserver.TestHdfsSnapshotHRegion |
   |   |

49 matches

Mail list logo