Address gitar-bot review: let ValueError propagate from clean_host_port

The broad 'except Exception' in model_post_init() was swallowing the

intentional ValueError that clean_host_port raises for invalid ports

(e.g. 'http://localhost:abc'), leaving a malformed hostPort on the

connection object and turning a clear user error into an obscure

downstream failure.

The local import of clean_host_port is from a sibling module that

always exists, so the try/except was unnecessary defence. Removed it

so pydantic surfaces validation errors where they belong.

Author: RajdeepKushwaha5

ingestion/src/metadata/ingestion/models/custom_pydantic.py

@@ -55,15 +55,12 @@ def model_post_init(self, context: Any, /):
         if "hostPort" in self.__pydantic_fields__:
             raw = getattr(self, "hostPort", None)
             if isinstance(raw, str) and "://" in raw:
-                try:
-                    from metadata.utils.db_utils import clean_host_port
-
-                    object.__setattr__(self, "hostPort", clean_host_port(raw))
-                except Exception:
-                    logger.warning(
-                        "Failed to clean hostPort '%s'; leaving as-is",
-                        raw[:50],
-                    )
+                from metadata.utils.db_utils import clean_host_port
+
+                # Let ValueError propagate: if clean_host_port cannot parse
+                # the input (e.g. non-numeric port), the user must fix their
+                # config rather than silently getting a broken hostPort.
+                object.__setattr__(self, "hostPort", clean_host_port(raw))
 
         try:
             for field in self.__pydantic_fields__: