Skip to content

Emit correctly endianized locklists by default#5795

Merged
markhannum merged 1 commit intobloomberg:mainfrom
markhannum:default_endianize_locklist
Mar 9, 2026
Merged

Emit correctly endianized locklists by default#5795
markhannum merged 1 commit intobloomberg:mainfrom
markhannum:default_endianize_locklist

Conversation

@markhannum
Copy link
Contributor

This has been fixed for a year- this PR enables endianized locklist by default.

Signed-off-by: Mark Hannum <mhannum72@gmail.com>
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.

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
sc_resume_logicalsc_generated
nogensc
consumer_non_atomic_default_consumer_generated
sc_transactional_rowlocks_generated
truncatesc_offline_generated
sc_truncate_lockorder_generated

@SChakravorti21
Copy link
Contributor

On by default:

int gbl_endianize_locklist = 1;

@markhannum markhannum merged commit b82e1dc into bloomberg:main Mar 9, 2026
4 checks passed
@markhannum markhannum deleted the default_endianize_locklist branch March 9, 2026 19:33
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.

3 participants