Uploaded image for project: 'Jackrabbit Content Repository'
  1. Jackrabbit Content Repository
  2. JCR-3360

Performance degradation in concurrent performance tests due to list of open sessions

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: test
    • Labels:
      None

      Description

      julian and myself found an oddity while looking at the concurrent read tests: while running ConcurrentReadAccessControlledTreeTest we experienced that performance dropped during the
      multiple executions of the runTest method.

      this behavior occurred both with admin and anonymous session and irrespective of the ac patches
      applied. the test code showing the problem was:

      protected void runTest() throws Exception

      { Session testSession = loginReader(); // keeps the session in a list for logout in teardown RandomRead randomRead = new RandomRead(testSession, true); randomRead.run(); }

      surprisingly the performance drop was also present if we added testSession.logout right after the
      run method.

      the performance drop disappeared in the following cases:

      protected void runTest() throws Exception

      { Session testSession = getRepository().login(); RandomRead randomRead = new RandomRead(testSession, true); randomRead.run(); testSession.logout(); // this one is optional and didn't have an positive effect }

      the original test code was very similar to the other concurrent performance tests. with those
      figures at hand, i would make sense to fix the performance tests in general... and maybe we should
      also look at Session.logout (but that's definitely a different issue).

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              angela Angela Schreiber
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: