mayya-sharipova commented on pull request #2186:
URL: https://github.com/apache/lucene-solr/pull/2186#issuecomment-783604414


   @jpountz  Adrien, I also wanted to ask your opinion about the behaviour of 
`writer.updateDocValues(Term term, fieldX)`:
   1.  What should happen if `fieldX` doesn't exist? 
   2. What should happen if `fieldX` doesn't have docValues enabled?
   
   Before in both of these cases `updateDocValues` worked well creating a field 
if it doesn't exist or enabling doc values.
   
   But with current PR as the field schema is fixed, we can:
   1. Allow the 1st case  with the assumption that the created field will only 
has docValues and nothing more.
   2. Throw an error on 2nd case, as docValues are not enabled.
   
   WDYT?


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