apache / lucenenet

Apache Lucene.NET
https://lucenenet.apache.org/
Apache License 2.0
2.24k stars 639 forks source link

System properties specified as environment variables throw unhelpful ArgumentException #827

Open NightOwl888 opened 1 year ago

NightOwl888 commented 1 year ago

System properties are supposed to be prefixed with lucene: when using environment variables. And when you do, it works.

However, if you forget and specify them like you would in a configuration file or NUnit property, incorrectly using something like tests:crashmode instead of lucene:tests:crashmode, it throws the exception:

System.ArgumentException : Value does not fall within the expected range.

Clearly, this message doesn't lead you to the correct solution. This message should be fixed to clearly spell out that the "lucene:" prefix is missing.

farQtech commented 1 year ago

@NightOwl888 I would like to take this up. Please assign it to me.

NightOwl888 commented 1 year ago

@jeme - thanks for assigning this one.

@farQtech - thanks for volunteering.

NightOwl888 commented 9 months ago

@farQtech - Are you still interested in doing this? If not, please let us know so we can assign it to someone else. Thanks.