Skip to content

Conversation

@xingbowang
Copy link
Contributor

Summary:

When UDT is enabled, key validation in stress test needs to strip timestamp away.

Test Plan:

Stress test

Reviewers:

Subscribers:

Tasks:

Tags:

Summary:

When UDT is enabled, key validation in stress test needs to strip
timestamp away.

Test Plan:

Stress test

Reviewers:

Subscribers:

Tasks:

Tags:
@meta-cla meta-cla bot added the CLA Signed label Jan 13, 2026
@meta-codesync
Copy link

meta-codesync bot commented Jan 13, 2026

@xingbowang has imported this pull request. If you are a Meta employee, you can view this in D90621816.

@xingbowang xingbowang marked this pull request as ready for review January 13, 2026 21:25
Copy link
Contributor

@hx235 hx235 left a comment

Choose a reason for hiding this comment

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

I thought key() has its timestamp stripped off usually already

return Slice(ukey_and_ts.data(), ukey_and_ts.size() - timestamp_size_);

@xingbowang
Copy link
Contributor Author

I thought key() has its timestamp stripped off usually already

return Slice(ukey_and_ts.data(), ukey_and_ts.size() - timestamp_size_);

Thanks for spotting the bug. The fix is wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants