rjgoyln commented on code in PR #11375:
URL: https://github.com/apache/ozone/pull/11375#discussion_r4155897253


##########
.github/workflows/ci.yml:
##########
@@ -43,8 +43,8 @@ env:
   TEST_JAVA_VERSION: 25 # JDK version used by CI build and tests; should match 
the JDK version in apache/ozone-runner image
   # MAVEN_ARGS and MAVEN_OPTS are duplicated in check.yml and 
populate-cache.yml, please keep in sync
   MAVEN_ARGS: --batch-mode --settings ${{ github.workspace 
}}/dev-support/ci/maven-settings.xml
-  # Maven 3.9 downloads dependencies via maven-resolver-transport-http, which 
reads aether.* and ignores maven.wagon.*
-  MAVEN_OPTS: -Dhttp.keepAlive=false 
-Daether.connector.http.retryHandler.count=10
+  # Maven 3.9 downloads dependencies via maven-resolver-transport-http, which 
reads aether.* and ignores both maven.wagon.* and JDK-level http.*
+  MAVEN_OPTS: -Daether.connector.http.connectionMaxTtl=25 
-Daether.connector.http.retryHandler.count=5 
-Daether.connector.http.retryHandler.intervalMax=30000

Review Comment:
   Yes, that's right — connectionMaxTtl isn't aimed at the 429s.
   
   WAGON-545 reports Azure dropping connections idle for more than four 
minutes; since connectionMaxTtl controls total connection lifetime rather than 
idle time, 25s keeps pooled connections well below that limit.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to