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