Skip to content

[*] optimize maintenance tasks in Postgres sink, closes #786 #861

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.

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pashagolub
Copy link
Collaborator

From now admin.config holds information about last successful operation. Maintenance routines will check if any other instance already processed it before starting it's own.

@pashagolub pashagolub self-assigned this Jul 24, 2025
@pashagolub pashagolub added the sinks Where and how to store monitored data label Jul 24, 2025
@coveralls
Copy link

coveralls commented Jul 24, 2025

Pull Request Test Coverage Report for Build 16646463238

Details

  • 2 of 45 (4.44%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.5%) to 63.519%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/sinks/postgres.go 2 45 4.44%
Files with Coverage Reduction New Missed Lines %
internal/sinks/postgres.go 1 37.21%
Totals Coverage Status
Change from base Build 16644125942: -0.5%
Covered Lines: 3188
Relevant Lines: 5019

💛 - Coveralls

@pashagolub pashagolub marked this pull request as draft July 24, 2025 15:54
@pashagolub pashagolub force-pushed the 786_optimize_pgsink_maintenance branch from 6c1aa4f to cca2712 Compare July 30, 2025 16:31
@pashagolub pashagolub force-pushed the 786_optimize_pgsink_maintenance branch from 44cbaaa to 2d73d13 Compare July 31, 2025 10:25
@pashagolub pashagolub force-pushed the 786_optimize_pgsink_maintenance branch from 2d73d13 to d50d5d8 Compare August 19, 2025 15:50
From now `admin.config` holds information about last successful
operation. Maintenance routines will check if any other instance
already processed it before starting it's own.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sinks Where and how to store monitored data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants