-
Notifications
You must be signed in to change notification settings - Fork 5
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
phase out ZKJobRegistry #244
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.
Not a fan of the naming of openeo_driver.jobregistry.JobRegistryInterface.set_results_metadata
; it's ultimately called by JobTracker upon completion of a job.
Mirrors ZK implementation introduced in Open-EO/openeo-geopyspark-driver@79340e9. Open-EO/openeo-geopyspark-driver#632
…-driver PR To be reverted! Open-EO/openeo-geopyspark-driver#632
TODO:
|
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.
Revert stuff temporarily introduced to get the entire thing to build.
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.
apart from some small notes, this PR mainly adds new APIs, leaving original logic basically alone, so looks fine to me
Open-EO/openeo-geopyspark-driver#632 Co-authored-by: Stefaan Lippens <soxofaan@users.noreply.github.com>
Implements part of Open-EO/openeo-geopyspark-driver#632.