Github user kinow commented on a diff in the pull request:

    https://github.com/apache/jena/pull/436#discussion_r195658758
  
    --- Diff: 
jena-text/src/main/java/org/apache/jena/query/text/TextIndexLucene.java ---
    @@ -316,6 +326,13 @@ protected Document doc(Entity entity) {
                         if (this.isMultilingual) {
                             // add a field that uses a language-specific 
analyzer via MultilingualAnalyzer
                             doc.add(new Field(e.getKey() + "_" + lang, 
(String) e.getValue(), ftText));
    +                        // add fields for any defined auxiliary indexes
    +                        List<String> auxIndexes = Util.getAuxIndexes(lang);
    +                        if (auxIndexes != null) {
    --- End diff --
    
    Never null I believe. We return an empty list when `lang` is empty. And use 
a `Hashtable` to keep data. But happy to leave it if you prefer to double-check 
it anyway (wonder if we should consider `@Nullable` et `@NotNull` in method 
signatures some day)


---

Reply via email to