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

cmd/benchmark: don't upload "name" #16816

Merged
merged 1 commit into from
Jun 30, 2017
Merged

cmd/benchmark: don't upload "name" #16816

merged 1 commit into from
Jun 30, 2017

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Jun 30, 2017

This seems to be the actual benchmark name which is parsed out by the
server; setting it manually results in a uniqueness violation.

This seems to be the actual benchmark name which is parsed out by the
server; setting it manually results in a uniqueness violation.
@tamird tamird requested a review from petermattis June 30, 2017 15:24
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@petermattis
Copy link
Collaborator

:lgtm:


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


Comments from Reviewable

@tamird tamird merged commit b3d4c20 into cockroachdb:master Jun 30, 2017
@tamird tamird deleted the bench-remove-name branch June 30, 2017 17:49
m-schneider pushed a commit to m-schneider/cockroach that referenced this pull request Jun 30, 2017
tamird referenced this pull request in golang/perf Jul 1, 2017
This adds an undocumented "raw=1" query parameter to visualize raw
results without any averaging or filtering. Results are colored
according to the upload-part they came from.

Change-Id: I25590ace89dfe55380e28182c38aeb223e2e73b9
Reviewed-on: https://go-review.googlesource.com/37160
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
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