Updated pool._logStats to throw an error if the pool is not valid

Author: cjbj

lib/pool.js

@@ -146,10 +146,10 @@ function getConnection(getConnectionCb) {
   // (which is a C layer getter) an error will be thrown.
   if (!self._isValid) {
     if (getConnectionCb && typeof getConnectionCb === 'function') {
-      getConnectionCb(new Error('NJS-002: invalid pool'));
+      getConnectionCb(new Error(nodbUtil.getErrorMessage('NJS-002')));
       return;
     } else {
-      throw new Error('NJS-002: invalid pool');
+      throw new Error(nodbUtil.getErrorMessage('NJS-002'));
     }
   }
 
@@ -236,8 +236,7 @@ function logStats() {
   var averageTimeInQueue;
 
   if (!self._isValid) {
-    console.log('_logStats must be called prior to terminating pool.');
-    return;
+    throw new Error(nodbUtil.getErrorMessage('NJS-002'));
   }
 
   if (self._enableStats !== true) {

lib/util.js

@@ -22,6 +22,7 @@ var util = require('util');
 // errorMessages is a temporary duplication of error messages defined in the C
 // layer that will be removed once a function to fetch from the C layer is added.
 var errorMessages = {
+  'NJS-002': 'NJS-002: invalid pool',
   'NJS-005': 'NJS-005: invalid value for parameter %d',
   'NJS-006': 'NJS-006: invalid type for parameter %d',
   'NJS-009': 'NJS-009: invalid number of parameters',