Fix OPENDJ-346 Consider using java.util.ServiceLoader for loading extension...

Nicolas Capponi

08.16.2013 f2090c0d863b07e3bad8d16a3efddfad6ff77960

 opendj3/opendj-ldap-sdk/src/main/java/com/forgerock/opendj/ldap/TimeoutChecker.java

@@ -63,8 +63,8 @@
     * The connection set must be safe from CMEs because expiring requests can
     * cause the connection to be closed.
     */
    private final Set<LDAPConnection> connections =
            newSetFromMap(new ConcurrentHashMap<LDAPConnection, Boolean>());
    private final Set<GrizzlyLDAPConnection> connections =
            newSetFromMap(new ConcurrentHashMap<GrizzlyLDAPConnection, Boolean>());

    /**
     * Used to signal thread shutdown.
@@ -80,7 +80,7 @@
                    final long currentTime = System.currentTimeMillis();
                    long delay = 0;

                    for (final LDAPConnection connection : connections) {
                    for (final GrizzlyLDAPConnection connection : connections) {
                        DEFAULT_LOG.trace("Checking connection {} delay = {}", connection, delay);

                        // May update the connections set.
@@ -113,12 +113,12 @@
        checkerThread.start();
    }

    void addConnection(final LDAPConnection connection) {
    void addConnection(final GrizzlyLDAPConnection connection) {
        connections.add(connection);
        signal();
    }

    void removeConnection(final LDAPConnection connection) {
    void removeConnection(final GrizzlyLDAPConnection connection) {
        connections.remove(connection);
        // No need to signal.
    }