Skip to content

Attach to the DSA only when we actually access the DSA - #810

Merged
jeltz merged 1 commit into
percona:mainfrom
jeltz:clean-attach-dsa
Aug 7, 2026
Merged

Attach to the DSA only when we actually access the DSA#810
jeltz merged 1 commit into
percona:mainfrom
jeltz:clean-attach-dsa

Conversation

@jeltz

@jeltz jeltz commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Clean up some confusion about "attaching" to shared memory by changing the function names and making sure to only call pgsm_attach_dsa() before accessing the query DSA.

PG-0

@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.22%. Comparing base (643897b) to head (b4655c4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #810      +/-   ##
==========================================
+ Coverage   88.07%   88.22%   +0.14%     
==========================================
  Files           3        3              
  Lines        1300     1299       -1     
  Branches      180      180              
==========================================
+ Hits         1145     1146       +1     
+ Misses         76       75       -1     
+ Partials       79       78       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Clean up some confusion about "attaching" to shared memory by changing
the function names and making sure to only call pgsm_attach_dsa() before
accessing the query DSA.
@jeltz
jeltz force-pushed the clean-attach-dsa branch from 455f2c0 to b4655c4 Compare August 3, 2026 19:14
@jeltz jeltz changed the title Attach to the DSA only when we actually access to DSA Attach to the DSA only when we actually access the DSA Aug 3, 2026
@jeltz
jeltz merged commit 7fb076f into percona:main Aug 7, 2026
57 of 61 checks passed
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