Revert "Add finalizer to Connection (logicalclocks#365)"

This reverts commit 44cedc0.

Author: manu-sj

python/hopsworks_common/connection.py

@@ -21,7 +21,6 @@
 import re
 import sys
 import warnings
-import weakref
 from typing import Any, Optional
 
 from hopsworks_common import client, usage, util, version
@@ -352,7 +351,6 @@ def connect(self) -> None:
         """
         client.stop()
         self._connected = True
-        finalizer = weakref.finalize(self, self.close)
         try:
             # determine engine, needed to init client
             if (self._engine is not None and self._engine.lower() == "spark") or (
@@ -415,7 +413,6 @@ def connect(self) -> None:
             self._provide_project()
         except (TypeError, ConnectionError):
             self._connected = False
-            finalizer.detach()
             raise
 
         self._check_compatibility()
@@ -449,7 +446,7 @@ def close(self) -> None:
         This will clean up any materialized certificates on the local file system of
         external environments such as AWS SageMaker.
 
-        Usage is optional.
+        Usage is recommended but optional.
 
         !!! example
             ```python
@@ -458,9 +455,6 @@ def close(self) -> None:
             conn.close()
             ```
         """
-        if not self._connected:
-            return  # the connection is already closed
-
         from hsfs import engine
 
         if OpenSearchClientSingleton._instance: