gf2121 commented on pull request #541: URL: https://github.com/apache/lucene/pull/541#issuecomment-1006376840
Thanks @jpountz ! I have two points to conifrm: 1. Do you mean only **specialize** a few numbers of bits per value (other bpvs using `decodeslow`), or just **support** these numbers when writing (round up bpvs like what we did in DirectWriter) ? 2. This issue is having the same problem as https://github.com/apache/lucene/pull/545 that constants with complex names making codes harder to read, maybe we need to solve it first? I made some change in that approach, hoping it will make sense to you :) -- 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
