Qpid
  1. Qpid
  2. QPID-3419

qpid-config --file-size parameter's help text uses incorrect units

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 0.10
    • Fix Version/s: 0.13
    • Component/s: Python Tools
    • Labels:
      None

      Description

      It seems to me, that the qpid-config --file-size parameter's help text uses incorrect units. It shows:

      --file-size=<n> File size in pages (64Kib/page)

      which would indicate that 24-pages (the default) would yield 192KiB journal files (24 * 64 / 8). But instead, the journal files I get are ~1.6MiB, which would suggest that the 64Kib is meant to be 64KiB (ie Kibibytes instead of Kibibits).

        Activity

        Paul Colby created issue -
        Hide
        Paul Colby added a comment -

        I just noticed that C++ broker uses KiB for the equivalent setting (--jfile-size-pgs), so I guess that implies that qpid-config should be using KiB too.

        Show
        Paul Colby added a comment - I just noticed that C++ broker uses KiB for the equivalent setting ( --jfile-size-pgs ), so I guess that implies that qpid-config should be using KiB too.
        Hide
        Paul Colby added a comment -

        Added a very small diff file to make the one (very minor) change.

        Show
        Paul Colby added a comment - Added a very small diff file to make the one (very minor) change.
        Paul Colby made changes -
        Field Original Value New Value
        Attachment QPID-3419.diff [ 12491396 ]
        Hide
        Kim van der Riet added a comment -

        Correct, the async store write page size is 64 kBytes (KiB).

        Show
        Kim van der Riet added a comment - Correct, the async store write page size is 64 kBytes (KiB).
        Ted Ross made changes -
        Assignee Ted Ross [ tedross ]
        Ted Ross made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.13 [ 12316854 ]
        Resolution Fixed [ 1 ]
        Hide
        Paul Colby added a comment -

        Confirmed fixed. Thanks

        Show
        Paul Colby added a comment - Confirmed fixed. Thanks
        Paul Colby made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Robbie Gemmell made changes -
        Component/s Python Tools [ 12312301 ]
        Component/s Tools [ 12312450 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        9d 14h 13m 1 Ted Ross 24/Aug/11 16:22
        Resolved Resolved Closed Closed
        28d 12h 35m 1 Paul Colby 22/Sep/11 04:57

          People

          • Assignee:
            Ted Ross
            Reporter:
            Paul Colby
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development