Skip to content
This repository was archived by the owner on Jan 9, 2025. It is now read-only.

Support for CDI 3+ . fixes #509#513

Merged
fh-ms merged 1 commit intomicrostream-one:masterfrom
rdebusscher:integration-cdi3
Apr 21, 2023
Merged

Support for CDI 3+ . fixes #509#513
fh-ms merged 1 commit intomicrostream-one:masterfrom
rdebusscher:integration-cdi3

Conversation

@rdebusscher
Copy link
Copy Markdown

jakarta namespace version of the CDI integration.

@otaviojava
Copy link
Copy Markdown
Contributor

otaviojava commented Mar 2, 2023

@rdebusscher could you isolate the configuration in a module?

I will create a Jakarta-Data integration; thus, I will use the same configuration.

Also, please, provide one way to register the type handle.

@rdebusscher
Copy link
Copy Markdown
Author

@otaviojava

Not sure what you exactly need, is it just the cache part that you do not need? Can you just use this module as is for the moment?

Also, there will be no release of MicroStream 8 (listen here https://youtu.be/T5cJTF3zkuU?t=807) but only the Eclipse project release later on. So it might be easier that you copy what is needed for the demo at the moment so that we can do a proper solution when we have the Eclipse repositories after the creation review is finished.

@fh-ms fh-ms self-requested a review April 19, 2023 11:10
Copy link
Copy Markdown
Contributor

@fh-ms fh-ms left a comment

Choose a reason for hiding this comment

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

A link to cdi.adoc is missing in nav.adoc

@fh-ms fh-ms added this to the 08.00.00 milestone Apr 20, 2023
@rdebusscher
Copy link
Copy Markdown
Author

A link to cdi.adoc is missing in nav.adoc

I see the link present in the nav.adoc file. It is present in the current version of master and this PR does not make a change to the nav.adoc

@rdebusscher rdebusscher requested a review from fh-ms April 20, 2023 12:05
@fh-ms
Copy link
Copy Markdown
Contributor

fh-ms commented Apr 20, 2023

I see, my fault, the cdi.adoc was updated, not added.

@fh-ms fh-ms merged commit d013470 into microstream-one:master Apr 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants