Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SmallRye Reactive Messaging In Memory connector #53

Closed
wants to merge 292 commits into from
Closed

Conversation

kabir
Copy link
Owner

@kabir kabir commented Jul 5, 2023

No description provided.

@github-actions
Copy link

github-actions bot commented Jul 5, 2023

Dependency Tree Analyzer Output:

New Dependencies:

  • org.wildfly:wildfly-microprofile-reactive-messaging-inmemory:jar:29.0.0.Beta1-SNAPSHOT:compile

CC @wildfly/prod

<artifactItems>
<artifactItem>
<groupId>io.smallrye.reactive</groupId>
<artifactId>smallrye-reactive-messaging-in-memory</artifactId>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should try to find another way, and use the original artifact directly in the module.xml. Or it makes it hard to monitor for security vulnerabilities etc.

I understand the reason is to add a beans.xml. I think there is a way, but need to look into it. Hopefully this module will not be needed.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, so it seems that there is another way :-)
Under galleon-pack/galleon-shared/src/main/resources/packages, create a folder called the same as the module, i.e. io.smallrye.reactive.messaging.connector.inmemory. In there adjust the instructions in https://docs.wildfly.org/galleon-plugins/#_defining_packages to add a META-INF/beans.xml.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this method but in the end I can only add files to the root of the server, but the beans.xml is required in the /modules/system/layers/base/io/smallrye/reactive/messaging/connector/inmemory/main/smallrye-reactive-messaging-in-memory-4.6.0.jar I am not sure how to do the trick with the mentioned instructions.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kuhhpid If you have some WIP, let me know the branch and I can see if I can figure out that part

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kabir Here is the branch with the changes to use the ja directly https://github.com/kuhhpid/wildfly-smallrye-in-memory/tree/direct-jar-usage

@kabir
Copy link
Owner Author

kabir commented Jul 5, 2023

Although, this will need some changes, it will still be relevant once you add the in memory connector dependency 'properly' :-)

Basically, the only places we should use version properties is in the section of a parent pom or a BOM. In this case this will be boms/common-expansion/pom.xml (look for where io.smallrye.reactive:smallrye-reactive-messaging-kafka is added). Then we can refer to that in the rest of the project without a version.

Then it will also need adding in these files

% git grep "<artifactId>smallrye-reactive-messaging-kafka</artifactId>"
boms/common-expansion/pom.xml:                <artifactId>smallrye-reactive-messaging-kafka</artifactId>
galleon-pack/galleon-shared/pom.xml:            <artifactId>smallrye-reactive-messaging-kafka</artifactId>
galleon-pack/galleon-shared/src/main/resources/license/licenses.xml:            <artifactId>smallrye-reactive-messaging-kafka</artifactId>

and should be automatically picked up when the module.xml is created

<module xmlns="urn:jboss:module:1.9" name="io.smallrye.reactive.messaging.connector.inmemory">

<resources>
<artifact name="${org.wildfly:wildfly-microprofile-reactive-messaging-inmemory}"/>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the other stuff I have mentioned, adjust this to be the smallrye in memory connector maven artifact

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yrodiere and others added 25 commits July 7, 2023 18:27
Fix for WFLY-18222, Add some metadata to Galleon layers to allow for layers discovery
[WFLY-16528] JSFDeploymentProcessorTestCase fails with Faces 4
…ghlighter.WFLY-18213

WFLY-18213 asciidoctor-maven-plugin attribute sourceHighlighter shoul…
WFLY-18225 Upgrade smallrye-open-api to 3.4.0
WFLY-18227 Upgrade to Hibernate Search 6.2.0.Final
WFLY-18239 Upgrade Infinispan to 14.0.12.Final
[WFLY-18238] Upgrade Byteman to 4.0.21
WFLY-18242 Upgrade Weld to 5.1.1.SP1
WFLY-18150 DistributableTimerService.getTimers() collection may omit timers during concurrent rescheduling process
bstansberry and others added 27 commits September 5, 2023 14:33
…ly using the HotRod-based HttpSession manager
* use correct heading level
* include the references in the doc instead of having a title that links to it

Signed-off-by: Jeff Mesnil <jmesnil@redhat.com>
[WFLY-18324] JCA: allow for empty username and password in recovery configuration
[WFLY-18380]: Message-destination-type is ejb-jar.xml is ignored.
WFLY-18389 <max-active-sessions/> causes sessions to expire prematurely using the HotRod-based HttpSession manager
WFLY-18404 HotRod-based session manager creates too many threads for handling concurrent expiration events
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
[WFLY-18237]: Adding a connector shouldn't require to reload.
WIP WFLY-18414 Create a standard-test-expansion bom
…ns/actions/checkout-4

Bump actions/checkout from 3 to 4
WFLY-18413 Ensure PropertyChangeListener is removed when MetricRegistration is unregistered
…ache.qpid-proton-j-0.34.1

[WFLY-18421] Bump org.apache.qpid:proton-j from 0.34.0 to 0.34.1
…n.org.glassfish.soteria-3.0.3

WFLY-18418] Bump version.org.glassfish.soteria from 3.0.0 to 3.0.3
WFLY-18414 Remove manages deps on jaxen & spring-kafka-test
…n.software.amazon.awssdk-2.20.126

Bump version.software.amazon.awssdk from 2.20.77 to 2.20.126
…n.io.netty-4.1.97.Final

[WFLY-18419] Bump version.io.netty from 4.1.96.Final to 4.1.97.Final
@kabir kabir closed this Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.