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

ALTER TABLE ... SPLIT AT now requires merge queue to be disabled #3586

Closed
benesch opened this issue Aug 27, 2018 · 5 comments
Closed

ALTER TABLE ... SPLIT AT now requires merge queue to be disabled #3586

benesch opened this issue Aug 27, 2018 · 5 comments
Labels
C-product-change P-1 High priority; must be done this release
Milestone

Comments

@benesch
Copy link
Contributor

benesch commented Aug 27, 2018

cockroachdb/cockroach#28961 (which will soon be backported) has the following release note:

Release note (sql change): ALTER TABLE ... SPLIT AT now produces an
error if executed while the merge queue is enabled, as the merge queue
is likely to immediately discard any splits created by the statement.

More details from the issue:

The above has implications for the documentation of ALTER TABLE ... SPLIT AT. Users are now prevented from splitting a table unless they disable the merge queue with SET CLUSTER SETTING kv.range_merge.queue_enabled = false. This is so that users are not confused when their manual splits are immediately merged away by the merge queue. (We're planning to come up with a more user friendly solution to this in v2.2.) They can turn the merge queue back on once they've ensured that their newly split ranges exceed the minimum range size (1MB), guaranteeing that they won't be merged away.

Let me know if you need any more details!

/cc @jseldess

@jseldess jseldess added this to the 2.1 milestone Nov 9, 2018
@rmloveland rmloveland added the P-1 High priority; must be done this release label Jan 16, 2019
@rmloveland
Copy link
Contributor

@awoods187 went with P-1 on this one since (IIRC) range merges will be on by default in 2.2, which means this docs issue will need to be addressed in that timeframe. Do you agree?

@jseldess
Copy link
Contributor

Related, from @bdarnell:

we should update our guidance around manual splits. now you should either A) do nothing and trust in load-based spliting or B) disable the merge queue, split manually, and re-enable the merge queue after load/capacity has ramped up and the manual splits are no longer necessary

@sploiselle
Copy link
Contributor

@awoods187 Who would be the best engineering resource for this?

@bdarnell
Copy link
Contributor

Probably me.

@rmloveland
Copy link
Contributor

Closing since AFAICT this was fixed by @ericharmeling in #5642

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-product-change P-1 High priority; must be done this release
Projects
None yet
Development

No branches or pull requests

5 participants