Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

java.util.ConcurrentModificationException: mutation occurred during iteration #325

Closed
weiting-chen opened this issue May 19, 2021 · 3 comments · Fixed by #332
Closed

java.util.ConcurrentModificationException: mutation occurred during iteration #325

weiting-chen opened this issue May 19, 2021 · 3 comments · Fixed by #332
Labels
backport bug Something isn't working

Comments

@weiting-chen
Copy link
Collaborator

Facing one ConcurrentModificationException when running compile on a whole new server for branch-1.1-spark-3.x.
One solution has been merged for master and branch-1.1-spark-3.1.1.
https://github.com/oap-project/native-sql-engine/pull/298/files

This issue may happen in all branches, check if we can backport the pr to other releases, at least for OAP v1.1 support.

Thanks,
Weiting

@weiting-chen weiting-chen added backport bug Something isn't working labels May 19, 2021
@zhouyuan zhouyuan pinned this issue May 19, 2021
@zhouyuan
Copy link
Collaborator

zhouyuan commented May 19, 2021

-Pfull-scala-compiler or using old version scala-maven-plugin could bypass this

diff --git a/native-sql-engine/core/pom.xml b/native-sql-engine/core/pom.xml
index 36341873..b7ed89b1 100644
--- a/native-sql-engine/core/pom.xml
+++ b/native-sql-engine/core/pom.xml
@@ -308,6 +308,7 @@
       <plugin>
         <groupId>net.alchim31.maven</groupId>
         <artifactId>scala-maven-plugin</artifactId>
+        <version>4.4.0</version>
         <configuration>
           <recompileMode>${scala.recompile.mode}</recompileMode>
         </configuration>

@weiting-chen
Copy link
Collaborator Author

I assume this issue only happens in 1st time to compile the code.
After 1st time compilation, the developers can use incremental mode, right?

@zhouyuan
Copy link
Collaborator

the issue should be fixed by 4.5.2 release, so no need to patch the 1.1 branch
https://github.com/davidB/scala-maven-plugin/releases/tag/4.5.2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants