Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-21636

ReplaceAll() -> replace() for non regex strings

    XMLWordPrintableJSON

    Details

    • Target Version/s:
    • Flags:
      Patch

      Description

      Use String.replace() instead of String.replaceAll() when replaceAll() does not use a regex

      replace() does not need extra compilation / performance overhead when a non-regex string is used. Thus changing replaceAll() to replace() can remove the associated performance overhead.

      Affected files:

      1. accumulo-handler/src/java/org/apache/hadoop/hive/accumulo/predicate/compare/StringCompare.java
      2. hcatalog/core/src/main/java/org/apache/hive/hcatalog/mapreduce/FileOutputCommitterContainer.java
      3. vector-code-gen/src/org/apache/hadoop/hive/tools/GenVectorCode.java

        Attachments

        1. HVIE-21636.patch
          3 kB
          bd2019us

          Issue Links

            Activity

              People

              • Assignee:
                bd2019us bd2019us
                Reporter:
                bd2019us bd2019us
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 20m
                  20m