SwitchYard
  1. SwitchYard
  2. SWITCHYARD-673

Consolidate HornetQ and ClientUtil classes used in quickstarts

    Details

    • Type: Task Task
    • Status: Resolved Resolved (View Workflow)
    • Priority: Major Major
    • Resolution: Done
    • Affects Version/s: 0.4
    • Fix Version/s: 0.4
    • Component/s: quickstarts, test
    • Labels:
      None
    • Similar Issues:
      Show 10 results 

      Description

      These classes are starting to pop up over and over again in JMS-related quickstarts. Instead of just copying them over and over again, we should create one version that can be shared amongst all quickstarts. There are two use cases here:

      1) Test cases which need to use JMS and/or a JMS server. These tests should be using the HornetQMixIn alone. Any additional dependencies on the ClientUtil classes need to be rolled into HornetQMixIn.

      2) Most of these quickstarts also offer the option to use "mvn exec:java [options]" to test the application independently with the quickstart deployed to AS7. This case also relies on the ClientUtil classes. We need to change this to use HornetQMixIn (e.g. static methods on the class) or something else packaged in the switchyard-test module. This module should be included in the test classpath and if the exec plugin is configured to include test dependencies in the classpath, all should be good.

      BTW, it would be great if we could build the configuration for the exec plugin into the pom.xml, like I have done in multiApp/demos/consumer-service. This makes it much easier for the user to simply type "mvn exec:java" without having to pass a bunch of -D options.

        Issue Links

          Activity

          Hide
          Keith Babo
          added a comment -

          Since this will touch all quickstarts that use JMS, would you mind updating the instructions to create queues in each README to reflect the new syntax in AS 7.1 ? Another thing we should explore is whether the JMS queues can be deployed independently in a -ds.xml and just copied to the servers deployment directory.

          Show
          Keith Babo
          added a comment - Since this will touch all quickstarts that use JMS, would you mind updating the instructions to create queues in each README to reflect the new syntax in AS 7.1 ? Another thing we should explore is whether the JMS queues can be deployed independently in a -ds.xml and just copied to the servers deployment directory.
          Hide
          Tomohisa igarashi
          added a comment -

          Sure - first one (to use jms-queue add instead of deprecated add-jms-queue) is really easy and the other (deploy by file) needs a bit trying but I guess it's also easy.

          Show
          Tomohisa igarashi
          added a comment - Sure - first one (to use jms-queue add instead of deprecated add-jms-queue) is really easy and the other (deploy by file) needs a bit trying but I guess it's also easy.
          Hide
          Keith Babo
          added a comment -

          On second thought, maybe we should wait until 0.5 to play around with the -ds.xml stuff. I have a feeling that resource dependencies will be required within AS for that to work correctly on startup.

          Show
          Keith Babo
          added a comment - On second thought, maybe we should wait until 0.5 to play around with the -ds.xml stuff. I have a feeling that resource dependencies will be required within AS for that to work correctly on startup.
          Hide
          Keith Babo
          added a comment -

          This looks great. Thanks, Tomo!

          Show
          Keith Babo
          added a comment - This looks great. Thanks, Tomo!

            People

            • Assignee:
              Tomohisa igarashi
              Reporter:
              Keith Babo
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: