forked from nsqio/TrendrrNSQClient
-
Notifications
You must be signed in to change notification settings - Fork 57
Open
Description
It looks like this code from NSQConsumer.java (around line 202) will sometimes throw a ConcurrentModificationException:
for (final ServerAddress server : Sets.difference(oldAddresses, newAddresses)) {
LogManager.getLogger(this).info("Remove connection " + server.toString());
connections.get(server).close();
connections.remove(server);
}
Since the exception is uncaught, the scheduler.scheduleAtFixedRate then silently fails, which is bad.
Metadata
Metadata
Assignees
Labels
No labels