Skip to content

[+] define sync metric operations types as constants#781

Merged
pashagolub merged 2 commits intocybertec-postgresql:masterfrom
0xgouda:const-sync-operations
Jun 4, 2025
Merged

[+] define sync metric operations types as constants#781
pashagolub merged 2 commits intocybertec-postgresql:masterfrom
0xgouda:const-sync-operations

Conversation

@0xgouda
Copy link
Copy Markdown
Collaborator

@0xgouda 0xgouda commented Jun 3, 2025

this adds AddOp DeleteOp constant operation types
and invalidOp as unexported testing-only operation type
modifies sinks/rpc_test.go to be at the sinks package not sinks_test

@0xgouda 0xgouda force-pushed the const-sync-operations branch from 2918156 to 83e78de Compare June 3, 2025 20:39
@0xgouda 0xgouda changed the title [*] Define Operation Sync Metric Operations types as constants [*] Define Sync Metric Operations types as constants Jun 3, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 4, 2025

Pull Request Test Coverage Report for Build 15438882202

Details

  • 5 of 12 (41.67%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 60.295%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/reaper/database.go 0 2 0.0%
internal/reaper/reaper.go 0 2 0.0%
internal/sinks/prometheus.go 0 3 0.0%
Totals Coverage Status
Change from base Build 15438684071: 0.0%
Covered Lines: 2653
Relevant Lines: 4400

💛 - Coveralls

@pashagolub pashagolub changed the title [*] Define Sync Metric Operations types as constants [+] define sync metric operations types as constants Jun 4, 2025
@pashagolub pashagolub self-assigned this Jun 4, 2025
@pashagolub pashagolub added enhancement New feature or request sinks Where and how to store monitored data labels Jun 4, 2025
this adds AddOp DeleteOp constant operation types
and invalidOp as unexported testing-only operation
modifies rpc_test to be at the sinks package not sinks_test
@pashagolub pashagolub force-pushed the const-sync-operations branch from 83e78de to 6dd8b5b Compare June 4, 2025 09:31
@pashagolub pashagolub merged commit a33e848 into cybertec-postgresql:master Jun 4, 2025
5 checks passed
@0xgouda 0xgouda deleted the const-sync-operations branch June 4, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request sinks Where and how to store monitored data

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants