Skip to content

Conversation

@colm-mchugh
Copy link
Contributor

Add tests to check that the GUC file_copy_method behaves as expected when database commands are propagated. It is not relevant for CREATE DATABASE .. WITH STRATEGY because citus only supports wal_log here, but ALTER DATABASE .. SET TABLESPACE can use it to determine the OS-level file management.

@codecov
Copy link

codecov bot commented Dec 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.74%. Comparing base (7157f30) to head (f95786a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8379      +/-   ##
==========================================
- Coverage   88.74%   88.74%   -0.01%     
==========================================
  Files         287      287              
  Lines       63020    63019       -1     
  Branches     7891     7891              
==========================================
- Hits        55925    55924       -1     
  Misses       4778     4778              
  Partials     2317     2317              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@colm-mchugh colm-mchugh changed the base branch from m3hm3t/pg18_beta_confs to main December 16, 2025 09:24
Add tests to check that the GUC `file_copy_method` behaves as expected
when database commands are propagated. It is not relevant for CREATE
DATABASE .. WITH STRATEGY because citus only supports wal_log here,
but ALTER DATABASE .. SET TABLESPACE can use it to determine the
OS-level file management.
@colm-mchugh colm-mchugh merged commit 6d15669 into main Dec 16, 2025
116 checks passed
@colm-mchugh colm-mchugh deleted the colm/pg18-dbguc branch December 16, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants