Uploaded image for project: 'JSword'
  1. JSword
  2. JS-158

Appropriate default logging level for performance

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.6.1
    • Component/s: o.c.common.util
    • Labels:
      None
    • Environment:

      Android

      Description

      The default logging level in jsword/src/main/resources/CWLogging.properties is:
      .level=FINEST

      Unfortunately this can cause libraries like HttpComponents to produce a lot of logging and have performance issues like slow downloads. Therefore this default logging level should be set higher and lowered explicitly for certain modules as required. Overriding logging level for each library would be laborious and error prone.

      I have experimented a little with And Bible and discovered that even INFO has a noticeable affect on download time (40 secs for esv) whereas WARNING seems to be a good default (the first level of messages you really need to look at) and also have no noticeable affect on download (<30 secs for esv). The starting point of FINEST resulted in a 4 minute download time for esv.

      So can I suggest the setting is changed to:
      .level=WARNING

        Attachments

          Activity

            People

            • Assignee:
              dmsmith DM Smith
              Reporter:
              mjdenham Martin Denham
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: