Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update process graph handling in POST/GET/PATCH requests to 1.0 API #34

Closed
soxofaan opened this issue Apr 6, 2020 · 2 comments
Closed

Update process graph handling in POST/GET/PATCH requests to 1.0 API #34

soxofaan opened this issue Apr 6, 2020 · 2 comments

Comments

@soxofaan
Copy link
Member

soxofaan commented Apr 6, 2020

For batch jobs (/jobs), services (/services) and sync. processing (/result) the property process_graph got replaced by process. It contains a process graph and optionally all process metadata. #260

also see Open-EO/openeo-api#260

methods and paths that have to be addressed:

  • POST /result request
  • PUT /process_graphs/{process_graph_id} request
  • POST /services request
  • PATCH /services/{service_id} request
  • GET /services/{service_id} response
  • POST /jobs request
  • PATCH /jobs/{job_id} request
  • GET /jobs/{job_id} response

also see python client: Open-EO/openeo-python-client#129
@soxofaan soxofaan mentioned this issue Apr 6, 2020
Closed
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 10, 2020
@soxofaan
EP-3352 Open-EO#34 Update process graph handling for POST /result
5d760ad
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 10, 2020
@soxofaan
fixup! EP-3352 Open-EO#34 Update process graph handling for POST /result
8d17c83
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 10, 2020
@soxofaan
EP-3352 Open-EO#34 Update process graph handling in /services
7ec33c6
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 10, 2020
@soxofaan
EP-3352 Open-EO#34 encapsulate service metadata to simplify API compl…
4b23e74 
…iance
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 10, 2020
@soxofaan
fixup! EP-3352 Open-EO#34 encapsulate service metadata to simplify AP…
3a379e6 
…I compliance
soxofaan added a commit to soxofaan/openeo-geopyspark-driver that referenced this issue Apr 10, 2020
@soxofaan
EP-3352/Open-EO#37 encapsulate service metadata better
c7c06eb 
- start using ServiceMetadata instead of free form dicts for better type hinting and stronger schemas
- add tests for ZooKeeperServiceRegistry
- related to Open-EO/openeo-python-driver#34
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 15, 2020
@soxofaan
EP-3352 Open-EO#34 Update process graph handling for POST /result
d0ba9d8
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 15, 2020
@soxofaan
EP-3352 Open-EO#34 Update process graph handling in /services
571319d
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 15, 2020
@soxofaan
EP-3352 Open-EO#34 encapsulate service metadata to simplify API compl…
6f2d29a 
…iance

Also fix location header of 'POST /services' endpoint (Open-EO/openeo-api#269)
soxofaan added a commit to soxofaan/openeo-geopyspark-driver that referenced this issue Apr 15, 2020
@soxofaan
EP-3352/Open-EO#37 update process graph handling in 1.0 API requests …
d4bb47b 
…and encapsulate service metadata better

- start using ServiceMetadata instead of free form dicts for better type hinting and stronger schemas
- add tests for ZooKeeperServiceRegistry
- related to Open-EO/openeo-python-driver#34
soxofaan added a commit that referenced this issue Apr 15, 2020
@soxofaan
EP-3352 #34 encapsulate service metadata to simplify API compliance
b4b2cf4 
Introduce a `ServiceMetadata` class to wrap service metadata in a
container with stronger schema/typing than free form dict.

Also fix "Location" header in 'POST /services' response (Open-EO/openeo-api#269):
should be service metadata endpoint, not service (base) url.
soxofaan added a commit to Open-EO/openeo-geopyspark-driver that referenced this issue Apr 15, 2020
@soxofaan
EP-3352 #37 update process graph handling in 1.0 API requests and enc…
2805999 
…apsulate service metadata better

- start using ServiceMetadata instead of free form dicts for better type hinting and stronger schemas
- add tests for ZooKeeperServiceRegistry
- related to Open-EO/openeo-python-driver#34
@soxofaan
Copy link
Member Author

Done/no change required:

  • POST /result request: done
  • PUT /process_graphs/{process_graph_id} request: no change required (endpoint not implemented yet)
  • POST /services request: done
  • PATCH /services/{service_id} request: no change required (endpoint not implemented yet)
  • GET /services/{service_id} response: done

Still to do:

  • POST /jobs request
  • PATCH /jobs/{job_id} request
  • GET /jobs/{job_id} response

soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 16, 2020
@soxofaan
EP-3352 Open-EO#34 Open-EO#8 encapsulate batch job metadata to simpli…
a3bdf1e 
…fy API compliance

Introduce a `BatchJobMetadata` class to wrap batch job metadata in a 
container with stronger schema/typing that a free form dict
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 17, 2020
@soxofaan
_version.py: bump to alpha version 0.2.0a1
18f2570 
bump minor level of version to signal substantial refactor
of "backend implementation" handling in openeo_driver (Open-EO#8, Open-EO#34, EP-3352)
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 20, 2020
@soxofaan
EP-3352 Open-EO#34 Open-EO#8 encapsulate batch job metadata to simpli…
88fb583 
…fy API compliance

Introduce a `BatchJobMetadata` class to wrap batch job metadata in a
container with stronger schema/typing that a free form dict
soxofaan added a commit to soxofaan/openeo-python-driver that referenced this issue Apr 20, 2020
@soxofaan
_version.py: bump to alpha version 0.2.0a1
6dec793 
bump minor level of version to signal substantial refactor
of "backend implementation" handling in openeo_driver (Open-EO#8, Open-EO#34, EP-3352)
@soxofaan
Copy link
Member Author

batch jobs are handled now as well

@soxofaan soxofaan mentioned this issue Apr 21, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@soxofaan