On 5/31/07, balasubramanian sudaakeran <[EMAIL PROTECTED]> wrote:
Hi, I found the following piece of logic in DisjunctionMaxScorer.skipTo function which may have a potential issue (marked in code as <<<<ISSUE>>>>).public boolean skipTo(int target) throws IOException { if (firstTime) { if (!more) return false; heapify(); firstTime = false; return true; // more would have been false if no subScorers had any docs <<<<ISSUE>>>> The issue is that we should not have his return statement over here. It should continue on the rest of the logic. Otherwise firsttime the skipping to target will not happen.
You are correct. It's a bug. -Yonik --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
