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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to