lucenenet icon indicating copy to clipboard operation
lucenenet copied to clipboard

Leave debugStream as null on construction of parser token managers

Open JayOfemi opened this issue 1 year ago • 0 comments

  • [x] You've read the Contributor Guide and Code of Conduct.
  • [x] You've included unit or integration tests for your change, where applicable.
  • [x] You've included inline docs for your change, where applicable.
  • [x] There's an open issue for the PR that you are making. If you'd like to propose a change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars) Leave debugStream as null on construction of parser token managers.

Fixes #{bug number} (in this specific format) Fixes #936

Description

{Detail}

  • When debugStream is set to Console.Out by default, StandardSyntaxParserTokenManager throws an exception when constructing a StandardQueryParser on OSes that do not support System.Console, such as iOS and Android.
  • debugStream can be set later using the setter, if needed.
  • The above is the same for QueryParserTokenManager when constructing a QueryParser.
  • See issue https://github.com/apache/lucenenet/issues/936 for details.

JayOfemi avatar Jul 24 '24 22:07 JayOfemi