Skip to content

backport-2.1: sql: fix two bugs with TRUNCATE#29262

Merged
craig[bot] merged 3 commits intocockroachdb:release-2.1from
vivekmenezes:backport2.1-28721
Aug 29, 2018
Merged

backport-2.1: sql: fix two bugs with TRUNCATE#29262
craig[bot] merged 3 commits intocockroachdb:release-2.1from
vivekmenezes:backport2.1-28721

Conversation

@vivekmenezes
Copy link
Copy Markdown
Contributor

Backport 3/3 commits from #28721.

/cc @cockroachdb/release


sql: correctly resolve schema mutations during TRUNCATE
related to #27687

sql: mark schema change job as successful when table DROP/TRUNCATE
The job is modified in the same transaction as the user's DROP/TRUNCATE transaction.

related to cockroachdb#27687

Release note (sql change): fixes table descriptor corruption when
TRUNCATE TABLE is run while DROP COLUMN is being processed.
The job is modified in the same transaction as the user's
DROP/TRUNCATE transaction.

Release note (sql change): Fixes problem where when a DROP/TRUNCATE is
run while a schema change like CREATE INDEX is running, the corresponding
job would stay running indefinitely.
@vivekmenezes vivekmenezes requested review from a team and knz August 29, 2018 13:56
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@knz knz left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1, 6 of 6 files at r2, 4 of 4 files at r3.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained

@vivekmenezes
Copy link
Copy Markdown
Contributor Author

bors r+

craig bot pushed a commit that referenced this pull request Aug 29, 2018
29262: backport-2.1: sql: fix two bugs with TRUNCATE r=vivekmenezes a=vivekmenezes

Backport 3/3 commits from #28721.

/cc @cockroachdb/release

---

sql: correctly resolve schema mutations during TRUNCATE
related to #27687

sql: mark schema change job as successful when table DROP/TRUNCATE
The job is modified in the same transaction as the user's DROP/TRUNCATE transaction.



Co-authored-by: Vivek Menezes <vivek@cockroachlabs.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 29, 2018

Build succeeded

@craig craig bot merged commit 7253a21 into cockroachdb:release-2.1 Aug 29, 2018
@vivekmenezes vivekmenezes deleted the backport2.1-28721 branch August 29, 2018 19:49
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.

3 participants