Skip to content
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

Fix newsi relinks ordering bug that occurs only on the master #4602

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

Conversation

markhannum
Copy link
Contributor

Pagelogs are placed on filed-queues in a slightly different order when snapshot isolation is run directly on master. This PR fixes the issue, and adds the 'newsi.testopts' file to the newsi_on_master test.

This is the 8.0 version of #4601

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: 2/551 tests failed ⚠.

The first 10 failing tests are:
analyze_exit_immediately
truncatesc_offline_generated

@markhannum markhannum force-pushed the fix_newsi_bug_8 branch 2 times, most recently from 32bf6bc to b18082b Compare August 9, 2024 13:04
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: 1/551 tests failed ⚠.

The first 10 failing tests are:
truncatesc_offline_generated

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: 3/551 tests failed ⚠.

The first 10 failing tests are:
sc_transactional_rowlocks_generated
sc_downgrade
truncatesc_offline_generated

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: 3/551 tests failed ⚠.

The first 10 failing tests are:
phys_rep_tiered
lostwrite
truncatesc_offline_generated

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