[GitHub] [solr-sandbox] markrmiller commented on pull request #43: Allow empty string for config property values.

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

markrmiller commented on PR #43:
URL: https://github.com/apache/solr-sandbox/pull/43#issuecomment-1257066671

   Yeah, I thought about this, and just did t really have a satisfactory answer. Do I just let some config be empty string? Which? Just the one I know needs to allow it so far? Add some others? What’s the rule? In the end it felt better to just consistently allow it. If you want to set it, we take what you said. The same as if you set something to ‘t’ for an address it will still end up failing and showing you set it to empty string. 


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

To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org

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


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