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

Omit default BatchReadConfig in Spi2davexRepositoryServiceFactory

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.0
    • 2.1
    • jackrabbit-spi2dav
    • None

    Description

      i'd like to remove the default batchread configuration created in Spi2davexRepositoryServiceFactory (ll 79) and instead pass
      null if the service configuration doesn't define the batch-read-config.

      for test execution e.g. the given default isn't really optimal as sessions only have a short life time and only read
      a very limited amount of items (in general)... always reading with depth 4 doesn't add any benefit in this case.
      running the level1 jcr tests in jcr2dav (that as far as i saw doesn't define an extra batchread-config took 1.5, 2.5 and 13 minutes
      from null-config -> depth2 -> depth4.

      if there is a strong reason for keeping that default in the factory we should at least change that for the tests.
      michael, what do you think?

      Attachments

        Activity

          People

            angela Angela Schreiber
            angela Angela Schreiber
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: