[ https://issues.apache.org/jira/browse/BEAM-5759?focusedWorklogId=155092&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-155092 ]
ASF GitHub Bot logged work on BEAM-5759: ---------------------------------------- Author: ASF GitHub Bot Created on: 16/Oct/18 19:52 Start Date: 16/Oct/18 19:52 Worklog Time Spent: 10m Work Description: jbonofre commented on issue #6702: [BEAM-5759] Ensuring JmsIO checkpoint state is accessed and modified safely URL: https://github.com/apache/beam/pull/6702#issuecomment-430376482 retest this please ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 155092) Time Spent: 0.5h (was: 20m) > ConcurrentModificationException on JmsIO checkpoint finalization > ---------------------------------------------------------------- > > Key: BEAM-5759 > URL: https://issues.apache.org/jira/browse/BEAM-5759 > Project: Beam > Issue Type: Bug > Components: io-java-jms > Affects Versions: 2.8.0 > Reporter: Andrew Fulton > Assignee: Andrew Fulton > Fix For: 2.9.0 > > Time Spent: 0.5h > Remaining Estimate: 0h > > When reading from a JmsIO source, a ConcurrentModificationException can be > thrown when checkpoint finalization occurs under heavy load. > For example: > {{jsonPayload: {}} > {{ exception: "java.util.ConcurrentModificationException}} > {{ at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:903)}} > {{ at java.util.ArrayList$Itr.next(ArrayList.java:853)}} > {{ at > org.apache.beam.sdk.io.jms.JmsCheckpointMark.finalizeCheckpoint(JmsCheckpointMark.java:65)}} > {{ at > com.google.cloud.dataflow.worker.StreamingModeExecutionContext$1.run(StreamingModeExecutionContext.java:379)}} > {{ at > com.google.cloud.dataflow.worker.StreamingDataflowWorker$8.run(StreamingDataflowWorker.java:846)}} > {{ at > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)}} > {{ at > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)}} > {{ at java.lang.Thread.run(Thread.java:745)}} > {{"}} > {{ job: "2018-09-27_08_55_18-6454085774348718625" }} > {{ logger: "com.google.cloud.dataflow.worker.StreamingDataflowWorker" }} > {{ message: "Source checkpoint finalization failed:" }} > {{ thread: "309" }} > {{ work: "<nil>" }} > {{ worker: "test-andrew-092715504-09270855-tkfp-harness-dnmb" }} > > Looking at the JmsCheckpointMark code, it appears that access to the pending > message list is unprotected - thus if a thread calls finalizeCheckpoint while > a separate processing thread adds more messages to the checkpoint mark list > then an exception will be thrown. -- This message was sent by Atlassian JIRA (v7.6.3#76005)