Eclipse Community Forums: NatTable » ConcurrentModification-Exception in ViewportLayer(?)

Home » Eclipse Projects » NatTable » ConcurrentModification-Exception in ViewportLayer(?)(A possible race-condition occurs when handling many events in a short time. )

ConcurrentModification-Exception in ViewportLayer(?) [message #1719472]

Fri, 08 January 2016 04:02

Eclipse User

Hi everyone,
I'm using NatTable to display a rather large and dynamic set of entries.
The update ratio is very variable, varying from 0 entries per minute to several hundreds per second.

My underlying data structure is a combination of GlazedLists (EventList -> ObservableList -> FilterList -> SortedList).

Now sometimes I get this exception:

index.php/fa/24539/0/

Most of the time it happens right at the beginning, but it also happens after serveral minutes.

So far as I can see it happens either if too many events travel up the layer-stack in a rather short amount of time or if 2 or more different events reach the ViewportLayer at the same time (like a PropertyChangeEvent from the ObservableList and a StructuralChangEvent from the EventList). But I'm only guessing at this point.

I build the natTable like this:

index.php/fa/24541/0/

I hope you can help me with this,
regards Dennis

Attachment: stack-trace natTable-exception.JPG
(Size: 199.82KB, Downloaded 459 times)
Attachment: NatTableExamplePart1.JPG
(Size: 149.14KB, Downloaded 624 times)
Attachment: NatTableExamplePart3.JPG
(Size: 105.46KB, Downloaded 513 times)
Attachment: NatTableExamplePart2.JPG
(Size: 123.26KB, Downloaded 634 times)

Re: ConcurrentModification-Exception in ViewportLayer(?) [message #1719544 is a reply to message #1719472]

Fri, 08 January 2016 12:47

Eclipse User

Looking at the code at that position it is iterated over the list of EventHandlers. Are you dynamically adding and removing EventHandlers while processing? Or what version of NatTable are you using?

Since you are using the GlazedLIstsEventLayer, there should be no problem with the event processing, because the GlazedLIstsEventLayer collects the events and only processes one event per 100ms to avoid such heavy overprocessing.

Re: ConcurrentModification-Exception in ViewportLayer(?) [message #1719547 is a reply to message #1719544]

Fri, 08 January 2016 13:30

Eclipse User

Hi Dirk,
I'm using the current version of the NatTable (or so I believe, Build #465). I think I determind the source of the problem. My AutoScrollHandler is listening to RowStructuralChangeEvents. Each time it receives one it determines the last row and triggers moveRowPositionIntoView() of the ViewportLayer. If I understand the source code there correctly this method is registering an EventHandler and 100ms later it unregisters the same one.

When at the same time any other event is handled by the ViewportLayer the ConcurrentModification-Exception occurs.

I'm fresh out of ideas I could try in my own code to prevent this, but i believe if the ViewportLayer would override handleLayerEvent(), registerEventHandler() and the unregisterEventHandler() this could work.
By using a lock object the ViewportLayer could acquire a read-lock before iterating over the eventHandler map and a write-lock before adding or removing a handler. This souldn't decrease the performance by much and it should increase thread-safety for the eventHandlers.

Regards Dennis

[Updated on: Fri, 08 January 2016 13:34] by Moderator

Re: ConcurrentModification-Exception in ViewportLayer(?) [message #1719548 is a reply to message #1719547]

Fri, 08 January 2016 13:50

Eclipse User

Open a ticket and provide a patch if you have a solution for this.

Re: ConcurrentModification-Exception in ViewportLayer(?) [message #1719550 is a reply to message #1719548]

Fri, 08 January 2016 14:02

Eclipse User

BTW, I think it should be fine to to synchronize in the AbstractLayer. It should be quite uncommon to register and unregister event handler while processing events, therefore a read lock should not have too much impact.

Re: ConcurrentModification-Exception in ViewportLayer(?) [message #1719559 is a reply to message #1719548]

Fri, 08 January 2016 15:06

Eclipse User

Reported possible patch in Bug 485459 via Bugzilla. Hope this will work in general and in my case especially Very Happy

Previous Topic:	multi check ComboBox values not gettign saved
Next Topic:	New Features for NatTable 1.4

Goto Forum:

-=] Back to Top [=-

Current Time: Sat Jul 05 00:47:12 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter