uschindler commented on a change in pull request #480:
URL: https://github.com/apache/lucene/pull/480#discussion_r757783926
##########
File path:
lucene/luke/src/java/org/apache/lucene/luke/models/analysis/AnalysisImpl.java
##########
@@ -152,6 +152,14 @@ public Analyzer createAnalyzerFromClassName(String
analyzerType) {
}
}
+ private Analyzer defaultAnalyzer() {
+ try {
+ return CustomAnalyzer.builder().withTokenizer("standard").build();
Review comment:
> Or should we mimic the snapshot of the current StandardAnalyzer by
CustomAnalyzer and set it as the default?
that was my idea.
With hard dependency I meant that upon starting the app the analysis-common
must be part of classpath. If that's the case we are fine.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
