gsmiller commented on pull request #227:
URL: https://github.com/apache/lucene/pull/227#issuecomment-888465657


   This is really interesting/exciting!
   
   I'm working through this PR now but I notice you've used a slightly 
different approach to the FOR encoding (compared to what's done in the 
postings). Is this intentional for some reason, or is it more to get something 
out quickly for benchmarking (results were interesting by the way!)? Is there a 
reason you chose not to use the existing `ForUtil` directly?


-- 
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

Reply via email to