fix: SaslServer busy-loop causes process hang after test suite (#3305)

Author: kevinjqliu

tests/catalog/test_hive.py

@@ -226,21 +226,24 @@ def run(self) -> None:
                     client.write(self._response)
                     client.flush()
             except Exception:
-                pass
+                break
 
     @property
     def port(self) -> int | None:
         self._port_bound.wait()
         return self._port
 
     def close(self) -> None:
-        # Close all client connections first
+        try:
+            self._socket.close()
+        except Exception:
+            pass
+        self.join(timeout=5)
         for client in self._clients:
             try:
                 client.close()
             except Exception:
                 pass
-        self._socket.close()
 
 
 @pytest.fixture(scope="session")