[jira] [Resolved] (ACCUMULO-4864) PATCH - Use String.replace() instead of String.replaceAll() when replaceAll() does not use a regex

["Christopher Tubbs (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ACCUMULO-4864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christopher Tubbs resolved ACCUMULO-4864.
-----------------------------------------
    Resolution: Duplicate

Accumulo no longer uses JIRA to track new issues. New issues should go to GitHub.

I see you have already created a pull request at https://github.com/apache/accumulo-wikisearch/pull/5  ; That is sufficient, since pull requests on GitHub serve as their own issue number.

> PATCH - Use String.replace() instead of String.replaceAll() when replaceAll() does not use a regex
> --------------------------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-4864
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4864
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: wikisearch
>            Reporter: bd2019us
>            Priority: Trivial
>              Labels: pull-request-available
>         Attachments: ACCUMULO-4864.patch
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Use String.replace() instead of String.replaceAll() when replaceAll() does not use a regex
> Affected files
> # query/src/main/java/org/apache/accumulo/examples/wikisearch/iterator/BooleanLogicIterator.java
> 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



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)