glassfish
  1. glassfish
  2. GLASSFISH-19303

[Perf] Glassfish 4.0 secure web requests have performance regression when compared to Glassfish 3.1.2

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0_b61
    • Fix Version/s: 4.0
    • Component/s: security
    • Labels:
      None
    • Environment:

      All OS/platforms

      Description

      When measuring performance for secure web requests (HTTPS), we are seeing performance degradation ranging from -6% to -20% on various platforms.

      Atleast a part of the degradation is coming from following method call in Glassfish 4.0:
      org.glassfish.grizzly.ssl.SSLEncoderTransformer.wrapAll – total time 32.76s
      The equivalent call in Glassfish 3.1.2 takes 40% less time:
      com.sun.grizzly.util.SSLUtils.wrap – total time 19.66s

      The wrapAll method in Glassfish 4.0 calls SSLEngine.wrap in a loop and looks more heavy weight than the wrap method call in Glassfish 3.1.2 .

        Activity

        Hide
        oleksiys added a comment -
        Show
        oleksiys added a comment - grizzly issue http://java.net/jira/browse/GRIZZLY-1387
        Hide
        deep_singh added a comment -

        Tested recent changes with Glassfish b69 nightly (dec 23' 12 build) and found that
        1. 11% improvement compared to Glassfish b64 promoted build
        2. No regression compared to Glassfish 3.1.2 b23

        Show
        deep_singh added a comment - Tested recent changes with Glassfish b69 nightly (dec 23' 12 build) and found that 1. 11% improvement compared to Glassfish b64 promoted build 2. No regression compared to Glassfish 3.1.2 b23
        Hide
        oleksiys added a comment -

        we had to revert the Grizzly 2.3-beta9 integration due to nucleus ql tests failure.
        will let you know when another integration happens.

        thx.

        Show
        oleksiys added a comment - we had to revert the Grizzly 2.3-beta9 integration due to nucleus ql tests failure. will let you know when another integration happens. thx.
        Hide
        oleksiys added a comment -

        fixed

        Show
        oleksiys added a comment - fixed

          People

          • Assignee:
            oleksiys
            Reporter:
            deep_singh
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: