[GitHub] [jena] xristy commented on a change in pull request #669: JENA-1837: MaxBasicQueries

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

xristy commented on a change in pull request #669:
URL: https://github.com/apache/jena/pull/669#discussion_r418058361



##########
File path: jena-text/src/main/java/org/apache/jena/query/text/TextIndexLucene.java
##########
@@ -112,6 +112,7 @@
     private final FieldType        ftTextNotStored ; // used for lang derived fields
     private final FieldType        ftTextStoredNoIndex ; // used for lang derived fields
     private final boolean          isMultilingual ;
+    private final int              maxBasicQueries ;

Review comment:
       I think L#136, below, ensures a value for ```maxBasicQueries```. Perhaps L#30 above in ```TextIndexConfig.java``` should be set to 1024 since that's the default in ```org.apache.lucene.queryparser.surround.query. BasicQueryFactory ```




----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org