Skip to content

Disable logical logging for modsnap #5159

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

morgando
Copy link
Contributor

Turning on modsnap turns on BDB_ATTR_SNAPISOL, which enables logical logging. Logical logging is not required for modsnap. The changes in this PR only turn on BDB_ATTR_SNAPISOL if the snapshot implementation is not modsnap or serializable is enabled.

Signed-off-by: mdouglas47 <[email protected]>
@morgando morgando marked this pull request as draft May 15, 2025 15:35
Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 83/607 tests failed ⚠.

The first 10 failing tests are:
newsi_temptables [setup core dumped]
pglogs_seqnum [setup core dumped]
siconfig_orig_generated [setup core dumped]
siconfig_orig_after_newsi_generated [setup core dumped]
sc_timepart_logicalsc_generated
sc_timepart
fdb_compat
serialstep
selectv_recom_range
selectv_range

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 204/613 tests failed ⚠.

The first 10 failing tests are:
newsi_temptables [setup core dumped]
pglogs_seqnum [setup core dumped]
cldeadlock
sc_transactional_rowlocks_generated
comdb2sys
analyze
serialstep
selectv_recom_range
selectv_range
occ_selectv

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