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

Issue 3969 - Rename ingest job status store to tracker #3975

Draft
wants to merge 28 commits into
base: develop
Choose a base branch
from
Draft

Issue 3969 - Rename ingest job status store to tracker #3975

wants to merge 28 commits into from

Conversation

patchwork01
Copy link
Collaborator

Make sure you have checked all steps below.

Issue

Tests

  • My PR adds the following tests OR does not need testing for this extremely good reason:
    • Covered by existing tests

Documentation

  • In case of new functionality, my PR adds documentation that describes how to use it, or I have linked to a
    separate issue for that below.
  • If I have added or removed any dependencies from the project, I have updated the NOTICES file.

@patchwork01 patchwork01 added the pr-stacked-top A stacked pull request that we don't want to merge into its target until the target PR is merged label Dec 18, 2024
@patchwork01 patchwork01 force-pushed the 3969-rename-ingest-job-tracker branch from 9211361 to ab7c552 Compare December 18, 2024 14:49
@patchwork01
Move events to core module except IngestJobValidatedEvent
aaefe36
@patchwork01
Move method only used in tests to test
d1284cd
@patchwork01
Move IngestJobValidatedEvent to core module
4e940a4
@patchwork01
Remove IngestJob from IngestJobAcceptedStatus
42776f9
@patchwork01
Remove IngestJob from IngestJobRejectedStatus
8ebbfc3
@patchwork01
Remove IngestJob from IngestJobStartedStatus
7c56409
@patchwork01
Move ingest job status updates to core module
8193468
@patchwork01
Move ingest job tracker to core module
6a18853
@patchwork01
Create IngestJobStatusFromJobTestData
be1a4b8
@patchwork01
Rename test helper
d4ffd1b
@patchwork01
Move ingestAcceptedStatus to IngestJobStatusFromJobTestData
5c14c25
@patchwork01
Move ingestStartedStatus to IngestJobStatusFromJobTestData
b71aec1
@patchwork01
From IngestJob from ingestStartedStatus helper
7d2e268
@patchwork01
Add validatedIngestStartedStatus to IngestJobStatusFromJobTestData
7f3f94d
@patchwork01
Remove IngestJob from ingestFinishedStatus
8f6d452
@patchwork01
Remove IngestJob from ingestFinishedStatusUncommitted
0991ee8
@patchwork01
Fix Javadoc
2f6b1f3
@patchwork01
Remove trailing spaces in Javadoc
5b3c18b
Base automatically changed from 3968-rename-compaction-task-tracker to develop December 19, 2024 08:43
@patchwork01 patchwork01 removed the pr-stacked-top A stacked pull request that we don't want to merge into its target until the target PR is merged label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rename ingest job status store to ingest job tracker
1 participant
@patchwork01