Skip to content

Commit

Permalink
Merge pull request #138 from wri/hotfix/make_integrated_alerts_work
Browse files Browse the repository at this point in the history
Increase integrated alerts job worker count and timeout
  • Loading branch information
jterry64 authored Nov 1, 2023
2 parents 41c481e + 5d2e59d commit 99d3b8a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ RUN pip install . -t python
# to change the hash of the file and get TF to realize it needs to be
# redeployed. Ticket for a better solution:
# https://gfw.atlassian.net/browse/GTC-1250
# change 1
# change 2

RUN yum install -y zip geos-devel

Expand Down
7 changes: 3 additions & 4 deletions src/datapump/jobs/geotrellis.py
Original file line number Diff line number Diff line change
Expand Up @@ -664,11 +664,10 @@ def _calculate_worker_count(self, limiting_src) -> int:
:return: number of workers appropriate for job size
"""
if (
self.sync_type == SyncType.rw_areas
or self.table.analysis == Analysis.integrated_alerts
):
if self.sync_type == SyncType.rw_areas:
return 30
elif self.table.analysis == Analysis.integrated_alerts:
return 60
elif self.change_only and self.table.analysis == Analysis.glad:
return 10

Expand Down
1 change: 1 addition & 0 deletions src/datapump/sync/sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -251,6 +251,7 @@ def build_jobs(self, config: DatapumpConfig) -> List[Job]:
),
features_1x1=config.metadata["features_1x1"],
geotrellis_version=config.metadata["geotrellis_version"],
timeout_sec=6 * 3600,
)
)

Expand Down

0 comments on commit 99d3b8a

Please sign in to comment.