ConcurrentModificationException of org.jeromq.ZMQ.Poller about jeromq HOT 4 CLOSED

Basically, I don't like using ThreadLocal, but it was not easy to make API compatible with jzmq without performance penalty.

The first implementation was to create Selector at the poller constructor. This was safer but some usage and example pattern was

while
    create poller            # creates Selector every time and has severe performance penalty
    register handles
    poll

At that time, there was a document to guide to create a poller outside of a loop,

create poller
register handles
while
    poll

From the 0.2.0, ThreadLocal has been used as both patterns could have same performance based on assumptions.

• Poller should be created and used on the same thread
• There would not be a case which one thread needs to wait on multiple poller.

We should have enhanced the document.

from jeromq.

But this still force me to change my code...

Have you considered to move the Poller.open() from the constructor to the Poller.poll() where the selector is eventually been used in the Poller. In this way, the Poller object shoud not need to hold the selector instance and when call Poller.poll, just get it from the ThreadLocal variable and pass it to zmq.ZMQ.zmq_poll

This seems can fix my issue and have no influence on performance as 0.2.0.

from jeromq.

I would still recommend not sharing the Poller object between threads for a future change.

If you think the change is meaningful to the community, please send a pull request on it.

from jeromq.

It seems 0.3.0-SNAPSHOT have deprecated org.jeromq.ZMQ and org.zeromq.ZMQ.Poller does not holder the threadlocal variable. I will wait 0.3.0 and move to the new api

from jeromq.