thomaswoeckinger edited a comment on pull request #2508:
URL: https://github.com/apache/lucene-solr/pull/2508#issuecomment-856900350


   > If some of those methods are only used in range faceting, perhaps it 
doesn't make sense to support range faceting on an enum field any way? Thus 
it'd be okay to throw as being unsupported.
   
   I checked the call hierarchy and both methods (parseStr, parseAndAddGap) are 
only used by FacetRangeProcessor.
   
   Regarding range faceting: Is it impossible to use enum values?
   
   Because they have a ordinal associated with their value, anyway i will 
change it to throw an appropriate exception.
   
   


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