Skip to content

Feature/8604 add aggregated download job status - #335

Merged
utas-raymondng merged 4 commits into
mainfrom
feature/8604-add-aggregated-download-job-status
Aug 26, 2026
Merged

Feature/8604 add aggregated download job status#335
utas-raymondng merged 4 commits into
mainfrom
feature/8604-add-aggregated-download-job-status

Conversation

@grojeda

@grojeda grojeda commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@grojeda
grojeda requested a review from utas-raymondng August 26, 2026 01:19
@grojeda
grojeda marked this pull request as ready for review August 26, 2026 01:19

@utas-raymondng utas-raymondng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something to consider whether we can reuse existing ogc model

Comment thread processes/src/main/resources/schema/processes/openapi.yaml Outdated

@Slf4j
@Service
public class DownloadJobStatusService {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar if we can reuse the definition in process model?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This service already reuses the generated OGC StatusInfo and StatusCode models; only the AWS aggregation logic is custom

@utas-raymondng utas-raymondng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please write a small summary in the comment section next time, this can help speed up the PR with some brief understanding of the PR

return ResponseEntity.status(HttpStatus.NOT_IMPLEMENTED).build();
}

@Override

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we will implement this later so that we can query the job status later, this may be useful for help desk to locate all job

@utas-raymondng
utas-raymondng merged commit 588c6a2 into main Aug 26, 2026
4 checks passed
@utas-raymondng
utas-raymondng deleted the feature/8604-add-aggregated-download-job-status branch August 26, 2026 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants