From db051377ef5e4225923e11590d3e75b55998d2f7 Mon Sep 17 00:00:00 2001 From: Steve Yurong Su Date: Thu, 13 Jun 2024 12:50:20 +0800 Subject: [PATCH] Pipe: fix threads of IoTDB-Pipe-Processor-Executor-Pool stucked by PipeTsFileInsertionEvent#waitForTsFileClose (#12727) --- .../common/tsfile/PipeTsFileInsertionEvent.java | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tsfile/PipeTsFileInsertionEvent.java b/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tsfile/PipeTsFileInsertionEvent.java index cdab8d422159..55739bf8f329 100644 --- a/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tsfile/PipeTsFileInsertionEvent.java +++ b/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tsfile/PipeTsFileInsertionEvent.java @@ -135,12 +135,22 @@ public PipeTsFileInsertionEvent( */ public boolean waitForTsFileClose() throws InterruptedException { if (!isClosed.get()) { + isClosed.set(resource.isClosed()); + synchronized (isClosed) { while (!isClosed.get()) { - isClosed.wait(); + isClosed.wait(100); + + final boolean isClosedNow = resource.isClosed(); + if (isClosedNow) { + isClosed.set(true); + isClosed.notifyAll(); + break; + } } } } + // From illustrations above we know If the status is "closed", then the tsFile is flushed // And here we guarantee that the isEmpty() is set before flushing if tsFile is empty // Then we know: "isClosed" --> tsFile flushed --> (isEmpty() <--> tsFile is empty)