Re: svn commit: r467846 - in /xml/xindice/trunk/java/src/org/apache/xindice/core: Collection.java Database.java DatabaseShutdownHandler.java

[Vadim Gritsenko]

Todd Byrne wrote:

In the DatabaseShudownHandler.run method the synchronized block needs to be outside the loop. Its possible for the Map to be modified by a different thread and that will throw a ConcurrentModificationExpection from the iteration.

Yep, it is there:

>     +        synchronized (databases) {
>     +            Iterator dbs = databases.iterator();


Vadim