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

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

    XMLWordPrintableJSON

Details

    • 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

              bd2019us bd2019us
              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