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>

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

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

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

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

The display now shows the top N labels, and shows common labels
separately. Each label is a link that filters the results based on
that label.

Also fixes a typo and removes a harmless double Close.

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

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

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

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

The database format allows multiple lines to be stored in a single
record, to reduce the number of rows in the database. This change
allows reading such records, though there is not yet any code which
can write them. This change also adds a test that will benefit from
this record compression.

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

I copied the code from various dependencies in go-moremath
into a single 'internal/stats' package. That package is at the top
level of the repo because I expect to pull much of benchcmp
into an importable package.

For golang/go#14304.

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

Show a meaningful error to the user if their query is invalid or
matches zero results.

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

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

The app can a parse query and fetch relevant results. It can run on
both App Engine and locally.

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

- uploadid -> upload
- fileid -> upload-part
- uploadtime -> upload-time
- uploadfile -> upload-file

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

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