uschindler commented on PR #13051:
URL: https://github.com/apache/lucene/pull/13051#issuecomment-1915761017

   When backporting this to Lucene 9.x, it confliced because the sets have 
different contents in older version. I fixed this.
   
   Nevertheless, the static final constants should be unmodifiable sets, so we 
should possibly use Java 9+ `Set.of()` instead of `EnumSet.of()`(which is 
modifiable) or wrap the `EnumSet` with `Collections.unmodifiableSet()`.
   
   Could you open an issue about this?


-- 
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: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to