Skip to content

roachtest: fix store dump version#26807

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
nvb:nvanbenschoten/fixStoreDump
Jun 18, 2018
Merged

roachtest: fix store dump version#26807
craig[bot] merged 1 commit intocockroachdb:masterfrom
nvb:nvanbenschoten/fixStoreDump

Conversation

@nvb
Copy link
Copy Markdown
Contributor

@nvb nvb commented Jun 18, 2018

This store dump version was accidentally broken in #25541.

Release note: None

This store dump version was accidentally broken in cockroachdb#25541.

Release note: None
@nvb nvb requested review from andreimatei and m-schneider June 18, 2018 20:12
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@andreimatei
Copy link
Copy Markdown
Contributor

:lgtm:


Review status: :shipit: complete! 0 of 0 LGTMs obtained (and 1 stale)


Comments from Reviewable

@nvb
Copy link
Copy Markdown
Contributor Author

nvb commented Jun 18, 2018

bors r+

@m-schneider
Copy link
Copy Markdown
Contributor

:lgtm:


Comments from Reviewable

craig bot pushed a commit that referenced this pull request Jun 18, 2018
26787: rfc: ALTER COLUMN TYPE TableAction plan r=bobvawter a=bobvawter

FYI: I was doing this as a separate document, but since it's motivated by `ALTER_COLUMN_TYPE`, it seems like it belongs in this doc.  The TL;DR is that `TableMutations` is doing two jobs and doesn't have enough specificity to effectively handle multi-step schema changes.

Update the `ALTER COLUMN TYPE` RFC with a proposed plan for migrating state
management from `TableDescriptor.Mutations` into a new `TableActions`
collection.

Release note: None

26807: roachtest: fix store dump version r=nvanbenschoten a=nvanbenschoten

This store dump version was accidentally broken in #25541.

Release note: None

Co-authored-by: Bob Vawter <bob@cockroachlabs.com>
Co-authored-by: Nathan VanBenschoten <nvanbenschoten@gmail.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jun 18, 2018

Build succeeded

@craig craig bot merged commit abd2d0a into cockroachdb:master Jun 18, 2018
@nvb nvb deleted the nvanbenschoten/fixStoreDump branch June 18, 2018 21:08
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.

4 participants