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

sql: sensible error when column default dropped in transaction #26781

Merged
merged 1 commit into from
Jun 18, 2018

Conversation

vivekmenezes
Copy link
Contributor

The COMMIT statement used to return
error: got 0 values but expected 1

The column backfiller's updateExprs[] was not being
initialized properly. A NULL default value reflects now in
a proper initialization of the default value to tree.DNull.

related to #26422

Release note: None

The COMMIT statement used to return
error: got 0 values but expected 1

The column backfiller's updateExprs[] was not being
initialized properly. A NULL default value reflects now in
a proper initialization of the default value to tree.DNull.

related to cockroachdb#26422

Release note: None
@vivekmenezes vivekmenezes requested review from bobvawter and a team June 18, 2018 04:04
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@bobvawter
Copy link
Member

:lgtm:


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


Comments from Reviewable

@vivekmenezes
Copy link
Contributor Author

bors r+

craig bot pushed a commit that referenced this pull request Jun 18, 2018
26781: sql: sensible error when column default dropped in transaction r=vivekmenezes a=vivekmenezes

The COMMIT statement used to return
error: got 0 values but expected 1

The column backfiller's updateExprs[] was not being
initialized properly. A NULL default value reflects now in
a proper initialization of the default value to tree.DNull.

related to #26422

Release note: None

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

craig bot commented Jun 18, 2018

Build succeeded

@craig craig bot merged commit 600fe29 into cockroachdb:master Jun 18, 2018
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