[
https://issues.apache.org/jira/browse/LUCENE-997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12563698#action_12563698
]
Paul Elschot commented on LUCENE-997:
-------------------------------------
The idea of System.currentTimeMillis() is to guard against misbehaviour of the
java wait() method and against unexpected delays because of thread scheduling
during the jump back for the loop around the wait() call.
One way to reduce such misbehaviour under heavy load is by increasing the
scheduling priority of the timing thread, but I don't think that is necessary.
Also System.currentTimeMillis() is obviously correct, whereas timeout +=
resolution is never more than an assumption about correct wait() behaviour.
Clock changes by NTP are normally so slow that they don't really matter for
query time outs.
> Add search timeout support to Lucene
> ------------------------------------
>
> Key: LUCENE-997
> URL: https://issues.apache.org/jira/browse/LUCENE-997
> Project: Lucene - Java
> Issue Type: New Feature
> Reporter: Sean Timm
> Priority: Minor
> Attachments: HitCollectorTimeoutDecorator.java,
> LuceneTimeoutTest.java, LuceneTimeoutTest.java, MyHitCollector.java,
> timeout.patch, timeout.patch, timeout.patch, timeout.patch,
> TimerThreadTest.java
>
>
> This patch is based on Nutch-308.
> This patch adds support for a maximum search time limit. After this time is
> exceeded, the search thread is stopped, partial results (if any) are returned
> and the total number of results is estimated.
> This patch tries to minimize the overhead related to time-keeping by using a
> version of safe unsynchronized timer.
> This was also discussed in an e-mail thread.
> http://www.nabble.com/search-timeout-tf3410206.html#a9501029
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
