Skip to content

tpccbench: fix existing dataset fast-path#26048

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
nvb:nvanbenschoten/tpccFix
May 24, 2018
Merged

tpccbench: fix existing dataset fast-path#26048
craig[bot] merged 1 commit intocockroachdb:masterfrom
nvb:nvanbenschoten/tpccFix

Conversation

@nvb
Copy link
Copy Markdown
Contributor

@nvb nvb commented May 24, 2018

This skewed with #24735, which changed the output of information_schema
when connecting to an implicit database.

Release note: None

This skewed with cockroachdb#24735, which changed the output of information_schema
when connecting to an implicit database.

Release note: None
@nvb nvb requested a review from a-robinson May 24, 2018 18:24
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@petermattis
Copy link
Copy Markdown
Collaborator

:lgtm:


Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@nvb
Copy link
Copy Markdown
Contributor Author

nvb commented May 24, 2018

TFTR.

bors r+

craig bot pushed a commit that referenced this pull request May 24, 2018
26044: misc: fix encryption stats nits. r=mberhault a=mberhault

Nothing major, followups to premature merge of #25850

Release note: None

26048: tpccbench: fix existing dataset fast-path r=nvanbenschoten a=nvanbenschoten

This skewed with #24735, which changed the output of `information_schema`
when connecting to an implicit database.

Release note: None

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

craig bot commented May 24, 2018

Build succeeded

@craig craig bot merged commit 01cb8d4 into cockroachdb:master May 24, 2018
@nvb nvb deleted the nvanbenschoten/tpccFix branch May 24, 2018 21:38
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