[ https://issues.apache.org/jira/browse/HBASE-20988?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16576454#comment-16576454 ]
Sean Busbey commented on HBASE-20988: ------------------------------------- I believe this is a duplicate of HBASE-19265 > TestShell shouldn't be skipped for hbase-shell module test > ---------------------------------------------------------- > > Key: HBASE-20988 > URL: https://issues.apache.org/jira/browse/HBASE-20988 > Project: HBase > Issue Type: Test > Reporter: Ted Yu > Priority: Major > > Here is snippet for QA run 13862 for HBASE-20985 : > {code} > 13:42:50 cd /testptch/hbase/hbase-shell > 13:42:50 /usr/share/maven/bin/mvn > -Dmaven.repo.local=/home/jenkins/yetus-m2/hbase-master-patch-1 > -DHBasePatchProcess -PrunAllTests > -Dtest.exclude.pattern=**/master.normalizer. > TestSimpleRegionNormalizerOnCluster.java,**/replication.regionserver.TestSerialReplicationEndpoint.java,**/master.procedure.TestServerCrashProcedure.java,**/master.procedure.TestCreateTableProcedure. > > java,**/TestClientOperationTimeout.java,**/client.TestSnapshotFromClientWithRegionReplicas.java,**/master.TestAssignmentManagerMetrics.java,**/client.TestShell.java,**/client. > > TestCloneSnapshotFromClientWithRegionReplicas.java,**/master.TestDLSFSHLog.java,**/replication.TestReplicationSmallTestsSync.java,**/master.procedure.TestModifyTableProcedure.java,**/regionserver. > > TestCompactionInDeadRegionServer.java,**/client.TestFromClientSide3.java,**/master.procedure.TestRestoreSnapshotProcedure.java,**/client.TestRestoreSnapshotFromClient.java,**/security.access. > > TestCoprocessorWhitelistMasterObserver.java,**/replication.regionserver.TestDrainReplicationQueuesForStandBy.java,**/master.procedure.TestProcedurePriority.java,**/master.locking.TestLockProcedure. > > java,**/master.cleaner.TestSnapshotFromMaster.java,**/master.assignment.TestSplitTableRegionProcedure.java,**/client.TestMobRestoreSnapshotFromClient.java,**/replication.TestReplicationKillSlaveRS. > > java,**/regionserver.TestHRegion.java,**/security.access.TestAccessController.java,**/master.procedure.TestTruncateTableProcedure.java,**/client.TestAsyncReplicationAdminApiWithClusters.java,**/ > > coprocessor.TestMetaTableMetrics.java,**/client.TestMobSnapshotCloneIndependence.java,**/namespace.TestNamespaceAuditor.java,**/master.TestMasterAbortAndRSGotKilled.java,**/client.TestAsyncTable.java,**/master.TestMasterOperationsForRegionReplicas.java,**/util.TestFromClientSide3WoUnsafe.java,**/client.TestSnapshotCloneIndependence.java,**/client.TestAsyncDecommissionAdminApi.java,**/client. > > TestRestoreSnapshotFromClientWithRegionReplicas.java,**/master.assignment.TestMasterAbortWhileMergingTable.java,**/client.TestFromClientSide.java,**/client.TestAdmin1.java,**/client. > > TestFromClientSideWithCoprocessor.java,**/replication.TestReplicationKillSlaveRSWithSeparateOldWALs.java,**/master.procedure.TestMasterFailoverWithProcedures.java,**/regionserver. > TestSplitTransactionOnCluster.java clean test -fae > > /testptch/patchprocess/patch-unit-hbase-shell.txt 2>&1 > {code} > In this case, there was modification to shell script, leading to running > shell tests. > However, TestShell was excluded in the QA run, defeating the purpose. > Meanwhile QA posted the following onto HBASE-20985 : > bq. +1 unit 7m 4s hbase-shell in the patch passed. > That is misleading - no related test was actually run. -- This message was sent by Atlassian JIRA (v7.6.3#76005)