From ea7eec0664d4207bddb125db48fd4e8c5a8fa8c0 Mon Sep 17 00:00:00 2001
From: Stefan Wang <1fannnw@gmail.com>
Date: Thu, 5 Sep 2024 13:04:56 -0700
Subject: [PATCH] add timeout for artifact resolver via ivy settings

---
 .../src/main/resources/ivy.settings.xml       | 34 ++++++++++++++++---
 1 file changed, 30 insertions(+), 4 deletions(-)

diff --git a/coral-hive/src/main/resources/ivy.settings.xml b/coral-hive/src/main/resources/ivy.settings.xml
index 7400f4234..28957c66f 100644
--- a/coral-hive/src/main/resources/ivy.settings.xml
+++ b/coral-hive/src/main/resources/ivy.settings.xml
@@ -6,36 +6,62 @@
 
 -->
 <ivysettings>
+
+  <!-- Define timeouts for resolvers
+    Reference: https://ant.apache.org/ivy/history/2.5.0-rc1/settings/timeout-constraint.html
+
+    The <timeout-constraint> element allows you to define named timeout constraints that can be referenced 
+    in other parts of the Ivy settings file, such as resolvers.
+
+    Attributes:
+    - name: (Required) The name of the timeout constraint.
+    - connectionTimeout: (Optional) Specifies the time, in milliseconds, to wait for a connection to be established.
+      - A value greater than 0 is used literally as the timeout.
+      - A value of 0 indicates no timeout (wait-forever).
+      - A value less than 0 lets the users of this timeout constraint decide what semantics to use.
+    - readTimeout: (Optional) Specifies the time, in milliseconds, to wait when reading content from a resource after a connection is established.
+      - A value greater than 0 is used literally as the timeout.
+      - A value of 0 indicates no timeout (wait-forever).
+      - A value less than 0 lets the users of this timeout constraint decide what semantics to use.
+      
+    For large files, this setting helps avoid premature timeouts during the download process. The `readTimeout` is 
+    applied between receiving data packets, not for the entire download process. If the server continuously sends data, 
+    this timeout won't be triggered.
+  -->
+  <timeout-constraint name="default-timeout" connect-timeout="10000" read-timeout="30000" />
+
   <property name="artifactory.base.url" value="http://artifactory.corp.linkedin.com:8081/artifactory"/>
   <property name="artifactory.failover.url" value="http://artifactory-failover.corp.linkedin.com:8081/artifactory"/>
+
   <caches lockStrategy="artifact-lock" />
   <settings defaultResolver="downloadGrapes" />
+
   <resolvers>
     <chain name="downloadGrapes" returnFirst="true">
 
       <dual name="artifactory-release-resolver">
-        <url name="public" m2compatible="true">
+        <url name="public" m2compatible="true" timeout-constraint="default-timeout">
           <ivy pattern="${artifactory.base.url}/release/[organization]/[module]/[revision]/[module]-[revision](-[classifier]).ivy" />
         </url>
         <ibiblio name="public" m2compatible="true" root="${artifactory.base.url}/release" />
       </dual>
 
       <dual name="artifactory-release-failover-resolver">
-        <url name="public" m2compatible="true">
+        <url name="public" m2compatible="true" timeout-constraint="default-timeout">
           <ivy pattern="${artifactory.failover.url}/release/[organization]/[module]/[revision]/[module]-[revision](-[classifier]).ivy" />
         </url>
         <ibiblio name="public" m2compatible="true" root="${artifactory.failover.url}/release" />
       </dual>
 
       <dual name="artifactory-snapshot-resolver">
-        <url name="public" m2compatible="true">
+        <url name="public" m2compatible="true" timeout-constraint="default-timeout">
           <ivy pattern="${artifactory.base.url}/snapshot/[organization]/[module]/[revision]/[module]-[revision](-[classifier]).ivy" />
         </url>
         <ibiblio name="public" m2compatible="true" root="${artifactory.base.url}/snapshot" />
       </dual>
 
       <dual name="artifactory-snapshot-failover-resolver">
-        <url name="public" m2compatible="true">
+        <url name="public" m2compatible="true" timeout-constraint="default-timeout">
           <ivy pattern="${artifactory.failover.url}/snapshot/[organization]/[module]/[revision]/[module]-[revision](-[classifier]).ivy" />
         </url>
         <ibiblio name="public" m2compatible="true" root="${artifactory.failover.url}/snapshot" />