Skip to content

Conversation

@iambriccardo
Copy link
Contributor

@iambriccardo iambriccardo commented Oct 27, 2025

This PR adds two tests for validating column filtering in publications that was already supported but untested.

@iambriccardo iambriccardo changed the title riccardobusetti/etl 352 add support for publication with subset of columns feat(replication): Add support for column filtering in publication Oct 27, 2025
@iambriccardo iambriccardo marked this pull request as ready for review October 27, 2025 14:09
@iambriccardo iambriccardo requested a review from a team as a code owner October 27, 2025 14:09
@coveralls
Copy link

Pull Request Test Coverage Report for Build 18843893090

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 17 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 82.507%

Files with Coverage Reduction New Missed Lines %
etl/src/state/table.rs 1 62.96%
etl/src/workers/table_sync.rs 16 77.72%
Totals Coverage Status
Change from base Build 18842048906: -0.1%
Covered Lines: 15484
Relevant Lines: 18767

💛 - Coveralls

1 similar comment
@coveralls
Copy link

coveralls commented Oct 27, 2025

Pull Request Test Coverage Report for Build 18843893090

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 17 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 82.507%

Files with Coverage Reduction New Missed Lines %
etl/src/state/table.rs 1 62.96%
etl/src/workers/table_sync.rs 16 77.72%
Totals Coverage Status
Change from base Build 18842048906: -0.1%
Covered Lines: 15484
Relevant Lines: 18767

💛 - Coveralls

@iambriccardo iambriccardo merged commit 66175c6 into main Oct 28, 2025
9 checks passed
@iambriccardo iambriccardo deleted the riccardobusetti/etl-352-add-support-for-publication-with-subset-of-columns branch October 28, 2025 10:23
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.

4 participants