[GitHub] [cordova-android] breautek commented on issue #924: Crash Report: ConcurrentModificationException

[GitBox <gi...@apache.org>]

breautek commented on issue #924:
URL: https://github.com/apache/cordova-android/issues/924#issuecomment-708395325


   > My initial thought was using Collection.synchronizedMap was sufficient, but the exception still occurs in productions.
   
   Are you saying the PR https://github.com/apache/cordova-android/pull/1073 did not address the issue for you?
   
   > Note the downside of doing this is that your app load may be abit slow as now the plugins different threads need to wait for the threads of another plugin.
   
   Personally I don't see a way around that if unrelated plugin code is causing conflicts. My motto is make the framework work, then find a way to optimise later if necessary. So I think that's okay for now.
   
   > The 2 changesets that fix this problem. Feel free to use as you see fit.
   
   A PR would be appreciated. I was never able to reproduce this issue myself, and the combination of plugins in my production apps doesn't appear to trigger this issue so I'm relying on your observations. If you can prepare a branch with those commits and create a PR, it will make it easy for @EinfachHans to try it on his own apps to see if it resolves the issue for him.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org