[GitHub] [lucene-solr] jaisonbi commented on pull request #2213: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted DocValues

[GitBox <gi...@apache.org>]

jaisonbi commented on pull request #2213:
URL: https://github.com/apache/lucene-solr/pull/2213#issuecomment-771296030


   If I understood correctly, the route via PerFieldDocValuesFormat need to change the usage of SortedSetDocValues.
   The idea is adding another constructor for enabling terms dict compression, as below:
   ```
   public SortedSetDocValuesField(String name, BytesRef bytes, boolean compression) {
       super(name, compression ? COMPRESSION_TYPE: TYPE);
       fieldsData = bytes;
   }
   ```
   In COMPRESSION_TYPE, add one new attribute for telling PerFieldDocValuesFormat that terms dict compression is enabled for this field.   Not sure if I've got it right :)
   @msokolov @bruno-roustant 
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org