Skip to content

Run wal2json cdc test for pg16 as well#7361

Merged
naisila merged 2 commits intomainfrom
naisila/wal2json_pg16
Nov 24, 2023
Merged

Run wal2json cdc test for pg16 as well#7361
naisila merged 2 commits intomainfrom
naisila/wal2json_pg16

Conversation

@naisila
Copy link
Copy Markdown
Contributor

@naisila naisila commented Nov 23, 2023

pg16 wal2json package is now available, adding the tests back. Basically reverting f253bb3

Sister PR citusdata/the-process#153

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 23, 2023

Codecov Report

Merging #7361 (b18e35a) into main (0620c8f) will decrease coverage by 0.67%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7361      +/-   ##
==========================================
- Coverage   89.57%   88.90%   -0.67%     
==========================================
  Files         278      278              
  Lines       59975    59973       -2     
  Branches     7469     7469              
==========================================
- Hits        53721    53321     -400     
- Misses       4106     4443     +337     
- Partials     2148     2209      +61     

@naisila naisila requested a review from rajeshkt78 November 23, 2023 11:47
Copy link
Copy Markdown
Contributor

@rajeshkt78 rajeshkt78 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

naisila added a commit to citusdata/the-process that referenced this pull request Nov 23, 2023
Wal2json PG16 package is now available.

Sister PR citusdata/citus#7361
@naisila naisila force-pushed the naisila/wal2json_pg16 branch from c60b78d to b18e35a Compare November 24, 2023 11:22
@naisila naisila merged commit c019acc into main Nov 24, 2023
@naisila naisila deleted the naisila/wal2json_pg16 branch November 24, 2023 11:40
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.

2 participants