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

Multiregion movr workload performance discrepancies after primary key change #50019

Closed
rohany opened this issue Jun 9, 2020 · 7 comments
Closed
Labels
C-investigation Further steps needed to qualify. C-label will change.

Comments

@rohany
Copy link
Contributor

rohany commented Jun 9, 2020

The demos here https://github.com/cockroachlabs/scripted-demos/tree/movr-20.1 set up the movr workload in 2 ways:

  • Create the tables from the get go in a multi region fashion with partitions and zone configs
  • Start the application in single region mode, and transition the schema to multi region using primary key changes.

There have been reported performance differences between the two -- p90, p99 latencies are 10-20 ms higher in the post primary key change state. We need to investigate what is causing the regression.

@rohany rohany added the C-investigation Further steps needed to qualify. C-label will change. label Jun 9, 2020
@rohany
Copy link
Contributor Author

rohany commented Jun 9, 2020

cc @awoods187

@awoods187
Copy link
Contributor

@rohany I know you mentioned SFU as a possible cause here...not sure why that would be different on one cluster vs. another. cc @nvanbenschoten

What are our next steps?

@nvanbenschoten
Copy link
Member

I'd recommend waiting until the fix for #49658 lands later today and then testing again.

@awoods187
Copy link
Contributor

Looks like everything is merged--can we try it out now?

@nvanbenschoten
Copy link
Member

Let's wait until #50173 is merged and backported (if you're running straight off release-20.1).

@nvanbenschoten
Copy link
Member

That patch is now merged on master. The backport is here: #50216.

@rohany
Copy link
Contributor Author

rohany commented Jun 18, 2020

I reran these demos with these changes and could not replicate the performance differences with the related kv changes.

@rohany rohany closed this as completed Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-investigation Further steps needed to qualify. C-label will change.
Projects
None yet
Development

No branches or pull requests

3 participants