632 phase out zkjobregistry

Open-EO / openeo-geopyspark-driver

OpenEO driver for GeoPySpark (Geotrellis)

Apache License 2.0

26 stars 4 forks source link

632 phase out zkjobregistry #635

Closed bossie closed 8 months ago

bossie commented 8 months ago

Implements #632.

bossie commented 8 months ago

Also consider related TODOs: https://github.com/Open-EO/openeo-geopyspark-driver/issues/632#issuecomment-1876808925

bossie commented 8 months ago

Leveraged DoubleJobRegistry because it is a convenient way to disable the ZkJobRegistry at runtime (as opposed to @ development time because e.g. CDSE prod and C-SCALE still need it) without littering the code with if/else branches. It was already used in the web app but now JobTracker uses it as well.

bossie commented 8 months ago

TODO:

[x] reference openeo_driver package again once that is merged;
[x] revert quickfix introduced in 4b4fe78650ef31cbeab9d136de85966edef361fd as this PR also addresses it in a more general way;
[x] bump version and adapt CHANGELOG.