Skip to content

Commit

Permalink
Minimize Automaton used in APMTracer (elastic#113775)
Browse files Browse the repository at this point in the history
This thing was taking up 400k of heap since it wasn't minimized, turns
into ~2k after minimization.
  • Loading branch information
original-brownbear authored Oct 2, 2024
1 parent 35cdce5 commit fca267e
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
import org.apache.lucene.util.automaton.Automata;
import org.apache.lucene.util.automaton.Automaton;
import org.apache.lucene.util.automaton.CharacterRunAutomaton;
import org.apache.lucene.util.automaton.MinimizationOperations;
import org.apache.lucene.util.automaton.Operations;
import org.apache.lucene.util.automaton.RegExp;
import org.elasticsearch.Build;
Expand Down Expand Up @@ -439,7 +440,7 @@ private static CharacterRunAutomaton buildAutomaton(List<String> includePatterns
? includeAutomaton
: Operations.minus(includeAutomaton, excludeAutomaton, Operations.DEFAULT_DETERMINIZE_WORK_LIMIT);

return new CharacterRunAutomaton(finalAutomaton);
return new CharacterRunAutomaton(MinimizationOperations.minimize(finalAutomaton, Operations.DEFAULT_DETERMINIZE_WORK_LIMIT));
}

private static Automaton patternsToAutomaton(List<String> patterns) {
Expand Down

0 comments on commit fca267e

Please sign in to comment.