malinjawi opened a new pull request, #11553: URL: https://github.com/apache/incubator-gluten/pull/11553
What changes are proposed in this pull request? Fix concurrent modification in TreeMemoryConsumer when one thread adds children while another spills or collects stats. Switch children maps (root + node) to ConcurrentHashMap so iteration during spill/stats is weakly‑consistent and doesn’t throw ConcurrentModificationException. Use putIfAbsent for atomic child registration to avoid check‑then‑act races on duplicate names. Add concurrent unit tests covering: concurrent add + spill concurrent stats collection high‑contention mixed operations These tests fail with HashMap (reproduces the bug) and pass with the fix. Fixes #11509. How was this patch tested? mvn -pl gluten-core -am clean test Was this patch authored or co-authored using generative AI tooling? Generated-by: OpenAI Codex (GPT-5) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
