Git Product home page Git Product logo

gs-messaging-jms's People

Contributors

alek-sys avatar bclozel avatar btalbott avatar cbeams avatar fisco-unimatic avatar fml2 avatar gregturn avatar habuma avatar jochen777 avatar joshlong avatar robertmcnees avatar royclarkson avatar sergiovlvitorino avatar snicoll avatar spring-operator avatar wilkinsona avatar zyro23 avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

gs-messaging-jms's Issues

Upgrade to latest Spring Boot version

An easy place to get started with contributing to Spring is keeping the Spring Boot version up-to-date, so I'm adding this issue as an invitation to first-time contributors.

You need to set the Spring Boot version in:

  • initial/build.gradle
  • initial/pom.xml
  • complete/build.gradle
  • complete/pom.xml

Then run test/run.sh to validate the upgrade worked as expected.

Remove unused CI files

The file test/run.sh is no longer required. This file and the test directory can be deleted.

The file was originally used for Jenkins CI. Now that this project has been converted to GitHub Actions, this file can be removed.

Please clarify DefaultMessageListenerContainer and CachingConnectionFactory relation

Readme says:

That’s why we recommend that you use a listener container such as DefaultMessageListenerContainer with a cache-based connection factory

This statement is fulfilled by AbstractJmsListenerContainerFactory#createListenerContainer - on debug I see CachingConnectionFactory is passed in line 190.

On the other hand, javadoc https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/jms/listener/DefaultMessageListenerContainer.html says:

Note: Don't use Spring's CachingConnectionFactory in combination with dynamic scaling. Ideally, don't use it with a message listener container at all, since it is generally preferable to let the listener container itself handle appropriate caching within its lifecycle. Also, stopping and restarting a listener container will only work with an independent, locally cached Connection - not with an externally cached one.

I'm confused: Should CachingConnectionFactory be used or not?

Dependencies should be listed clearly.

I built the project by the Spring Initializer on VS Code and I had to check this github repo code for the differences because after all the steps I followed the app had not worked. I believe the dependecies artemis-jakarta-server and spring-boot-starter-json should be clearly stated in the tutorial.

The code fails with Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.jms.support.converter.MessageConverter]: Factory method 'jascksonJmsMessageConverter' threw exception with message: com/fasterxml/jackson/databind/ObjectMapper

Perhaps it needs to include jackson databind in the maven

com.fasterxml.jackson.core jackson-databind 2.14.1

After including this library the error changes to
jakarta.jms.JMSException: Failed to create session factory

Perhaps it needs a JMS broker running. However the guide does not provide any instructions. It would be better if it could provide steps like it was provided for messaging with RabbitMq and Redis

PubSubDomain

I don't know why, but on my computer the code works only when container.setPubSubDomain; is set to false and not true like the code.
It's a bug ?

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.