[BUG] [CCI] BaseConnectionPool empty methods

[timursaurus]

What is the bug?

BaseConnectionPool methods markAlive and markDead are not implemented, but are called and passed connection argument, example:

// BaseConnectionPool.js
if (connectionById) {
  ...
  this.markAlive(connectionById);
  ...

They just return instance of the class. See:

// BaseConnectionPool.js
markAlive() { // <-- takes no arguments
  return this;
}
markDead() {
  return this;
}

Should be

markAlive(connection) {
  connection.status = Connection.statuses.ALIVE;
  return this;
 }

Two classes, ConnectionPool and CloudConnectionPool, are extensions of the BaseConnectionPool class. While ConnectionPool implements the markDead and markAlive methods within the class, but CloudConnectionPool does not. This means that when using CloudConnectionPool, calling these methods has no effect.