hadoop-zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hadoop QA (JIRA)" <j...@apache.org>
Subject [jira] Commented: (ZOOKEEPER-524) DBSizeTest is not really testing anything
Date Wed, 03 Feb 2010 21:17:28 GMT

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12829267#action_12829267

Hadoop QA commented on ZOOKEEPER-524:

-1 overall.  Here are the results of testing the latest attachment 
  against trunk revision 903483.

    +1 @author.  The patch does not contain any @author tags.

    -1 tests included.  The patch doesn't appear to include any new or modified tests.
                        Please justify why no tests are needed for this patch.

    -1 patch.  The patch command could not apply the patch.

Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/63/console

This message is automatically generated.

> DBSizeTest is not really testing anything
> -----------------------------------------
>                 Key: ZOOKEEPER-524
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-524
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: server, tests
>            Reporter: Patrick Hunt
>            Assignee: Benjamin Reed
>            Priority: Minor
>             Fix For: 3.3.0
> DBSizeTest looks like it should be testing latency, but it doesn't seem to do it (assert
is commented out).
> We need to decide if this test should be fixed, or just dropped.
> Also note: this test takes 40seconds on my system. Way too long. Perhaps async create
operations should be used
> to populate the database. I also noticed that data size has a big impact on overall test
time (1k vs 5 bytes is something
> like a 2x time diff for time to run the test).

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

View raw message