ODE
  1. ODE
  2. ODE-139

Lifecycle management for BpelEventListeners

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0-incubating
    • Fix Version/s: 1.1
    • Component/s: BPEL Runtime
    • Labels:
      None

      Description

      The BPEL Event Listener interface provides a nice way to monitor execution events. In my use case I'm publishing BPEL events via JMS to a JMS topic. However, this requires some initialization and clean up functionality to setup the JMS connection on Ode's startup and to gently close it when Ode is shutting down. The attached patch extends the BpelEventListener interface with startup() and shutdown() methods and calls them in BpelServerImpl in order to give implementers the possibility to implement their own initialization and finalization code.

      1. eventlistener-lifecycle.patch
        2 kB
        Tammo van Lessen
      2. configurable-event-listeners.patch
        6 kB
        Tammo van Lessen

        Activity

        Hide
        Matthieu Riou added a comment -

        Patch applied on r549636. Thanks Tammo!

        Show
        Matthieu Riou added a comment - Patch applied on r549636. Thanks Tammo!
        Hide
        Tammo van Lessen added a comment - - edited

        This patch enables lifecycle management for BpelEventListeners and passes properties defined in Ode's configuration file to the listeners startup() methods. This makes them configurable. The patch integrates with Axis2, JBI and Test IL.

        This patch makes the old patch obsolete.

        Show
        Tammo van Lessen added a comment - - edited This patch enables lifecycle management for BpelEventListeners and passes properties defined in Ode's configuration file to the listeners startup() methods. This makes them configurable. The patch integrates with Axis2, JBI and Test IL. This patch makes the old patch obsolete.
        Hide
        Matthieu Riou added a comment -

        Sounds good. I'll try to apply it correctly this time

        Show
        Matthieu Riou added a comment - Sounds good. I'll try to apply it correctly this time
        Hide
        Tammo van Lessen added a comment - - edited

        When looking into the repository, it seems that the patch has not been applied yet.
        However, it would be also good to pass the OdeConfigProperties to the startup() method so that listeners can share the Ode configuration file. I'll make another patch for that.

        Show
        Tammo van Lessen added a comment - - edited When looking into the repository, it seems that the patch has not been applied yet. However, it would be also good to pass the OdeConfigProperties to the startup() method so that listeners can share the Ode configuration file. I'll make another patch for that.
        Hide
        Matthieu Riou added a comment -

        Patch applied some time ago already.

        Show
        Matthieu Riou added a comment - Patch applied some time ago already.

          People

          • Assignee:
            Matthieu Riou
            Reporter:
            Tammo van Lessen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development