SortBy seems to sometimes sort by the second column instead of the
first. Until that bug is figured out, do the sorting one column at a
time to control the ordering.

Without this patch, try
/trend?q=upload%3A20170216.3+name%3ARegexpMatchEasy0_1K&raw=1
and
/trend?q=trend%3Abuild.golang.org%2Fwindows-amd64-gce+name%3AGzip+commit-time%3E2016-11-08+commit-time%3C2016-11-30&raw=1

The former seems to sort first by "commit" instead of "commit-time",
but the latter sorts by "commit-time" first. Apply the patch and both
are sorted by commit-time.

Change-Id: I8ebd2720fa1147fc1d468a6bab515a224537a98e
Reviewed-on: https://go-review.googlesource.com/37471
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

Since the upload ID is part of the primary key, we can avoid using the
RecordLabels table to find matching records. This takes the query
"upload:20170127.22 name:Gzip" from ~0.3s to ~0.04s on our production
database.

Change-Id: Id31624ec1b81f3a7f648c6597392718c56a10edd
Reviewed-on: https://go-review.googlesource.com/37236
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I7bd1ed03127830d6530415d77263ee8b0155a2c4
Reviewed-on: https://go-review.googlesource.com/37230
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I0aad9567cc8887922541145868fd927d8c4fb10b
Reviewed-on: https://go-review.googlesource.com/37167
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

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>

Change-Id: I405ff8d89593365b0e27925e7640bee35c723682
Reviewed-on: https://go-review.googlesource.com/37158
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

Queries with multiple parts referring to the same key are now
simplified before sending to the database. If the query can be proven
to return no results, the query is not sent to the database at all.

Change-Id: I2d307d09f463fb0e6e7bd9b9902115916e7ffffa
Reviewed-on: https://go-review.googlesource.com/36953
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

This is still not feature complete, so the page is not linked from
anywhere. It supports graphing benchmark results that match any
perf query. One line is shown per matching benchmark name. Demo
available at:

https://quentin.perf.golang.org/trend?q=upload:20170127.25

Big TODO tasks:
- improve display of multiple benchmarks (multiple graphs?
  click-to-select? additional user controls?)
- correlate benchmarks across uploads (currently all uploads are
  assumed to be from identical machines)
- optimize/cache queries (6 months of a single benchmark loads in 12s;
  6 months of all benchmarks times out at >60s)

Change-Id: I7d1f6073c4837ef63205a10dd4e18085fe7d00ac
Reviewed-on: https://go-review.googlesource.com/36612
Run-TryBot: Quentin Smith <quentin@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

Change-Id: I89a2ad20d0378077fb90cae45fcc8cab33c147ec
Reviewed-on: https://go-review.googlesource.com/36952
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

With this CL, the perf repo should now pass all tests on Windows.

Change-Id: I91fead1e065f22b54236ff96c6ace529dc3964a8
Reviewed-on: https://go-review.googlesource.com/36627
Run-TryBot: Quentin Smith <quentin@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

The query term "key>" can be used to search for records that have any
value for the key. This optimizes the query with a separate SQL
expression and adds tests for this behavior.

Change-Id: I5e2734734e0911a3ed12c87e7ada776ad309a90d
Reviewed-on: https://go-review.googlesource.com/36590Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I268e3731b51b2c7cb327fc47913058490c439d6a
Reviewed-on: https://go-review.googlesource.com/36610
Run-TryBot: Quentin Smith <quentin@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>

- storage/db/dbtest, storage/cmd/reindex: hide cloud SQL behind a build tag
  (run go test -tags cloud golang.org/x/perf/storage/db -cloud to run tests)
- analysis/appengine, storage/appengine: hide behind appengine build tag
  (automatically supplied by goapp)
- vendor: used govendor to import non-x non-appengine dependencies

Change-Id: Ia5590599e7191ab3427618204fab7d2cec907579
Reviewed-on: https://go-review.googlesource.com/36592
Run-TryBot: Quentin Smith <quentin@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I08e93f469558ca3db356cb03eba345bfe29e36f8
Reviewed-on: https://go-review.googlesource.com/36252Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I7f8c384dc9dd47b2706392188685a106a1ed487a
Reviewed-on: https://go-review.googlesource.com/36251Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I8b4b85a2c38deb53bd30a26447aef8c6ce1b49d3
Reviewed-on: https://go-review.googlesource.com/36117Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: Id9eecd9bc5bf7be4ffcdd5a120c7ccc4afe2fb17
Reviewed-on: https://go-review.googlesource.com/36115Reviewed-by: Russ Cox <rsc@golang.org>

