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

HDFS-13799. TestEditLogTailer#testTriggersLogRollsForAllStandbyNN fails due to missing synchronization

Browse Source 
between rollEditsRpcExecutor and tailerThread shutdown. Contributed by Hrishikesh Gadre.
This commit is contained in:
Xiao Chen 2018-08-07 16:11:37 -07:00
parent d838179d8d
commit 0f8cb127cd
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/EditLogTailer.java
View File

@ -234,7 +234,6 @@ public void start() {
} }
public void stop() throws IOException { public void stop() throws IOException {
rollEditsRpcExecutor.shutdown();
tailerThread.setShouldRun(false); tailerThread.setShouldRun(false);
tailerThread.interrupt(); tailerThread.interrupt();
try { try {
@ -242,6 +241,8 @@ public void stop() throws IOException {
} catch (InterruptedException e) { } catch (InterruptedException e) {
LOG.warn("Edit log tailer thread exited with an exception"); LOG.warn("Edit log tailer thread exited with an exception");
throw new IOException(e); throw new IOException(e);
} finally {
rollEditsRpcExecutor.shutdown();
} }
} }