You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by mm...@apache.org on 2019/04/22 11:45:40 UTC

[accumulo-wikisearch] branch master updated: [ACCUMULO-4864]

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)) {