In a test database containing 309,000 real benchmark records, given
the query

  select t1.UploadID, (select t2.Value from RecordLabels t2 where
  t2.UploadID = t1.UploadID AND Name = 'upload-time' limit 1) from
  Uploads t1;

used to obtain sample values from each upload, the PRIMARY KEY index
causes the query to fall from 1.49s to 0.03s.

The NOT NULL constraint is necessary to make the PRIMARY KEY effective.

Change-Id: I9e663766e2f99f1096952e4c20eecbf8faab577b
Reviewed-on: https://go-review.googlesource.com/36114Reviewed-by: Russ Cox <rsc@golang.org>

This will eventually be used to serve a "Recent Uploads" section on
the analysis server, and potentially a "My Uploads" section.

Change-Id: I532fc79310d6f98ff12c80a628606845b40ceb60
Reviewed-on: https://go-review.googlesource.com/36012Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I1d0d574a78559f08125d8aa03ea4f6ab7912e28c
Reviewed-on: https://go-review.googlesource.com/36028Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I04949398205be2eaf5e462bdd65282f23419e85d
Reviewed-on: https://go-review.googlesource.com/36027Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I127a31864e96691bca504e81662a2992d5ee9f58
Reviewed-on: https://go-review.googlesource.com/36011Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: Ib710cac2d82ccd37821cd78b33550368c242d266
Reviewed-on: https://go-review.googlesource.com/36026
Run-TryBot: Russ Cox <rsc@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>

Change-Id: I80e47f05530260b09dd92a04ddbf4799731ee99c
Reviewed-on: https://go-review.googlesource.com/35944
Run-TryBot: Russ Cox <rsc@golang.org>
Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: Ia8761c7709e09196dbb2499bcec76ab5bfc0c715
Reviewed-on: https://go-review.googlesource.com/35949Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: I371274fad0b6ef8d79a1c5581917c847c3e94830
Reviewed-on: https://go-review.googlesource.com/35948Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: Idfc4d6b3cbd1e3c17182fe9565149e77ee1edb46
Reviewed-on: https://go-review.googlesource.com/35943Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: I90c73aa77b97b4921e839b376616646604e85a09
Reviewed-on: https://go-review.googlesource.com/35942Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: I74ce7947e800c4db6dd67e96580a1f8a3ce8d52f
Reviewed-on: https://go-review.googlesource.com/35941Reviewed-by: Quentin Smith <quentin@golang.org>

Now that it's one table, move configurations in general case
to top row, not repeated once per metric.

Change-Id: I3c5c53da09b98a6d010ce51cdd7326ce5ebd635e
Reviewed-on: https://go-review.googlesource.com/35940Reviewed-by: Quentin Smith <quentin@golang.org>

- tag old-vs-new tables with tbody class oldnew
- tag better, worse, unchanged rows with classes
- print one big table, to keep alignment across tables
- tag note column and don't right-align

Change-Id: Ifd3b2103e02781904f6b73eeb47b00340484caf6
Reviewed-on: https://go-review.googlesource.com/35939Reviewed-by: Quentin Smith <quentin@golang.org>

The generated html changes in a few minor ways,
either unavoidable or not worth working around:

- Trailing <td></td> at end of row no longer removed.
- <td>~</td> instead of <td>~    </td>
- Literal + escaped to &#43;

Change-Id: I73a0d1f74ff4c3597dd96b1a4dac2d6a3d5cc462
Reviewed-on: https://go-review.googlesource.com/35938Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: I33d4b6006f4df6b253133c4467488fd6da68a34f
Reviewed-on: https://go-review.googlesource.com/35937Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: I4e47c206f12359d60d7f8889fb71514242bfd19b
Reviewed-on: https://go-review.googlesource.com/35936Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: Ied07ec9cec690a1927e9e32fc8d47f36521f658f
Reviewed-on: https://go-review.googlesource.com/35935Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: I20d5f0e2cebef4f577074b45b7849a2698310909
Reviewed-on: https://go-review.googlesource.com/35934Reviewed-by: Quentin Smith <quentin@golang.org>

Change-Id: I185b853a2580ee16741cbf1581ba465400a5b28b
Reviewed-on: https://go-review.googlesource.com/35947Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: Ica888f19254759a060015e9635cffd0231721dd4
Reviewed-on: https://go-review.googlesource.com/35946Reviewed-by: Russ Cox <rsc@golang.org>

Change-Id: Iedf55dbc792d9cc12fc5d2a0a674628eed8ad726
Reviewed-on: https://go-review.googlesource.com/35945Reviewed-by: Russ Cox <rsc@golang.org>

These will work better in the eventual package benchstat API.

Change-Id: I95c63f2fe7c1a0f6db09afbe9776539f264a577c
Reviewed-on: https://go-review.googlesource.com/35933Reviewed-by: Quentin Smith <quentin@golang.org>