Merge pull request #302 from smart-on-fhir/mikix/tweaks

A couple tweaks: bump pyarrow and give nicer error ctakes hostname error message
smart-on-fhir · Mar 15, 2024 · f70fd6b · f70fd6b
2 parents 98c750e + 9664d56
commit f70fd6b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/cumulus_etl/cli_utils.py b/cumulus_etl/cli_utils.py
@@ -87,7 +87,9 @@ def is_url_available(url: str, retry: bool = True) -> bool:
         try:
             socket.create_connection((url_parsed.hostname, url_parsed.port))
             return True
-        except ConnectionRefusedError:
+        except socket.gaierror:  # hostname didn't resolve
+            return False
+        except ConnectionRefusedError:  # service is not ready yet
             if i < num_tries - 1:
                 time.sleep(3)
 

diff --git a/pyproject.toml b/pyproject.toml
@@ -20,7 +20,7 @@ dependencies = [
     "label-studio-sdk < 1",
     "oracledb < 3",
     "philter-lite < 1",
-    "pyarrow < 15",
+    "pyarrow < 16",
     "rich < 14",
     "s3fs",
 ]