Fix ConcurrentModification bug in TestNettyAllocator.testMemoryUsage()

danepitkin · Apr 4, 2024 · 7f2a480 · 7f2a480
1 parent 7fafa78
commit 7f2a480
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestNettyAllocator.java b/java/memory/memory-netty/src/test/java/org/apache/arrow/memory/netty/TestNettyAllocator.java
@@ -69,9 +69,12 @@ public void testMemoryUsage() {
  break;
  }
  }
- assertTrue("Log messages are:\n" +
+ synchronized (memoryLogsAppender.list) {
+ assertTrue("Log messages are:\n" +
  memoryLogsAppender.list.stream().map(ILoggingEvent::toString).collect(Collectors.joining("\n")),
  result);
+ }
+
  } finally {
  memoryLogsAppender.stop();
  logger.detachAppender(memoryLogsAppender);