fix: cleanup of resources when using replicas

Closes: #1045

src/connection/index.ts

@@ -105,16 +105,6 @@ export class Connection extends EventEmitter implements ConnectionContract {
     }
   }
 
-  /**
-   * Cleanup references
-   */
-  private cleanup(): void {
-    this.client = undefined
-    this.readClient = undefined
-    this.readReplicas = []
-    this.roundRobinCounter = 0
-  }
-
   /**
    * Does cleanup by removing knex reference and removing all listeners.
    * For the same of simplicity, we get rid of both read and write
@@ -127,15 +117,17 @@ export class Connection extends EventEmitter implements ConnectionContract {
      */
     this.pool!.on('poolDestroySuccess', () => {
       this.logger.trace({ connection: this.name }, 'pool destroyed, cleaning up resource')
-      this.cleanup()
+      this.client = undefined
       this.emit('disconnect', this)
       this.removeAllListeners()
     })
 
     if (this.readPool !== this.pool) {
       this.readPool!.on('poolDestroySuccess', () => {
         this.logger.trace({ connection: this.name }, 'pool destroyed, cleaning up resource')
-        this.cleanup()
+        this.roundRobinCounter = 0
+        this.readClient = undefined
+        this.readReplicas = []
         this.emit('disconnect', this)
         this.removeAllListeners()
       })