Skip to content

Only initialize WAL shmem structures once in EXEC_BACKEND builds #530

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

Open
wants to merge 1 commit into
base: TDE_REL_17_STABLE
Choose a base branch
from

Conversation

jeltz
Copy link
Collaborator

@jeltz jeltz commented Aug 13, 2025

This only affects EXEC_BACKEND/Windows builds which we currently do not support, but we fix this anyway to make the code more consistent and easier to understand since we try to care about this in other places.

In the future we may want to add CI and proper support for EXEC_BACKEND builds.

The issue was originally found by Zsolt Parragi.

@jeltz jeltz force-pushed the tde/only-init-shmen-once branch 2 times, most recently from f219a47 to 9bc76c4 Compare August 13, 2025 08:06
@codecov-commenter
Copy link

codecov-commenter commented Aug 13, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 82.48%. Comparing base (167aef2) to head (108aac3).

❌ Your project status has failed because the head coverage (82.48%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@                  Coverage Diff                  @@
##           TDE_REL_17_STABLE     #530      +/-   ##
=====================================================
- Coverage              82.50%   82.48%   -0.03%     
=====================================================
  Files                     25       25              
  Lines                   3207     3208       +1     
  Branches                 508      509       +1     
=====================================================
  Hits                    2646     2646              
  Misses                   452      452              
- Partials                 109      110       +1     
Components Coverage Δ
access 84.79% <80.00%> (-0.12%) ⬇️
catalog 87.68% <ø> (ø)
common 77.77% <ø> (ø)
encryption 72.97% <ø> (ø)
keyring 73.21% <ø> (ø)
src 94.15% <ø> (ø)
smgr 96.53% <ø> (ø)
transam ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This only affects EXEC_BACKEND/Windows builds which we currently do not
support, but we fix this anyway to make the code more consistent and
easier to understand since we try to care about this in other places.

In the future we may want to add CI and proper support for EXEC_BACKEND
builds.

The issue was originally found by Zsolt Parragi.
@jeltz jeltz force-pushed the tde/only-init-shmen-once branch from 9bc76c4 to 108aac3 Compare August 14, 2025 21:50
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