java.util.ConcurrentModificationException with cfadmin action="surveillance"

Description

just opening /lucee/admin/web.cfm?action=overview

Environment

None

Attachments

1

Activity

Show:

Zac Spitzer 3 April 2020 at 12:33
Edited

just crashed again

Michael Offner 23 March 2020 at 16:29

Michael Offner 23 March 2020 at 15:58

we use ArrayList internally for Arrays what is as written above not thread safe, so i did some testing with alternatives.

i did the following test with different List implementation

 

 

times to execute it with different implementation

  • test(new ArrayList<String>()); == 2ms

  • test(new CopyOnWriteArrayList<String>()); == 2342ms

  • test(Collections.synchronizedList(new ArrayList<String>())); == 3ms

increase to 10’000’000 and remove CopyOnWriteArrayList

run multiple time at once i get not consistent numbers

synchronized get faster

I will enable synchronized for arrays by default with the option to disable, we will see if we get any negative feedback on this.

 

 

Michael Offner 6 December 2019 at 16:49

Problem is that ArrayImpl is not thread safe.

Fixed

Details

Assignee

Reporter

Priority

Labels

Fix versions

New Issue warning screen

Before you create a new Issue, please post to the mailing list first https://dev.lucee.org

Once the issue has been verified, one of the Lucee team will ask you to file an issue

Sprint

Affects versions

Created 9 September 2019 at 10:14
Updated 7 April 2020 at 10:28
Resolved 24 March 2020 at 08:17