Code review comment for lp://staging/~daniel-nichter/drizzle/query-log-plugin

Latest rev 2321 is idempotent: it passes --repeat-test 1, 2, 5, and 10. :-)
It also removes its tmp and .bak files.

Le 2 août 2011 à 20:51, Mark Atwood a écrit :

> Review: Approve
> Failed Jenkins Build Test
>
> http://jenkins.drizzle.org/view/Drizzle-build/job/drizzle-build-repeat-tests-twice/567/console
>
> query_log.file [ fail ]
>
> The tests have to be idempotent and not leave behind a trace. The repeat-tests-twice test helps discover test cases that are not.
>
> Please fix and resubmit.
> --
> https://code.launchpad.net/~daniel-nichter/drizzle/query-log-plugin/+merge/61034
> You are the owner of lp:~daniel-nichter/drizzle/query-log-plugin.