[ https://issues.apache.org/jira/browse/LUCENE-3501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13124997#comment-13124997 ]
Doron Cohen commented on LUCENE-3501: ------------------------------------- Fixed in trunk: r1181760 Shai's comment on catching AssertionError made me search for other cases of catching this error in Lucene. Few such cases exist, and they all seem wrong, as they call fail when failing fail :) due to assert not enabled but fail to detect that failure since they then silently ignore AssertionError thrown by fail(). Opened LUCENE-3506 for this. > random sampler is not random (and so facet SamplingWrapperTest occasionally > fails) > ---------------------------------------------------------------------------------- > > Key: LUCENE-3501 > URL: https://issues.apache.org/jira/browse/LUCENE-3501 > Project: Lucene - Java > Issue Type: Bug > Components: modules/facet > Reporter: Doron Cohen > Assignee: Doron Cohen > Priority: Minor > Attachments: LUCENE-3501.patch > > > RandomSample is not random at all: > It does not even import java.util.Random, and its behavior is deterministic. > in addition, the test testCountUsingSamping() never retries as it was > supposed to (for taking care of the hoped-for randomness). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org