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

Quarkus integration#422

Merged
rdebusscher merged 14 commits intomicrostream-one:masterfrom
rdebusscher:quarkus-integration
Nov 18, 2022
Merged

Quarkus integration#422
rdebusscher merged 14 commits intomicrostream-one:masterfrom
rdebusscher:quarkus-integration

Conversation

@rdebusscher
Copy link
Copy Markdown

@rdebusscher rdebusscher commented Aug 31, 2022

Extension for Quarkus.

Support for StorageManager configuration through config parameters
Root bean creation (@Storage)
Customizer of Foundation and initial data.

Examples, see https://github.com/rdebusscher/microstream-quarkus-patterns

@rdebusscher rdebusscher marked this pull request as ready for review September 2, 2022 11:19
@fh-ms fh-ms added this to the 08.00.00 milestone Sep 5, 2022
Rudy De Busscher and others added 4 commits September 5, 2022 16:05
* setup javac as default
* add check if the spring-configuration-metadata.json exists in target.

(cherry picked from commit ac6eadd)
@zdenek-jonas
Copy link
Copy Markdown
Contributor

Please do sync merge with master first... now we have a lot of changes that are already in master...

@hg-ms hg-ms self-requested a review October 27, 2022 07:24
Copy link
Copy Markdown
Contributor

@hg-ms hg-ms left a comment

Choose a reason for hiding this comment

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

I saw several unnecessary empty line in several files, maybe you can remove some of them

@rdebusscher rdebusscher requested a review from hg-ms November 2, 2022 09:58
.done();

}
else
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unnecessary else block

@fh-ms
Copy link
Copy Markdown
Contributor

fh-ms commented Nov 14, 2022

Another thought, shouldn't the @Storage annotation be called @Root or @StorageRoot?

@rdebusscher
Copy link
Copy Markdown
Author

Another thought, shouldn't the @Storage annotation be called @root or @storageRoot?

Maybe/probably. But I'm just following the same naming which was introduced at first with the CDI integration
https://github.com/microstream-one/microstream/blob/master/integrations/cdi/src/main/java/one/microstream/integrations/cdi/types/Storage.java

And later on also used in Spring Boot integration.

If it needs to be different, we should also use the new name for the 2 other integrations.

@rdebusscher rdebusscher requested a review from fh-ms November 17, 2022 12:26
@rdebusscher rdebusscher merged commit 9fa3542 into microstream-one:master Nov 18, 2022
@rdebusscher rdebusscher deleted the quarkus-integration branch December 13, 2022 09:00
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.

4 participants