You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sdap.apache.org by GitBox <gi...@apache.org> on 2022/07/14 17:18:11 UTC
[GitHub] [incubator-sdap-nexus] tloubrieu-jpl commented on a diff in pull request #174: CDMS-122 - Limited Zarr Proxy
tloubrieu-jpl commented on code in PR #174:
URL: https://github.com/apache/incubator-sdap-nexus/pull/174#discussion_r921396416
##########
analysis/webservice/algorithms_spark/Matchup.py:
##########
@@ -218,12 +218,17 @@ def calc(self, request, **args):
self.log.debug("Querying for tiles in search domain")
# Get tile ids in box
- tile_ids = [tile.tile_id for tile in
- self._get_tile_service().find_tiles_in_polygon(bounding_polygon, primary_ds_name,
- start_seconds_from_epoch, end_seconds_from_epoch,
- fetch_data=False, fl='id',
- sort=['tile_min_time_dt asc', 'tile_min_lon asc',
- 'tile_min_lat asc'], rows=5000)]
+ if not self._get_tile_service().supports_direct_bounds_to_tile():
+ tile_ids = [tile.tile_id for tile in
+ self._get_tile_service().find_tiles_in_polygon(bounding_polygon, primary_ds_name,
+ start_seconds_from_epoch, end_seconds_from_epoch,
+ fetch_data=False, fl='id',
+ sort=['tile_min_time_dt asc', 'tile_min_lon asc',
+ 'tile_min_lat asc'], rows=5000)]
+ else:
+ bb = bounding_polygon.bounds
+
+ tile_ids = [self._get_tile_service().bounds_to_direct_tile_id(bb[1], bb[0], bb[3], bb[2], start_time, end_time)]
Review Comment:
If I understand well, that is a smart way of still using SDAP framework without need to use the solr index when the collection is stored in ZARR. Great job. So that is how you made the performances better.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@sdap.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org