Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8148936

Adapt UUID.toString() to Compact Strings

    XMLWordPrintable

Details

    • Enhancement
    • Status: Closed
    • P3
    • Resolution: Fixed
    • 9
    • 9
    • core-libs
    • b105
    • Verified

    Description

      JDK-8006627 did the JavaLangAccess hack to improve UUID.toString() performance:

          public String toString() {
              char[] chars = new char[36];
              jla.formatUnsignedLong(mostSigBits >> 32, 4, chars, 0, 8);
              chars[8] = '-';
              jla.formatUnsignedLong(mostSigBits >> 16, 4, chars, 9, 4);
              chars[13] = '-';
              jla.formatUnsignedLong(mostSigBits, 4, chars, 14, 4);
              chars[18] = '-';
              jla.formatUnsignedLong(leastSigBits >> 48, 4, chars, 19, 4);
              chars[23] = '-';
              jla.formatUnsignedLong(leastSigBits, 4, chars, 24, 12);
              return jla.newStringUnsafe(chars);
          }

      This is a good performance improvement, but it clashes with Compact Strings which now have to re-compress the resulting char[] array into byte[]. And we know that UUID would always produce Latin1 String.

      If we follow the same route, and drill more holes in JavaLangAccess to accept Strings with a preset coder, we will also have to take care of the case when Compact Strings are turned off. It might turn beneficial to wait for JDK-8148604 to land, and revert toString() back to plain Java concatenation, letting advanced concat strategies to take care of optimal concat, even in the presence of unsigned long conversions.

      Attachments

        Issue Links

          Activity

            People

              shade Aleksey Shipilev
              shade Aleksey Shipilev
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: