-
Notifications
You must be signed in to change notification settings - Fork 28.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-49523][CONNECT] Increase maximum wait time for connect server to come up for testing #47994
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM we can try
@WweiL mind filing a JIRA please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM
Ok, let's get this one merged. I will file the JIRA and update the description. |
.build()) | ||
.create() | ||
|
||
// Execute an RPC which will get retried until the server is up. | ||
assert(spark.version == SparkBuildInfo.spark_version) | ||
eventually(timeout(1.minute)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically this adds another layer of retries. We could also set the maxBackoff to 1 minute.
Oops... thanks guys! |
…to come up for testing ### What changes were proposed in this pull request? This PR increases the max time we wait for a connect server to come up for testing. The current threshold is too low, and is causing flakyness. ### Why are the changes needed? It makes connect tests less flaky. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? It is test infra code. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#47994 from WweiL/deflake-will-it-work. Authored-by: Wei Liu <wei.liu@databricks.com> Signed-off-by: Herman van Hovell <herman@databricks.com>
…to come up for testing ### What changes were proposed in this pull request? This PR increases the max time we wait for a connect server to come up for testing. The current threshold is too low, and is causing flakyness. ### Why are the changes needed? It makes connect tests less flaky. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? It is test infra code. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#47994 from WweiL/deflake-will-it-work. Authored-by: Wei Liu <wei.liu@databricks.com> Signed-off-by: Herman van Hovell <herman@databricks.com>
What changes were proposed in this pull request?
This PR increases the max time we wait for a connect server to come up for testing. The current threshold is too low, and is causing flakyness.
Why are the changes needed?
It makes connect tests less flaky.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
It is test infra code.
Was this patch authored or co-authored using generative AI tooling?
No.