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

Configuration transactions #6

Open
wants to merge 2,329 commits into
base: #15752
Choose a base branch
from

Conversation

marcelhanser
Copy link

Merged main & implemented the caching remark.

@marcelhanser
Copy link
Author

If i got the contributing workflow totally wrong - i apologize. My thoughts have been - if i want to get my additions into your pull-request at the quarkus-repo i need to make a pull-request 1st to your branch in your repo-fork.

@geoand
Copy link
Owner

geoand commented May 19, 2021 via email

@marcelhanser
Copy link
Author

Hi, Thx for the fast response.

Makes sense^^

I just did that, but now there are 2 pull-requests:
quarkusio#15832
quarkusio#17357
And i just wanted to add my changes to you branch and so to your pull-request^^ - but maybe this is not how it works.

@geoand
Copy link
Owner

geoand commented May 19, 2021

Ah, now I see what you mean.

What you have one in quarkusio#17357 is fine. I'll close my old PR in favor of yours

@marcelhanser marcelhanser force-pushed the configuration-transactions branch 2 times, most recently from 4c41c6c to ffa831e Compare May 20, 2021 06:23
stuartwdouglas and others added 24 commits June 2, 2021 09:55
Pass max frame size to VertxServerWebSocketContainer
This is not perfect, as this information
should be read from the plugin config,
but it will prevent many issues.

Relates to quarkusio#17558
Preserve TCCL when using coroutines in RESTEasy Reactive
…losingTarget()"

This reverts the workaround from commit f6cffe0,
but preserves the test.

Assuming jandex indexes are always built with Jandex 2.3.0+, we no
longer need this workaround.
Update Vert.x Mutiny bindings to version 2.6.0
Update SmallRye Reactive Messaging to version: 3.4.0
Don't run maven IT names by default
Do not delete BrotliDecoder and let GraalVM DCE do it for us
The current short option to display the version is "-V" (uppercase): `quarkus -V`. This is set by Picocli, but can be changed using the `picocli.version.name.0` property.

The problem with `-V` is that does not match with the other short options to display version in the extensions and other commands which is "-v".
Also, in other command tools like `mvn`, the option is `-v`.
I did check that the `-v` is not conflicting with other existing options.
…ssdk.version-2.16.75

Bump awssdk.version from 2.16.74 to 2.16.75
Remove support for hibernate.properties (long deprecated)
…ive-messaging-mutiny-bindings

Update Vert.x to version 4.1.0 and related dependencies
…test

Add rule for @QuarkusIntegrationTest to the bot
Upgrade to Hibernate Reactive 1.0.0.CR6
Ensure @QuarkusIntegrationTest works properly with any Filesystem path
Also use the executor included in the container info even if the code
was working ok.

Fixes quarkusio#17571
…t-docs

Update to GraphQL client user guide
gsmet and others added 29 commits June 10, 2021 15:43
…g.yaml-snakeyaml-1.29

Bump snakeyaml from 1.28 to 1.29
This simplifies things a lot and will also avoid race conditions.
This interface doesn't seem to be used anywhere
…he-kotlin-guide

Create a MongoDB with Panache and Kotlin guide
Signed-off-by: Phillip Kruger <phillip.kruger@gmail.com>
Bumps [flyway-core](https://github.com/flyway/flyway) from 7.9.2 to 7.10.0.
- [Release notes](https://github.com/flyway/flyway/releases)
- [Commits](flyway/flyway@flyway-7.9.2...flyway-7.10.0)

---
updated-dependencies:
- dependency-name: org.flywaydb:flyway-core
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [postgresql](https://github.com/pgjdbc/pgjdbc) from 42.2.20 to 42.2.21.
- [Release notes](https://github.com/pgjdbc/pgjdbc/releases)
- [Changelog](https://github.com/pgjdbc/pgjdbc/blob/REL42.2.21/CHANGELOG.md)
- [Commits](pgjdbc/pgjdbc@REL42.2.20...REL42.2.21)

---
updated-dependencies:
- dependency-name: org.postgresql:postgresql
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Refactor Amazon Services initialization a bit
3 is the Micrometer default, 1024 is way
to big and results in excessive memory usage.
Signed-off-by: Phillip Kruger <phillip.kruger@gmail.com>
- Press [q] to quit
- Accept live reload commands when tests are not running
- Don't display live reload commands in quarkus:test
- Fix quarkus:test if vertx-http was not present
…raphql-1.1.0

Upgrade to SmallRye GraphQL 1.2.5 and MicroProfile GraphQL 1.1.0
…ql-guide-update

Update the SmallRye GraphQL Extension documentation
…pl-parentFirst

Add org.apache.maven.resolver:maven-resolver-impl as parentFirstArtifact
Fix potential NPE in RESTEasy Classic build time processing
Resolves: quarkusio#15752
@gsmet gsmet force-pushed the configuration-transactions branch from e727d9d to bbd8f25 Compare June 11, 2021 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.