Re: [PR] Merge related HashMaps in FieldInfos into one map [lucene]

2024-06-06 Thread via GitHub
iverase commented on code in PR #13460: URL: https://github.com/apache/lucene/pull/13460#discussion_r1629276194 ## lucene/core/src/java/org/apache/lucene/index/FieldInfos.java: ## @@ -701,9 +684,7 @@ synchronized Set getFieldNames() { synchronized void clear() { numb

Re: [PR] Merge related HashMaps in FieldInfos into one map [lucene]

2024-06-06 Thread via GitHub
iverase commented on code in PR #13460: URL: https://github.com/apache/lucene/pull/13460#discussion_r1629275099 ## lucene/core/src/java/org/apache/lucene/index/FieldInfos.java: ## @@ -403,11 +385,8 @@ static final class FieldNumbers { FieldNumbers(String softDeletesFieldNam