Default to seed router on construction

Author: ali-ince

src/v1/internal/connection-providers.js

@@ -72,7 +72,7 @@ export class LoadBalancer extends ConnectionProvider {
     this._hostNameResolver = hostNameResolver;
     this._dnsResolver = new HostNameResolver();
     this._log = log;
-    this._useSeedRouter = false;
+    this._useSeedRouter = true;
   }
 
   acquireConnection(accessMode) {

src/v1/internal/pool.js

@@ -61,7 +61,7 @@ class Pool {
       if (resource) {
         resourceAcquired(key, this._activeResourceCounts);
         if (this._log.isDebugEnabled()) {
-          this._log.debug(`${resource} acquired from the pool`);
+          this._log.debug(`${resource} acquired from the pool ${key}`);
         }
         return resource;
       }
@@ -94,7 +94,7 @@ class Pool {
           }
         }, this._acquisitionTimeout);
 
-        request = new PendingRequest(resolve, reject, timeoutId, this._log);
+        request = new PendingRequest(key, resolve, reject, timeoutId, this._log);
         allRequests[key].push(request);
       });
     });
@@ -261,7 +261,8 @@ function resourceReleased(key, activeResourceCounts) {
 
 class PendingRequest {
 
-  constructor(resolve, reject, timeoutId, log) {
+  constructor(key, resolve, reject, timeoutId, log) {
+    this._key = key;
     this._resolve = resolve;
     this._reject = reject;
     this._timeoutId = timeoutId;
@@ -281,7 +282,7 @@ class PendingRequest {
 
     clearTimeout(this._timeoutId);
     if (this._log.isDebugEnabled()) {
-      this._log.debug(`${resource} acquired from the pool`);
+      this._log.debug(`${resource} acquired from the pool ${this._key}`);
     }
     this._resolve(resource);
   }

test/internal/bolt-stub.js

@@ -112,8 +112,15 @@ class StubServer {
 }
 
 function newDriver(url, config = {}) {
+  // left here for debugging purposes
+  const logging = {
+    level: 'debug',
+    logger: (level, msg) => console.log(`${level}: ${msg}`)
+  };
   // boltstub currently does not support encryption, create driver with encryption turned off
   const newConfig = Object.assign({encrypted: 'ENCRYPTION_OFF'}, config);
+  // use for logging enabled
+  // const newConfig = Object.assign({encrypted: 'ENCRYPTION_OFF', logging}, config);
   return neo4j.driver(url, sharedNeo4j.authToken, newConfig);
 }
 

test/internal/node/routing.driver.boltkit.test.js

@@ -1522,7 +1522,7 @@ describe('routing driver with stub server', () => {
       return;
     }
 
-    const router1 = boltStub.start('./test/resources/boltstub/acquire_endpoints.script', 9010);
+    const router1 = boltStub.start('./test/resources/boltstub/acquire_endpoints_and_exit.script', 9011);
     // start new router on a different port to emulate host name resolution
     // this router uses different script that contains itself as reader
     const router2 = boltStub.start('./test/resources/boltstub/rediscover_using_initial_router.script', 9009);
@@ -1957,13 +1957,13 @@ describe('routing driver with stub server', () => {
 
     boltStub.run(() => {
       const resolverFunction = address => {
-        if (address === '127.0.0.1:9001') {
-          return ['127.0.0.1:9010', '127.0.0.1:9011', '127.0.0.1:9042'];
+        if (address === '127.0.0.1:9000') {
+          return ['127.0.0.1:9010', '127.0.0.1:9001', '127.0.0.1:9042'];
         }
         throw new Error(`Unexpected address ${address}`);
       };
 
-      const driver = boltStub.newDriver('bolt+routing://127.0.0.1:9001', {resolver: resolverFunction});
+      const driver = boltStub.newDriver('bolt+routing://127.0.0.1:9000', { resolver: resolverFunction });
 
       const session = driver.session(READ);
       // run a query that should trigger discovery against 9001 and then read from it