This is an automated email from the ASF dual-hosted git repository. mmiller pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/accumulo-wikisearch.git
The following commit(s) were added to refs/heads/master by this push: new b520a32 [ACCUMULO-4864] new 3246e20 Merge pull request #5 from bd2019us/ACCUMULO-4864 b520a32 is described below commit b520a3266f705da646a0a3fb614fbbbed89571db Author: bd2019us <> AuthorDate: Sat Apr 20 15:13:56 2019 -0500 [ACCUMULO-4864] A performance overhead is incurred by replaceAll() because it attempts to compile any regex expression. Using replace() instead when a regex is not utilized can save this performance overhead --- .../accumulo/examples/wikisearch/iterator/BooleanLogicIterator.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/query/src/main/java/org/apache/accumulo/examples/wikisearch/iterator/BooleanLogicIterator.java b/query/src/main/java/org/apache/accumulo/examples/wikisearch/iterator/BooleanLogicIterator.java index d9f3c94..1324006 100644 --- a/query/src/main/java/org/apache/accumulo/examples/wikisearch/iterator/BooleanLogicIterator.java +++ b/query/src/main/java/org/apache/accumulo/examples/wikisearch/iterator/BooleanLogicIterator.java @@ -680,7 +680,7 @@ public class BooleanLogicIterator implements SortedKeyValueIterator<Key,Value>, continue; } String fValue = t.getValue().toString(); - fValue = fValue.replaceAll("'", ""); + fValue = fValue.replace("'", ""); boolean negated = t.getOperator().equals("!="); if (!fName.startsWith(FIELD_NAME_PREFIX)) {