mrkm4ntr commented on PR #13066: URL: https://github.com/apache/lucene/pull/13066#issuecomment-1947841380
@jpountz Thank you for your review. > I see that we have a rewrite rule that inlines nested disjunctions, so presumably this is more about disjunctions within conjunctions than disjunctions within disjunctions? Ah, yes. That's true. > We have tasks in nightly benchmarks that should be impacted by this change, e.g. http://people.apache.org/~mikemccand/lucenebench/AndHighOrMedMed.html, I'm curious if you tried to run them? I'm not sure how to run these benchmarks. Is there any good instruction? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
