<div dir="ltr">Hi Chris,<br><br>Thanks! In r235586 and r235587.<br><div><br></div><div>With regards the tests, it looks like the Migrations test already runs my new migrations on all the existing DBs, so that's tested. Now that I've added "score" too, I don't need an extra sample type in the example DBs and can just use that, so I've added a test to check that 'score' is correctly settable and gettable.</div><div><br></div><div>Is that sufficient / what you were thinking?</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote">On Thu, 23 Apr 2015 at 01:11 Chris Matthews <<a href="mailto:chris.matthews@apple.com">chris.matthews@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Patch #1 LGTM.  Please also update the test cases in tests/server/db, and perhaps add an instance of a sample type to one of the example DBs so that it gets worked out when the tests are run.<br>
<br>
Patch #2 also LGTM.  I like the idea of a score type, it would be useful in a lot of situations.<br>
<br>
> On Apr 17, 2015, at 6:47 AM, James Molloy <<a href="mailto:james@jamesmolloy.co.uk" target="_blank">james@jamesmolloy.co.uk</a>> wrote:<br>
><br>
> Hi Chris,<br>
><br>
> Two patches are attached. The first introduces the concept of a "bigger is better" sample type. This is useful for benchmarks that emit metrics such as iterations/sec, or some abstract "score". The patch adds a database migration and does the appropriate plumbing.<br>
><br>
> The second patch is a bit more hacky, but it adds a "score" sample type to the default schema so any user can insert at least some samples with "bigger is better" ordering.<br>
><br>
> If you'd prefer not to have this sample type upstream because it hasn't got any use in the official instance, I'm happy to keep this as a downstream patch.<br>
><br>
> Cheers,<br>
><br>
> James<br>
> <0001-LNT-Add-the-ability-for-test-samples-to-be-interpret.patch><0002-LNT-Add-a-score-sample-type.patch><br>
<br>
</blockquote></div>