Remove unnecessary __del__ handlers

There normally should be no logic attached to del.  Cleanly disconnecting network resources is not needed at that time.

Author: kristjanvalur

redis/asyncio/connection.py

@@ -181,12 +181,6 @@ def __init__(self, socket_read_size: int):
         self._read_size = socket_read_size
         self._connected = False
 
-    def __del__(self):
-        try:
-            self.on_disconnect()
-        except Exception:
-            pass
-
     @classmethod
     def parse_error(cls, response: str) -> ResponseError:
         """Parse an error response"""
@@ -570,18 +564,6 @@ def repr_pieces(self):
             pieces.append(("client_name", self.client_name))
         return pieces
 
-    def __del__(self):
-        try:
-            if self.is_connected:
-                loop = asyncio.get_running_loop()
-                coro = self.disconnect()
-                if loop.is_running():
-                    loop.create_task(coro)
-                else:
-                    loop.run_until_complete(coro)
-        except Exception:
-            pass
-
     @property
     def is_connected(self):
         return self._reader is not None and self._writer is not None