big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HDDS-1332. Add some logging for flaky test testStartStopDatanodeStateMachine. Contributed by Arpit Agarwal. (#649)

Browse Source
This commit is contained in:
Arpit Agarwal 2019-03-27 08:23:30 -07:00 committed by Nanda kumar
parent b2269581f7
commit b4ed81c4e6
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/statemachine/DatanodeStateMachine.java
View File

@ -397,7 +397,6 @@ public synchronized void stopDaemon() {
@VisibleForTesting @VisibleForTesting
public boolean isDaemonStopped() { public boolean isDaemonStopped() {
return this.executorService.isShutdown() return this.executorService.isShutdown()
&& this.getContext().getExecutionCount() == 0
&& this.getContext().getState() == DatanodeStates.SHUTDOWN; && this.getContext().getState() == DatanodeStates.SHUTDOWN;
} }

9
hadoop-hdds/container-service/src/test/java/org/apache/hadoop/ozone/container/common/TestDatanodeStateMachine.java
View File

@ -16,6 +16,7 @@
*/ */
package org.apache.hadoop.ozone.container.common; package org.apache.hadoop.ozone.container.common;
import com.google.common.base.Supplier;
import com.google.common.collect.Maps; import com.google.common.collect.Maps;
import com.google.common.util.concurrent.ThreadFactoryBuilder; import com.google.common.util.concurrent.ThreadFactoryBuilder;
import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.conf.Configuration;
@ -165,8 +166,12 @@ public void testStartStopDatanodeStateMachine() throws IOException,
stateMachine.startDaemon(); stateMachine.startDaemon();
SCMConnectionManager connectionManager = SCMConnectionManager connectionManager =
stateMachine.getConnectionManager(); stateMachine.getConnectionManager();
GenericTestUtils.waitFor(() -> connectionManager.getValues().size() == 1, GenericTestUtils.waitFor(
1000, 30000); () -> {
LOG.info("connectionManager.getValues().size() is {}",
connectionManager.getValues().size());
return connectionManager.getValues().size() == 1;
}, 1000, 30000);
stateMachine.stopDaemon(); stateMachine.stopDaemon();
assertTrue(stateMachine.isDaemonStopped()); assertTrue(stateMachine.isDaemonStopped());