jpountz commented on pull request #1541: URL: https://github.com/apache/lucene-solr/pull/1541#issuecomment-635616067
The flags that can be passed to the constructor are about the supported operators. Case insentivity is not an operator so it feels wrong to ask users to configure it this way too? I think it's fine to record this information in the flags _internally_, but maybe we should make the constructor take an additional boolean instead of expecting users to configure case insensitivity via a syntax flag? ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
