Skip to content

Move backfill operations to separate class#247

Merged
s-vitaliy merged 20 commits intomainfrom
move-out-start-stop-backfill
Mar 5, 2026
Merged

Move backfill operations to separate class#247
s-vitaliy merged 20 commits intomainfrom
move-out-start-stop-backfill

Conversation

@s-vitaliy
Copy link
Copy Markdown
Contributor

@s-vitaliy s-vitaliy commented Mar 5, 2026 •
edited
Loading

Part of non-streaming YAMLs

Scope

Implemented:

  • Functions startBackfill and completeBackfill are removed from the stream reconciler class

Checklist

  • GitHub issue exists for this change.
  • Unit tests added and they pass.
  • Line Coverage is at least 80%.
  • Review requested on latest commit.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 5, 2026 •
edited
Loading

go-test-coverage report:

Package coverage threshold (70%) satisfied:	PASS
Total coverage threshold (75%) satisfied:	PASS
Total test coverage: 85.2% (499/586)

@s-vitaliy s-vitaliy marked this pull request as ready for review March 5, 2026 12:53
@s-vitaliy s-vitaliy requested a review from a team as a code owner March 5, 2026 12:53
@s-vitaliy s-vitaliy merged commit 7f6bc14 into main Mar 5, 2026
1 check passed
@s-vitaliy s-vitaliy deleted the move-out-start-stop-backfill branch March 5, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@george-zubrienko george-zubrienko george-zubrienko approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@s-vitaliy @george-zubrienko