Skip to content

[TEST ONLY]Toggle udt in stress test #13516

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jowlyzhang
Copy link
Contributor

No description provided.

@facebook-github-bot
Copy link
Contributor

@jowlyzhang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@jowlyzhang jowlyzhang force-pushed the toggle_udt_in_stress_test branch from d578aa5 to c03b04b Compare April 3, 2025 00:59
@facebook-github-bot
Copy link
Contributor

@jowlyzhang has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@jowlyzhang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@jowlyzhang jowlyzhang force-pushed the toggle_udt_in_stress_test branch from c03b04b to cf88cd8 Compare April 3, 2025 21:48
@facebook-github-bot
Copy link
Contributor

@jowlyzhang has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@jowlyzhang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@hx235
Copy link
Contributor

hx235 commented Apr 4, 2025

oh interesting - so we never toggle between db sessions in stress test now?

@jowlyzhang
Copy link
Contributor Author

oh interesting - so we never toggle between db sessions in stress test now?

Yeah, it's either running consistently with timestamp enabled, or not. Not in between, I'm working on adding it, there are some work needed for keeping track of the timestamp format for trace file replay since it could change.

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.

3 participants