-
Notifications
You must be signed in to change notification settings - Fork 127
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
[JVM] Add retry logic for JVM #740
Conversation
Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>
1f3e916
to
e39898e
Compare
Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>
e39898e
to
b619bff
Compare
/gcbrun exp -n ascc-jvm-pr-740-1 -m vertex_ai_gemini-1-5 -i -b jvm-small |
/gcbrun exp -n ascc-jvm-pr-740-2 -m vertex_ai_gemini-1-5 -i -b test-to-harness-jvm-small |
/gcbrun exp -n ascc-jvm-pr-740-3 -m vertex_ai_gemini-1-5 -i -b comparison |
@arthurscchan the java experiments didn't seem to work -- can you double check? |
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.
.
/gcbrun exp -n ascc-jvm-pr-740-4 -m vertex_ai_gemini-1-5 -i -b test-to-harness-jvm-small |
/gcbrun exp -n ascc-jvm-pr-740-5 -m vertex_ai_gemini-1-5 -i -b jvm-small |
both java experiments are looking good |
/gcbrun exp -n dgk123 -m vertex_ai_gemini-1-5 -i -b comparison |
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.
Java builds are looking real nice.
This PR add back the retry logic for JVM when the first generated harness is failed to build.