Concurrent Modification Exception on Android 10

parse-community / ParseLiveQuery-Android

Parse LiveQuery client for Android.

Other

84 stars 32 forks source link

Concurrent Modification Exception on Android 10 #117

Closed duocnguyen6799 closed 3 years ago

duocnguyen6799 commented 3 years ago

I used ParseLiveQuery SDK on version 1.1.0 and 1.2.2 . My app has been crashed like below. Please help me! Thanks.

mmimeault commented 3 years ago

Hello, could you please provide more information?

Stacktrace? SDK version? How to reproduce?

On my side, we are using the latest version intensively or many devices and we don't have that error.

Thanks

duocnguyen6799 commented 3 years ago

I have just updated the information on this issue.

mmimeault commented 3 years ago

@duocnguyen6799 Issue is fixed now?

mmimeault commented 3 years ago

Does make sense. The mCallbacks list is not thread safe. Just realized I don't have this stack trace because I don't use registerListener nor unregisterListener.

Easy fix could be

to add synchronized,
or use a thread-safe version of the Array list (Collections.synchronizedList or CopyOnWriteArrayList)
or any other better suggestion.