Geronimo
  1. Geronimo
  2. GERONIMO-6373

Expose HOWL flushPartialBuffer config via HOWLLog - useful under low concurrency

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.1
    • Fix Version/s: None
    • Component/s: transaction manager
    • Security Level: public (Regular issues)
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      HOWL XALogger supports flushPartialBuffers which is not configurable via Geronimo HOWLLog. It is useful in the case of low concurrency.

      It would be great to have this exposed in the HOWLLog constructor so that it be be configured.

        /**
         * Indicates whether LogBufferManager should flush buffers
         * before they are full.
         * 
         * <p>Normally, buffers are flushed to disk only when
         * they become full.  In lightly loaded situations,
         * one or more threads may have to wait until the
         * flushSleepTime expires before the buffer is written.
         * In the worst case, a single thread is using the
         * log, and every put() with sync requested will
         * be delayed flushSleepTime ms before the buffer is
         * written.
         * 
         * <p>Setting flushPartialBuffers true will allow
         * the LogBufferManager to flush buffers to disk
         * any time the channel is not busy.  This improves
         * throughput in single threaded and lightly loaded
         * environments. 
         * 
         * <p>By default, this feature is disabled (false) to
         * provide compatability with earlier versions of
         * this library.
         */
        private boolean flushPartialBuffers = false;

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          6d 16h 49m 1 Guillaume Nodet 24/Jul/12 13:51
          Guillaume Nodet made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Guillaume Nodet added a comment -

          Not sure which version I should put. djencks ?

          Show
          Guillaume Nodet added a comment - Not sure which version I should put. djencks ?
          Guillaume Nodet made changes -
          Assignee David Jencks [ djencks ] Guillaume Nodet [ gnt ]
          Guillaume Nodet made changes -
          Link This issue relates to ARIES-881 [ ARIES-881 ]
          Hide
          Gary Tully added a comment -

          I guess not based on the comment, it is false just for compatibility.

          Show
          Gary Tully added a comment - I guess not based on the comment, it is false just for compatibility.
          David Jencks made changes -
          Assignee David Jencks [ djencks ]
          Hide
          David Jencks added a comment -

          Is there a good reason not to just set this to true unilaterally? I wasn't aware of the flag but why would we ever want it false?

          Show
          David Jencks added a comment - Is there a good reason not to just set this to true unilaterally? I wasn't aware of the flag but why would we ever want it false?
          Gary Tully made changes -
          Comment [ Patch that adds boolean flushPartialBuffers to the constructor ]
          Gary Tully made changes -
          Attachment GERONIMO-6372.patch [ 12536872 ]
          Gary Tully made changes -
          Attachment GERONIMO-6372.patch [ 12536875 ]
          Hide
          Gary Tully added a comment -

          Patch that adds new constructor that takes additional flushPartialBuffers arg.

          Show
          Gary Tully added a comment - Patch that adds new constructor that takes additional flushPartialBuffers arg.
          Gary Tully made changes -
          Field Original Value New Value
          Attachment GERONIMO-6372.patch [ 12536872 ]
          Gary Tully created issue -

            People

            • Assignee:
              Guillaume Nodet
              Reporter:
              Gary Tully
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development