dnhatn commented on code in PR #13208:
URL: https://github.com/apache/lucene/pull/13208#discussion_r1537724985
##########
lucene/core/src/java/org/apache/lucene/codecs/perfield/PerFieldMergeState.java:
##########
@@ -31,40 +31,18 @@
import org.apache.lucene.index.MergeState;
import org.apache.lucene.index.Terms;
-/**
- * Utility class to update the {@link MergeState} instance to be restricted to
a set of fields.
- *
- * <p>Warning: the input {@linkplain MergeState} instance will be updated when
calling {@link
- * #apply(Collection)}.
- *
- * <p>It should be called within a {@code try {...} finally {...}} block to
make sure that the
- * mergeState instance is restored to its original state:
- *
- * <pre>
- * PerFieldMergeState pfMergeState = new PerFieldMergeState(mergeState);
- * try {
- * doSomething(pfMergeState.apply(fields));
- * ...
- * } finally {
- * pfMergeState.reset();
- * }
- * </pre>
- */
+/** Utility class to update the {@link MergeState} instance to be restricted
to a set of fields. */
final class PerFieldMergeState {
private final MergeState in;
- private final FieldInfos orgMergeFieldInfos;
- private final FieldInfos[] orgFieldInfos;
- private final FieldsProducer[] orgFieldsProducers;
+ private final MergeState copy;
Review Comment:
++ I pushed
https://github.com/apache/lucene/pull/13208/commits/ee3b0357e07e07cca76662df658d101f927254e4
--
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
