This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 730e23922438 [SPARK-55643][INFRA] Add connection timeout to JIRA
client to prevent hanging and enable retries
730e23922438 is described below
commit 730e239224386999683b0537573262ccedef6821
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Mon Feb 23 19:59:38 2026 -0800
[SPARK-55643][INFRA] Add connection timeout to JIRA client to prevent
hanging and enable retries
### What changes were proposed in this pull request?
This PR aims to add a generic connection and read timeout parameter
timeout=(3.05, 30) to the `jira.client.JIRA()` initializations in the
`dev/merge_spark_pr.py` script.
### Why are the changes needed?
By default, the `requests` library (used by the jira python package) can
hang indefinitely when attempting to establish a connection if the server is
unresponsive. Setting a short connection timeout of 3.05 seconds (slightly
larger than a standard 3-second TCP packet retransmission window) ensures that
the client will quickly fail the connection attempt and properly trigger its
internal retry logic, rather than stalling the entire script. The 30-second
read timeout allows sufficient t [...]
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Manual tests.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: `Gemini 3.1 Pro (High)` on `Antigravity`
Closes #54432 from dongjoon-hyun/SPARK-55643.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
dev/merge_spark_pr.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/dev/merge_spark_pr.py b/dev/merge_spark_pr.py
index ff38253bcf53..1eef16aa229f 100755
--- a/dev/merge_spark_pr.py
+++ b/dev/merge_spark_pr.py
@@ -573,7 +573,7 @@ def initialize_jira():
print_error("ERROR finding jira library. Run 'pip3 install jira' to
install.")
continue_maybe("Continue without jira?")
elif JIRA_ACCESS_TOKEN:
- client = jira.client.JIRA(jira_server, token_auth=JIRA_ACCESS_TOKEN)
+ client = jira.client.JIRA(jira_server, token_auth=JIRA_ACCESS_TOKEN,
timeout=(3.05, 30))
try:
# Eagerly check if the token is valid to align with the behavior
of username/password
# authn
@@ -592,7 +592,9 @@ def initialize_jira():
print("You can use JIRA_ACCESS_TOKEN instead of
JIRA_USERNAME/JIRA_PASSWORD.")
print("Visit https://issues.apache.org/jira/secure/ViewProfile.jspa ")
print("and click 'Personal Access Tokens' menu to manage your own
tokens.")
- asf_jira = jira.client.JIRA(jira_server, basic_auth=(JIRA_USERNAME,
JIRA_PASSWORD))
+ asf_jira = jira.client.JIRA(
+ jira_server, basic_auth=(JIRA_USERNAME, JIRA_PASSWORD),
timeout=(3.05, 30)
+ )
else:
print("Neither JIRA_ACCESS_TOKEN nor JIRA_USERNAME/JIRA_PASSWORD are
set.")
continue_maybe("Continue without jira?")
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
