-
Notifications
You must be signed in to change notification settings - Fork 129
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
add bedrock batch job post process function; enhance remote job status parsing #2955
Conversation
…s parsing Signed-off-by: Yaliang Wu <[email protected]>
0eda174
to
fa30628
Compare
…s parsing (#2955) Signed-off-by: Yaliang Wu <[email protected]> (cherry picked from commit 0d26931)
…s parsing (#2955) Signed-off-by: Yaliang Wu <[email protected]> (cherry picked from commit 0d26931)
…s parsing (#2955) (#2957) Signed-off-by: Yaliang Wu <[email protected]> (cherry picked from commit 0d26931) Co-authored-by: Yaliang Wu <[email protected]>
…s parsing (#2955) (#2956) Signed-off-by: Yaliang Wu <[email protected]> (cherry picked from commit 0d26931) Co-authored-by: Yaliang Wu <[email protected]>
@@ -8,7 +8,6 @@ | |||
public enum MLTaskType { | |||
TRAINING, | |||
PREDICTION, | |||
BATCH_PREDICTION, |
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.
Changing the enum order will introduce bwc issue. We should only add enum object to the very last one
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.
Saw it from a previous wrong order. So this is actually a fix. Ignore previous comment
Description
Fix two issues (find more details on #2954):
Related Issues
Resolves #2954
Check List
--signoff
.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.