Patrick Hunt updated ZOOKEEPER-516:

    Attachment: ZOOKEEPER-516.patch

updated patch addresses some questions from mahadev (added comments).

setup change was a mistake - that should have been local only change. I removed 
that from the patch.

I made one additional change in this patch - only a single zkServer is started 
for all of the c tests, rather than
starting a new one for each test (setup/teardown). This resulting in another 
~minute saved on my
laptop for overall time to run the tests. this basically means we need to be 
careful in the tests to
prefix paths for each test uniquely. (I updated a few tests for this).

> add support for 10 minute test ie "pre-commit" test
> ---------------------------------------------------
>                 Key: ZOOKEEPER-516
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-516
>             Project: Zookeeper
>          Issue Type: Task
>          Components: tests
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>             Fix For: 3.3.0
>         Attachments: ZOOKEEPER-516.patch, ZOOKEEPER-516.patch
> Our unit tests are now taking > 10 minutes (20 actually). We need to set 
> things up such that we have a < 10 minute "pre commit" build, and a longer 
> running test of tests (unbounded for all intents/purposes). Mainly this 
> requires us to categorize our tests.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to