From 8ef59acc7d89fce8a62bfc4934095a491714ba06 Mon Sep 17 00:00:00 2001 From: Guillaume Smet Date: Tue, 21 Jun 2022 17:49:14 +0200 Subject: [PATCH 01/36] Add some missing extension descriptions These are needed for code.quarkus.io. (cherry picked from commit d6b7cc956107758eb23782c0f8594037bb472fe9) --- extensions/amazon-lambda-rest/runtime/pom.xml | 1 + extensions/narayana-lra/runtime/pom.xml | 1 + .../runtime/pom.xml | 3 ++- extensions/schema-registry/apicurio/avro/runtime/pom.xml | 1 + extensions/schema-registry/confluent/avro/runtime/pom.xml | 1 + 5 files changed, 6 insertions(+), 1 deletion(-) diff --git a/extensions/amazon-lambda-rest/runtime/pom.xml b/extensions/amazon-lambda-rest/runtime/pom.xml index 18fd83da08af9..7d7d5564fcba4 100644 --- a/extensions/amazon-lambda-rest/runtime/pom.xml +++ b/extensions/amazon-lambda-rest/runtime/pom.xml @@ -13,6 +13,7 @@ quarkus-amazon-lambda-rest Quarkus - Amazon Lambda AWS Gateway REST API- Runtime + Build an API Gateway REST API with Lambda integration diff --git a/extensions/narayana-lra/runtime/pom.xml b/extensions/narayana-lra/runtime/pom.xml index f0d2d50344d8d..149efe5ab6cda 100644 --- a/extensions/narayana-lra/runtime/pom.xml +++ b/extensions/narayana-lra/runtime/pom.xml @@ -12,6 +12,7 @@ quarkus-narayana-lra Quarkus - Narayana LRA Participant - Runtime + Coordinate Long Running Actions (LRA) diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml index fbeebd946aecd..c3649a592ecdf 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml @@ -12,6 +12,7 @@ quarkus-resteasy-reactive-kotlin-serialization Quarkus - RestEasy Reactive Kotlin Serialization - Runtime + Kotlin Serialization support for RESTEasy Reactive. This extension is not compatible with the quarkus-resteasy extension, or any of the extensions that depend on it. @@ -96,4 +97,4 @@ - \ No newline at end of file + diff --git a/extensions/schema-registry/apicurio/avro/runtime/pom.xml b/extensions/schema-registry/apicurio/avro/runtime/pom.xml index 014dbf81b6dc5..3eab1ddf7b99c 100644 --- a/extensions/schema-registry/apicurio/avro/runtime/pom.xml +++ b/extensions/schema-registry/apicurio/avro/runtime/pom.xml @@ -12,6 +12,7 @@ quarkus-apicurio-registry-avro Quarkus - Apicurio Registry - Avro - Runtime + Use Apicurio as Avro schema registry diff --git a/extensions/schema-registry/confluent/avro/runtime/pom.xml b/extensions/schema-registry/confluent/avro/runtime/pom.xml index 357cbc3188f6e..ec4593f1cb9ee 100644 --- a/extensions/schema-registry/confluent/avro/runtime/pom.xml +++ b/extensions/schema-registry/confluent/avro/runtime/pom.xml @@ -12,6 +12,7 @@ quarkus-confluent-registry-avro Quarkus - Confluent Schema Registry - Avro - Runtime + Use Confluent as Avro schema registry From ebf5ee11e149de4942f0f0765d5174ebcdead881 Mon Sep 17 00:00:00 2001 From: Guillaume Smet Date: Tue, 21 Jun 2022 17:51:12 +0200 Subject: [PATCH 02/36] Fix case of RESTEasy Reactive in module names (cherry picked from commit 48a04029a95c9f4777023ede9ccb2d1b3cf61d94) --- .../deployment/pom.xml | 2 +- .../quarkus-resteasy-reactive-kotlin-serialization/pom.xml | 2 +- .../runtime/pom.xml | 2 +- integration-tests/oidc-client-reactive/pom.xml | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/deployment/pom.xml b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/deployment/pom.xml index 39ffdaafac328..ab282d873fc01 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/deployment/pom.xml +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/deployment/pom.xml @@ -11,7 +11,7 @@ quarkus-resteasy-reactive-kotlin-serialization-deployment - Quarkus - RestEasy Reactive Kotlin Serialization - Deployment + Quarkus - RESTEasy Reactive - Kotlin Serialization - Deployment diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/pom.xml b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/pom.xml index e34358d5e6b60..7e9effa35238a 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/pom.xml +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/pom.xml @@ -11,7 +11,7 @@ 4.0.0 quarkus-resteasy-reactive-kotlin-serialization-parent - Quarkus - RestEasy Reactive Kotlin Serialization + Quarkus - RESTEasy Reactive - Kotlin Serialization pom diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml index c3649a592ecdf..8a26a26c73fe6 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-kotlin-serialization/runtime/pom.xml @@ -11,7 +11,7 @@ 4.0.0 quarkus-resteasy-reactive-kotlin-serialization - Quarkus - RestEasy Reactive Kotlin Serialization - Runtime + Quarkus - RESTEasy Reactive - Kotlin Serialization - Runtime Kotlin Serialization support for RESTEasy Reactive. This extension is not compatible with the quarkus-resteasy extension, or any of the extensions that depend on it. diff --git a/integration-tests/oidc-client-reactive/pom.xml b/integration-tests/oidc-client-reactive/pom.xml index 885f1a80f0a48..73f001fa19940 100644 --- a/integration-tests/oidc-client-reactive/pom.xml +++ b/integration-tests/oidc-client-reactive/pom.xml @@ -11,8 +11,8 @@ 4.0.0 quarkus-integration-test-oidc-client-reactive - Quarkus - Integration Tests - OpenID Connect Client Reactive - Module that contains OpenID Connect Client tests using RestEasy Reactive + Quarkus - Integration Tests - OpenID Connect Client Reactive + Module that contains OpenID Connect Client tests using RESTEasy Reactive http://localhost:8180/auth From a25b60bc1cefed4355d9f3f79d5161f0b1a558c5 Mon Sep 17 00:00:00 2001 From: Stuart Douglas Date: Sun, 29 May 2022 11:08:43 +1000 Subject: [PATCH 03/36] Improve continuous testing output - Trim the console stack traces - Add a 2 line summary at the bottom to make it easier to view the full results (cherry picked from commit 4dea8f48286e2cfe5535fc181bafc8f36b4faf5b) --- .../dev/testing/JunitTestRunner.java | 6 ++++ .../dev/testing/TestConsoleHandler.java | 31 +++++++++++++++++++ .../test/WebsocketDevModeTestCase.java | 6 ++-- 3 files changed, 41 insertions(+), 2 deletions(-) diff --git a/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/JunitTestRunner.java b/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/JunitTestRunner.java index 328d1a12e2b13..30fad8206dfdf 100644 --- a/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/JunitTestRunner.java +++ b/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/JunitTestRunner.java @@ -339,6 +339,12 @@ public void executionFinished(TestIdentifier testIdentifier, TestExecutionResult.failed(failure), List.of(), false, runId, 0, false)); results.put(UniqueId.parse(currentNonDynamicTest.get().getUniqueId()), result); + } else if (testExecutionResult.getStatus() == TestExecutionResult.Status.FAILED) { + Throwable throwable = testExecutionResult.getThrowable().get(); + trimStackTrace(testClass, throwable); + for (var i : throwable.getSuppressed()) { + trimStackTrace(testClass, i); + } } } else if (testExecutionResult.getStatus() == TestExecutionResult.Status.FAILED) { //if a parent fails we fail the children diff --git a/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/TestConsoleHandler.java b/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/TestConsoleHandler.java index 9225b2b8fc46d..539cdc22c7cdd 100644 --- a/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/TestConsoleHandler.java +++ b/core/deployment/src/main/java/io/quarkus/deployment/dev/testing/TestConsoleHandler.java @@ -217,6 +217,7 @@ public void runComplete(TestRunResults results) { } else { currentlyFailing = true; //TODO: this should not use the logger, it should print a nicer status + //first print the full failures log.error(statusHeader("TEST REPORT #" + results.getId())); for (Map.Entry classEntry : results.getCurrentFailing().entrySet()) { for (TestResult test : classEntry.getValue().getFailing()) { @@ -227,6 +228,36 @@ public void runComplete(TestRunResults results) { } } } + //then print the summary + StringBuilder summary = new StringBuilder(statusFooter(RED + "Summary:") + "\n"); + for (Map.Entry classEntry : results.getCurrentFailing().entrySet()) { + for (TestResult test : classEntry.getValue().getFailing()) { + if (test.isReportable()) { + StackTraceElement testclass = null; + for (var i : test.getTestExecutionResult().getThrowable().get().getStackTrace()) { + if (i.getClassName().equals(test.testClass)) { + testclass = i; + break; + } + } + + if (summary.charAt(summary.length() - 1) != '\n') { + summary.append("\n"); + } + if (testclass != null) { + summary.append(testclass.getClassName() + "#" + testclass.getMethodName() + "(" + + testclass.getFileName() + ":" + testclass.getLineNumber() + ") "); + } + summary.append(RED + + test.getDisplayName() + RESET + + " " + test.getTestExecutionResult().getThrowable().get().getMessage()); + } + } + } + while (summary.charAt(summary.length() - 1) == '\n') { + summary.setLength(summary.length() - 1); + } + log.error(summary.toString()); log.error( statusFooter(RED + results.getCurrentFailedCount() + " " + pluralize("TEST", "TESTS", results.getCurrentFailedCount()) + " FAILED")); diff --git a/extensions/websockets/server/deployment/src/test/java/io/quarkus/websockets/test/WebsocketDevModeTestCase.java b/extensions/websockets/server/deployment/src/test/java/io/quarkus/websockets/test/WebsocketDevModeTestCase.java index 43b80163ec843..bc7aaac5fe6a9 100644 --- a/extensions/websockets/server/deployment/src/test/java/io/quarkus/websockets/test/WebsocketDevModeTestCase.java +++ b/extensions/websockets/server/deployment/src/test/java/io/quarkus/websockets/test/WebsocketDevModeTestCase.java @@ -13,6 +13,7 @@ import javax.websocket.Session; import org.jboss.shrinkwrap.api.ShrinkWrap; +import org.jboss.shrinkwrap.api.asset.StringAsset; import org.jboss.shrinkwrap.api.spec.JavaArchive; import org.junit.jupiter.api.Assertions; import org.junit.jupiter.api.Test; @@ -26,7 +27,7 @@ */ public class WebsocketDevModeTestCase { - @TestHTTPResource("echo") + @TestHTTPResource("api/echo") URI echoUri; @RegisterExtension @@ -35,7 +36,8 @@ public class WebsocketDevModeTestCase { @Override public JavaArchive get() { return ShrinkWrap.create(JavaArchive.class) - .addClasses(EchoWebSocket.class, EchoService.class); + .addClasses(EchoWebSocket.class, EchoService.class) + .addAsResource(new StringAsset("quarkus.http.root-path=/api/"), "application.properties"); } }); From b3dc894a04f6923bc18c4cfb7995569bb354c87f Mon Sep 17 00:00:00 2001 From: Julien Ponge Date: Tue, 21 Jun 2022 19:13:46 +0200 Subject: [PATCH 04/36] Upgrade Mutiny-dependent modules to 1.6.0 (cherry picked from commit 80542672539792720b74c2b9697125ef75be4831) --- bom/application/pom.xml | 2 +- independent-projects/qute/pom.xml | 2 +- independent-projects/resteasy-reactive/pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/bom/application/pom.xml b/bom/application/pom.xml index 523898e15a9d0..05cb5d2c70b0d 100644 --- a/bom/application/pom.xml +++ b/bom/application/pom.xml @@ -133,7 +133,7 @@ 4.1.74.Final 1.0.3 3.5.0.Final - 1.5.0 + 1.6.0 3.1.0 1.8.0 1.1.8.4 diff --git a/independent-projects/qute/pom.xml b/independent-projects/qute/pom.xml index be346059d4418..f8a83c5ba1c56 100644 --- a/independent-projects/qute/pom.xml +++ b/independent-projects/qute/pom.xml @@ -46,7 +46,7 @@ 3.5.0.Final 3.0.0-M7 1.6.8 - 1.5.0 + 1.6.0 diff --git a/independent-projects/resteasy-reactive/pom.xml b/independent-projects/resteasy-reactive/pom.xml index dfe5369df1037..72b88d8647a60 100644 --- a/independent-projects/resteasy-reactive/pom.xml +++ b/independent-projects/resteasy-reactive/pom.xml @@ -54,7 +54,7 @@ 1.6.8 2.0.1.Final 1.1.6 - 1.5.0 + 1.6.0 1.12.0 4.2.4 4.5.1 From 26e05123bdabb83d1ce236c4fcd4082ff03dc7f8 Mon Sep 17 00:00:00 2001 From: Martin Kouba Date: Tue, 21 Jun 2022 11:37:27 +0200 Subject: [PATCH 05/36] Qute - fix Template#getParameterDeclarations() - the param declaration node may be removed as a part of a standalone line (cherry picked from commit e2e0cd0eb3e8e750b9ba79fd7904dc70f15b12b8) --- ...tionDefaultValueValidationFailureTest.java | 2 +- .../io/quarkus/qute/ParameterDeclaration.java | 24 +++++++++++++++++++ .../java/io/quarkus/qute/ParserHelper.java | 3 ++- .../main/java/io/quarkus/qute/Template.java | 2 +- .../java/io/quarkus/qute/TemplateImpl.java | 5 +++- .../java/io/quarkus/qute/TemplateNode.java | 1 + .../ParamDeclarationDefaultValueTest.java | 14 +++++++++++ 7 files changed, 47 insertions(+), 4 deletions(-) diff --git a/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/ParamDeclarationDefaultValueValidationFailureTest.java b/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/ParamDeclarationDefaultValueValidationFailureTest.java index 01e276d389ce0..4487548a95a91 100644 --- a/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/ParamDeclarationDefaultValueValidationFailureTest.java +++ b/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/ParamDeclarationDefaultValueValidationFailureTest.java @@ -17,7 +17,7 @@ public class ParamDeclarationDefaultValueValidationFailureTest { static final QuarkusUnitTest config = new QuarkusUnitTest() .withApplicationRoot(root -> root .addAsResource(new StringAsset( - "{@java.lang.String myName=1}{myName}"), + "{@java.lang.String myName=1}\n{myName}"), "templates/myName.html")) .assertException(t -> { Throwable e = t; diff --git a/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParameterDeclaration.java b/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParameterDeclaration.java index 72b17671ba0e7..6a6da4bce112b 100644 --- a/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParameterDeclaration.java +++ b/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParameterDeclaration.java @@ -1,14 +1,38 @@ package io.quarkus.qute; +import io.quarkus.qute.Expression.Part; +import io.quarkus.qute.TemplateNode.Origin; + /** * Represents a parameter declaration, i.e. {@org.acme.Foo foo}. */ public interface ParameterDeclaration { + /** + * The type info for {@org.acme.Foo foo} is {@code |org.acme.Foo|}. + * + * @return the unparsed type info + * @see Part#getTypeInfo() + */ String getTypeInfo(); + /** + * The key for {@org.acme.Foo foo} is {@code foo}. + * + * @return the key + */ String getKey(); + /** + * + * @return the default value or {@code null} if no default value is set + */ Expression getDefaultValue(); + /** + * + * @return the origin of the template node + */ + Origin getOrigin(); + } diff --git a/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParserHelper.java b/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParserHelper.java index 2b22ccbb5d090..9d25886f270b2 100644 --- a/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParserHelper.java +++ b/independent-projects/qute/core/src/main/java/io/quarkus/qute/ParserHelper.java @@ -15,7 +15,8 @@ public interface ParserHelper { String getTemplateId(); /** - * Adds an implicit parameter declaration. This an alternative approach to explicit parameter declarations + * Adds an implicit parameter declaration. This is an alternative approach to explicit parameter + * declarations * used directly in the templates, e.g. {@org.acme.Foo foo}. * * @param name diff --git a/independent-projects/qute/core/src/main/java/io/quarkus/qute/Template.java b/independent-projects/qute/core/src/main/java/io/quarkus/qute/Template.java index 3a2e315e53877..6b66071cccdbe 100644 --- a/independent-projects/qute/core/src/main/java/io/quarkus/qute/Template.java +++ b/independent-projects/qute/core/src/main/java/io/quarkus/qute/Template.java @@ -152,7 +152,7 @@ default String render() { /** * - * @return an immutable list of parameter declarations + * @return an immutable list of all parameter declarations defined in the template */ List getParameterDeclarations(); diff --git a/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateImpl.java b/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateImpl.java index 9cd74648b889e..737f9283e7af9 100644 --- a/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateImpl.java +++ b/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateImpl.java @@ -23,6 +23,7 @@ class TemplateImpl implements Template { private final EngineImpl engine; private final Optional variant; final SectionNode root; + private final List parameterDeclarations; TemplateImpl(EngineImpl engine, SectionNode root, String templateId, String generatedId, Optional variant) { this.engine = engine; @@ -30,6 +31,8 @@ class TemplateImpl implements Template { this.templateId = templateId; this.generatedId = generatedId; this.variant = variant; + // Note that param declarations can be removed if placed on a standalone line + this.parameterDeclarations = ImmutableList.copyOf(root.getParameterDeclarations()); } @Override @@ -55,7 +58,7 @@ public Expression findExpression(Predicate predicate) { @Override public List getParameterDeclarations() { - return root.getParameterDeclarations(); + return parameterDeclarations; } @Override diff --git a/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateNode.java b/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateNode.java index a28e768609ebc..6b82450d057bb 100644 --- a/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateNode.java +++ b/independent-projects/qute/core/src/main/java/io/quarkus/qute/TemplateNode.java @@ -26,6 +26,7 @@ default List getExpressions() { } /** + * Returns the parameter declarations defined in this template node. * * @return a list of param declarations */ diff --git a/independent-projects/qute/core/src/test/java/io/quarkus/qute/ParamDeclarationDefaultValueTest.java b/independent-projects/qute/core/src/test/java/io/quarkus/qute/ParamDeclarationDefaultValueTest.java index 8720ad3dbb58c..ed5872e058209 100644 --- a/independent-projects/qute/core/src/test/java/io/quarkus/qute/ParamDeclarationDefaultValueTest.java +++ b/independent-projects/qute/core/src/test/java/io/quarkus/qute/ParamDeclarationDefaultValueTest.java @@ -3,6 +3,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; +import java.util.List; import org.junit.jupiter.api.Test; public class ParamDeclarationDefaultValueTest { @@ -24,6 +25,19 @@ public void testMultipleDefaultValues() { Template template = engine .parse("{@java.lang.String val='foo'}\n{@int anotherVal=1}\n{val}::{anotherVal}"); assertEquals("foo::2", template.data("anotherVal", 2).render()); + List parameterDeclarations = template.getParameterDeclarations(); + assertEquals(2, parameterDeclarations.size()); + for (ParameterDeclaration pd : parameterDeclarations) { + if (pd.getKey().equals("val")) { + assertEquals("'foo'", pd.getDefaultValue().toOriginalString()); + assertEquals("|java.lang.String|", pd.getTypeInfo()); + assertEquals(1, pd.getOrigin().getLine()); + } else { + assertEquals("anotherVal", pd.getKey()); + assertEquals("1", pd.getDefaultValue().toOriginalString()); + assertEquals("|int|", pd.getTypeInfo()); + } + } } private void assertDefaultValue(Template template, String expectedOutput) { From 19c7c9431322d6261a0c940efe1ba6dcb51c2ff1 Mon Sep 17 00:00:00 2001 From: Julien Ponge Date: Wed, 22 Jun 2022 14:21:06 +0200 Subject: [PATCH 06/36] Fix Asciidoc bug in the Mutiny primer (cherry picked from commit 70833b3f1d00767c0cf82a9062cbf0ce9f996786) --- docs/src/main/asciidoc/mutiny-primer.adoc | 44 +++++++++++++++++------ 1 file changed, 34 insertions(+), 10 deletions(-) diff --git a/docs/src/main/asciidoc/mutiny-primer.adoc b/docs/src/main/asciidoc/mutiny-primer.adoc index 24d268dee098c..cef721d5e8318 100644 --- a/docs/src/main/asciidoc/mutiny-primer.adoc +++ b/docs/src/main/asciidoc/mutiny-primer.adoc @@ -288,16 +288,40 @@ Fortunately, Mutiny provides a set of shortcut to make your code more concise: |=== -|Shortcut |Equivalent - -| uni.map(x → y) | uni.onItem().transform(x → y) -| uni.flatMap(x → uni2) | uni.onItem().transformToUni(x → uni2) -| uni.chain(x → uni2) | uni.onItem().transformToUni(x → uni2) -| uni.then(() → uni2) | uni.onItem().transformToUni(ignored → uni2) -| uni.invoke(x → System.out.println(x)) | uni.onItem().invoke(x → System.out.println(x)) -| uni.call(x → uni2) | uni.onItem().call(x → uni2) -| uni.eventually(() → System.out.println("eventually")) | uni.onItemOrFailure().invoke((ignoredItem, ignoredException) → System.out.println("eventually")) -| uni.eventually(() → uni2) | uni.onItemOrFailure().call((ignoredItem, ignoredException) → uni2) +|Shortcut | Equivalent + +|`uni.map(x -> y)` +|`uni.onItem().transform(x -> y)` + +|`uni.flatMap(x -> uni2)` +|`uni.onItem().transformToUni(x -> uni2)` + +|`uni.chain(x -> uni2)` +|`uni.onItem().transformToUni(x -> uni2)` + +|`uni.then(() -> uni2)` +|`uni.onItem().transformToUni(ignored -> uni2)` + +|`uni.invoke(x -> System.out.println(x))` +|`uni.onItem().invoke(x -> System.out.println(x))` + +|`uni.call(x -> uni2)` +|`uni.onItem().call(x -> uni2)` + +|`uni.eventually\(() -> System.out.println("eventually"))` +|`uni.onItemOrFailure().invoke\((ignoredItem, ignoredException) -> System.out.println("eventually"))` + +|`uni.eventually(() -> uni2)` +|`uni.onItemOrFailure().call((ignoredItem, ignoredException) -> uni2)` + +|`uni.replaceWith(x)` +|`uni.onItem().transform(ignored -> x)` + +|`uni.replaceWith(uni2)` +|`uni.onItem().transformToUni(ignored -> uni2)` + +|`uni.replaceIfNullWith(x)` +|`uni.onItem().ifNull().continueWith(x)` |=== == Reactive Streams From ac865fe846742dbdd4da2f9491eea3051886762f Mon Sep 17 00:00:00 2001 From: Marc Wrobel Date: Mon, 20 Jun 2022 14:42:23 +0200 Subject: [PATCH 07/36] Fix and improve javadoc in extensions projects (cherry picked from commit 8816386b87e145ade6ef55aa0d7572f20c6eec21) --- .../agroal/deployment/AgroalProcessor.java | 2 +- .../AmazonLambdaCommonProcessor.java | 2 +- .../runtime/AbstractLambdaPollLoop.java | 4 +-- .../lambda/runtime/AmazonLambdaApi.java | 2 +- .../ReflectiveBeanClassBuildItem.java | 2 +- .../InterceptedStaticMethodsProcessor.java | 2 +- .../autoscope/AutoScopeBuildItemTest.java | 2 +- .../quarkus/arc/config/ConfigProperties.java | 2 +- .../quarkus/arc/lookup/LookupIfProperty.java | 2 +- .../arc/lookup/LookupUnlessProperty.java | 2 +- .../arc/properties/IfBuildProperty.java | 2 +- .../arc/properties/UnlessBuildProperty.java | 2 +- .../spi/ContainerImageBuilderBuildItem.java | 2 +- .../DevServicesDatasourceProcessor.java | 2 +- .../ElasticsearchBuildTimeConfig.java | 2 +- .../QuarkusSecurityCommonProcessor.java | 2 +- .../elytron/security/common/BcryptUtil.java | 2 +- .../common/runtime/ElytronCommonRecorder.java | 2 +- .../LdapSecurityRealmBuildTimeConfig.java | 2 +- .../LdapSecurityRealmRuntimeConfig.java | 2 +- .../deployment/SecurityDomainBuildItem.java | 2 +- .../security/runtime/ElytronRecorder.java | 2 +- .../io/quarkus/flyway/FlywayProcessor.java | 2 +- .../flyway/test/FlywayDevModeTest.java | 2 +- .../test/FlywayExtensionConfigEmptyTest.java | 2 +- ...nsionConfigMissingNamedDataSourceTest.java | 2 +- .../runtime/FlywayContainerProducer.java | 2 +- .../FlywayDataSourceBuildTimeConfig.java | 2 +- .../FlywayDataSourceRuntimeConfig.java | 2 +- .../CloudFunctionsDeploymentBuildStep.java | 2 +- .../gcp/functions/event/PubsubMessage.java | 2 +- .../CloudFunctionsDeploymentBuildStep.java | 2 +- .../CloudFunctionDeploymentBuildStep.java | 2 +- .../config/GrpcClientConfiguration.java | 6 ++-- .../runtime/config/GrpcCodeGenConfig.java | 2 +- .../config/GrpcServerBuildTimeConfig.java | 4 +-- .../deployment/HibernateEntityEnhancer.java | 2 +- .../orm/deployment/HibernateOrmConfig.java | 2 +- .../MyEntityWithFailingDDLGeneration.java | 2 +- .../PublicFieldAccessFieldTypesTest.java | 2 +- .../PublicFieldAccessInheritanceTest.java | 2 +- .../hibernate/orm/runtime/JPAConfig.java | 2 +- .../runtime/boot/FastBootMetadataBuilder.java | 4 +-- .../QuarkusPersistenceUnitDefinition.java | 2 +- .../PreconfiguredServiceRegistryBuilder.java | 2 +- .../orm/runtime/proxies/ProxyDefinitions.java | 4 +-- .../schema/SchemaManagementIntegrator.java | 2 +- .../PublicFieldAccessFieldTypesTest.java | 2 +- .../PublicFieldAccessInheritanceTest.java | 2 +- ...tHibernateReactivePersistenceProvider.java | 2 +- ...figuredReactiveServiceRegistryBuilder.java | 2 +- .../customized/CheckingVertxContext.java | 2 +- ...uarkusReactiveConnectionPoolInitiator.java | 2 +- extensions/infinispan-client/README.MD | 4 +-- .../deployment/InfinispanClientProcessor.java | 2 +- .../runtime/InfinispanClientProducer.java | 2 +- .../InfinispanClientRuntimeConfig.java | 6 ++-- .../jaeger/runtime/JaegerBuildTimeConfig.java | 2 +- .../quarkus/jaeger/runtime/JaegerConfig.java | 2 +- .../DevServicesDerbyDatasourceTestCase.java | 2 +- .../DevServicesMariaDBDatasourceTestCase.java | 2 +- extensions/jdbc/jdbc-mssql/runtime/pom.xml | 2 +- .../DevServicesMySQLDatasourceTestCase.java | 2 +- .../deployment/OracleMetadataOverrides.java | 4 +-- .../DevServicesOracleDatasourceTestCase.java | 2 +- ...vServicesPostgresqlDatasourceTestCase.java | 2 +- .../postgresql/runtime/graal/PgSQLXML.java | 6 ++-- .../deployment/KafkaBuildTimeConfig.java | 4 +-- .../KafkaDevServicesBuildTimeConfig.java | 2 +- .../KafkaStreamsBuildTimeConfig.java | 2 +- .../deployment/KafkaStreamsProcessor.java | 2 +- .../KeycloakPolicyEnforcerTenantConfig.java | 2 +- .../kotlin/deployment/KotlinProcessor.java | 2 +- .../runtime/KubernetesClientBuildConfig.java | 2 +- .../deployment/KubernetesConfigProcessor.java | 2 +- .../AbstractKubernetesConfigSourceUtil.java | 4 +-- .../KubernetesConfigBuildTimeConfig.java | 2 +- .../runtime/KubernetesConfigSourceConfig.java | 4 +-- .../KubernetesServiceBindingConfig.java | 4 +-- .../buildtime/ServiceBindingProcessor.java | 16 +++++----- ...esOptionalResourceDefinitionBuildItem.java | 2 +- .../deployment/AutoScalingConfig.java | 2 +- .../deployment/GitRepoVolumeConfig.java | 2 +- .../deployment/GlobalAutoScalingConfig.java | 2 +- .../kubernetes/deployment/KnativeConfig.java | 10 +++--- .../deployment/KnativeProcessor.java | 2 +- .../deployment/KubernetesConfig.java | 7 ++-- .../deployment/KubernetesDeploy.java | 14 +++----- .../deployment/KubernetesDeployer.java | 7 ++-- .../deployment/MinikubeManifestGenerator.java | 4 +-- .../deployment/OpenshiftConfig.java | 6 ++-- .../deployment/OpenshiftProcessor.java | 2 +- .../kubernetes/deployment/ProbeConfig.java | 2 +- .../kubernetes/deployment/TrafficConfig.java | 2 +- .../deployment/LiquibaseProcessor.java | 2 +- ...nsionConfigMissingNamedDataSourceTest.java | 2 +- .../runtime/LiquibaseFactoryProducer.java | 2 +- .../mailer/deployment/MailerProcessor.java | 2 +- .../java/io/quarkus/mailer/Attachment.java | 2 +- .../src/main/java/io/quarkus/mailer/Mail.java | 6 ++-- .../mailer/runtime/DkimSignOptionsConfig.java | 4 +-- .../io/quarkus/mailer/runtime/MailConfig.java | 2 +- .../mailer/runtime/TrustStoreConfig.java | 4 +-- .../deployment/MicrometerProcessor.java | 2 +- .../binder/HttpBinderProcessor.java | 2 +- .../runtime/CompositeRegistryCreator.java | 2 +- .../runtime/MicrometerRecorder.java | 2 +- .../runtime/binder/RequestMetricInfo.java | 2 +- .../binder/RestClientMetricsListener.java | 4 +-- .../binder/vertx/VertxHttpServerMetrics.java | 6 ++-- .../binder/vertx/VertxMetricsTags.java | 2 +- .../runtime/binder/vertx/VertxTcpMetrics.java | 2 +- .../runtime/config/MicrometerConfig.java | 4 +-- .../MongoClientBuildTimeConfig.java | 6 ++-- .../deployment/MongoClientProcessor.java | 8 ++--- .../io/quarkus/mongodb/MongoMetricsTest.java | 2 +- .../io/quarkus/mongodb/MongoTestBase.java | 2 +- .../mongodb/MongoWithReplicasTestBase.java | 2 +- .../reactive/ReactiveMongoCollection.java | 32 +++++++++---------- .../mongodb/runtime/CredentialConfig.java | 2 +- .../mongodb/runtime/MongoClientConfig.java | 6 ++-- .../mongodb/runtime/MongodbConfig.java | 2 +- .../graal/MongoClientSubstitutions.java | 2 +- .../mongodb/reactive/MongoTestBase.java | 2 +- .../reactive/MongoWithReplicasTestBase.java | 2 +- .../io/quarkus/narayana/jta/RunOptions.java | 2 +- .../TestTransactionInterceptor.java | 2 +- .../TransactionalInterceptorBase.java | 2 +- .../proxy/nonjaxrs/LRAParticipant.java | 4 +-- .../runtime/graal/NettySubstitutions.java | 6 ++-- .../netty/runtime/virtual/VirtualChannel.java | 4 +-- .../runtime/virtual/VirtualServerChannel.java | 2 +- .../oidc/client/runtime/OidcClientImpl.java | 2 +- .../oidc/common/runtime/OidcCommonConfig.java | 2 +- .../quarkus/oidc/deployment/DevUiConfig.java | 2 +- .../keycloak/DevServicesConfig.java | 4 +-- .../io/quarkus/oidc/OidcTenantConfig.java | 6 ++-- .../io/quarkus/oidc/TokenStateManager.java | 2 +- .../runtime/DefaultTenantConfigResolver.java | 2 +- .../runtime/OidcAuthenticationMechanism.java | 2 +- .../oidc/runtime/OidcProviderClient.java | 2 +- .../io/quarkus/oidc/runtime/OidcUtils.java | 2 +- .../runtime/OpenTelemetryUtil.java | 2 +- .../runtime/pom.xml | 2 +- .../orm/panache/kotlin/PanacheQuery.kt | 8 ++--- .../panache/kotlin/PanacheRepositoryBase.kt | 2 +- .../runtime/AdditionalJpaOperations.java | 4 +-- .../runtime/CustomCountPanacheQuery.java | 2 +- .../runtime/CustomCountPanacheQuery.java | 2 +- .../common/binder/MongoParserVisitor.java | 2 +- .../runtime/ReactiveMongoOperations.java | 2 +- .../common/runtime/MongoOperations.java | 2 +- .../kotlin/PanacheMongoRepositoryBase.kt | 4 +-- .../reactive/ReactivePanacheMongoCompanion.kt | 8 ++--- .../ReactivePanacheMongoEntityBase.kt | 6 ++-- .../ReactivePanacheMongoRepositoryBase.kt | 12 +++---- .../panache/PanacheMongoEntityBase.java | 2 +- .../panache/PanacheMongoRepositoryBase.java | 4 +-- .../ReactivePanacheMongoEntityBase.java | 2 +- .../ReactivePanacheMongoRepositoryBase.java | 4 +-- .../deployment/MongoPanacheRestProcessor.java | 2 +- ...eEntityClassAccessorGenerationVisitor.java | 2 +- .../java/io/quarkus/panache/common/Sort.java | 2 +- .../methods/ListMethodImplementor.java | 2 +- .../methods/hal/ListHalMethodImplementor.java | 2 +- .../rest/data/panache/MethodProperties.java | 2 +- .../PicocliDeploymentConfiguration.java | 2 +- .../quartz/runtime/QuartzScheduler.java | 2 +- .../qute/deployment/QuteProcessor.java | 2 +- .../CheckedTemplateRequireTypeSafeTest.java | 2 +- .../quarkus/qute/runtime/EngineProducer.java | 6 ++-- .../devmode/QuteDevConsoleRecorder.java | 2 +- .../ReactiveDB2DataSourcesHealthCheck.java | 2 +- .../quarkus/vertx/web/cors/package-info.java | 6 ++-- .../io/quarkus/vertx/web/ReactiveRoutes.java | 10 +++--- ...allRyeReactiveTypeConvertersProcessor.java | 2 +- .../client/deployment/DevServicesConfig.java | 2 +- .../deployment/RedisBuildTimeConfig.java | 2 +- .../redis/client/runtime/RedisConfig.java | 4 +-- .../deployment/RestClientProcessor.java | 2 +- .../RestClientOverrideRuntimeConfigTest.java | 2 +- .../ProviderClientRegistrationTest.java | 2 +- .../runtime/QuarkusRestClientBuilder.java | 2 +- .../ClientHttpEngineBuilder43Replacement.java | 2 +- .../deployment/ResteasyCommonProcessor.java | 8 ++--- .../resteasy-links/runtime/pom.xml | 2 +- .../ResteasyServerCommonProcessor.java | 10 +++--- .../runtime/RESTEasyTestHttpProvider.java | 2 +- .../resteasy/test/ReadTimeoutTestCase.java | 4 +-- .../files/StaticFileWithResourcesTest.java | 2 +- .../quarkus/resteasy/runtime/ContextUtil.java | 2 +- .../runtime/ResteasyReactiveConfig.java | 6 ++-- .../ResteasyReactiveJacksonProcessor.java | 2 +- .../reactive/jackson/CustomSerialization.java | 2 +- .../deployment/test/sse/SseParserTest.java | 4 +-- .../deployment/LinksContainerFactory.java | 2 +- .../FormDataOutputMapperGenerator.java | 2 +- .../deployment/ResteasyReactiveProcessor.java | 2 +- .../resource/basic/DefaultMediaTypeTest.java | 2 +- .../reactive/server/test/simple/TestUtil.java | 8 ++--- .../VertxWebSocketParamExtractor.java | 2 +- .../runtime/RestClientReactiveConfig.java | 2 +- .../java/io/quarkus/scheduler/Scheduled.java | 2 +- .../test/metrics/MicrometerTimedTest.java | 4 +-- ...rioRegistryDevServicesBuildTimeConfig.java | 2 +- .../security/jpa/FormAuthJpaTestCase.java | 2 +- .../io/quarkus/security/jpa/Username.java | 2 +- .../webauthn/test/WebAuthnAutomaticTest.java | 2 +- .../webauthn/test/WebAuthnManualTest.java | 2 +- .../webauthn/WebAuthnRunTimeConfig.java | 12 +++---- .../webauthn/WebAuthnUserProvider.java | 10 +++--- .../interceptor/check/RolesAllowedCheck.java | 2 +- .../client/runtime/GraphQLClientsConfig.java | 2 +- .../runtime/SmallRyeGraphQLConfig.java | 12 +++---- .../runtime/SmallRyeGraphQLUIConfig.java | 2 +- .../spi/datafetcher/BlockingHelper.java | 4 +-- .../deployment/HealthBuildTimeConfig.java | 4 +-- .../deployment/SmallRyeHealthUIConfig.java | 2 +- .../test/HealthCheckDefaultScopeTest.java | 2 +- .../HealthCheckProducerDefaultScopeTest.java | 2 +- .../smallrye/health/test/HealthUnitTest.java | 2 +- .../deployment/spi/HealthBuildItem.java | 6 ++-- .../jwt/deployment/SmallRyeJwtProcessor.java | 2 +- .../jwt/test/RequiredClaimsUnitTest.java | 2 +- .../jwt/runtime/auth/SmallRyeJwtConfig.java | 2 +- .../deployment/SmallRyeMetricsProcessor.java | 4 +-- .../metrics/runtime/MetadataHolder.java | 2 +- .../runtime/QuarkusRestMetricsFilter.java | 2 +- .../runtime/SmallRyeMetricsRecorder.java | 2 +- .../deployment/SmallRyeOpenApiProcessor.java | 6 ++-- .../AmqpDevServicesBuildTimeConfig.java | 2 +- ...ReactiveMessagingKafkaBuildTimeConfig.java | 2 +- .../RabbitMQDevServicesBuildTimeConfig.java | 2 +- .../ReactiveMessagingBuildTimeConfig.java | 2 +- .../ReactiveMessagingConfiguration.java | 2 +- .../SpringCloudConfigClientConfig.java | 2 +- .../data/deployment/MethodNameParser.java | 2 +- .../generate/StockMethodsAdder.java | 2 +- .../di/deployment/SpringDIProcessor.java | 6 ++-- .../di/deployment/SpringDIProcessorTest.java | 2 +- .../BeanMethodInvocationGenerator.java | 2 +- .../deployment/SpringSecurityProcessor.java | 10 +++--- .../check/FromBeanRoleSupplier.java | 2 +- ...lNameFromParameterObjectSecurityCheck.java | 2 +- ...incipalNameFromParameterSecurityCheck.java | 2 +- .../swaggerui/deployment/SwaggerUiConfig.java | 12 +++---- .../deployment/SwaggerUiProcessor.java | 2 +- .../deployment/WebXmlParsingBuildStep.java | 2 +- extensions/undertow/runtime/pom.xml | 2 +- .../runtime/QuarkusExceptionHandler.java | 2 +- .../runtime/UndertowDeploymentRecorder.java | 4 +-- .../deployment/VertxGraphqlConfig.java | 2 +- .../devmode/HttpRemoteDevClient.java | 6 ++-- .../console/DevConsoleHttpHandler.java | 2 +- .../devmode/console/DevConsoleProcessor.java | 2 +- .../vertx/http/runtime/AccessLogConfig.java | 2 +- .../vertx/http/runtime/BodyConfig.java | 6 ++-- .../vertx/http/runtime/FormAuthConfig.java | 4 +-- .../http/runtime/PolicyMappingConfig.java | 2 +- .../vertx/http/runtime/ProxyConfig.java | 6 ++-- .../http/runtime/QuarkusErrorHandler.java | 4 +-- .../http/runtime/SameSiteCookieConfig.java | 2 +- .../vertx/http/runtime/VertxHttpRecorder.java | 2 +- .../attribute/ExchangeAttributeParser.java | 2 +- .../devmode/VertxHttpHotReplacementSetup.java | 6 ++-- .../logstream/WebSocketLogHandler.java | 2 +- .../BasicAuthenticationMechanism.java | 2 +- .../security/HttpAuthenticationMechanism.java | 4 +-- .../runtime/security/HttpAuthenticator.java | 2 +- .../security/HttpCredentialTransport.java | 2 +- .../security/HttpSecurityRecorder.java | 2 +- .../http/runtime/security/PathMatcher.java | 2 +- .../core/deployment/VertxCoreProcessor.java | 2 +- .../java/io/quarkus/vertx/ConsumeEvent.java | 2 +- .../vertx/core/runtime/VertxCoreRecorder.java | 8 ++--- .../runtime/config/EventBusConfiguration.java | 6 ++-- .../context/VertxContextSafetyToggle.java | 2 +- .../quarkus/vertx/runtime/VertxProducer.java | 2 +- .../jackson/QuarkusJacksonJsonCodec.java | 6 ++-- 279 files changed, 446 insertions(+), 452 deletions(-) diff --git a/extensions/agroal/deployment/src/main/java/io/quarkus/agroal/deployment/AgroalProcessor.java b/extensions/agroal/deployment/src/main/java/io/quarkus/agroal/deployment/AgroalProcessor.java index 88147a50d750b..51ad9f96a6ec9 100644 --- a/extensions/agroal/deployment/src/main/java/io/quarkus/agroal/deployment/AgroalProcessor.java +++ b/extensions/agroal/deployment/src/main/java/io/quarkus/agroal/deployment/AgroalProcessor.java @@ -231,7 +231,7 @@ void generateDataSourceBeans(AgroalRecorder recorder, .setRuntimeInit() .unremovable() // pass the runtime config into the recorder to ensure that the DataSource related beans - // are created after runtime configuration has been setup + // are created after runtime configuration has been set up .supplier(recorder.agroalDataSourceSupplier(dataSourceName, dataSourcesRuntimeConfig)); if (entry.getValue().isDefault) { diff --git a/extensions/amazon-lambda/common-deployment/src/main/java/io/quarkus/amazon/lambda/deployment/AmazonLambdaCommonProcessor.java b/extensions/amazon-lambda/common-deployment/src/main/java/io/quarkus/amazon/lambda/deployment/AmazonLambdaCommonProcessor.java index eafa77114b062..817273fd6fef1 100644 --- a/extensions/amazon-lambda/common-deployment/src/main/java/io/quarkus/amazon/lambda/deployment/AmazonLambdaCommonProcessor.java +++ b/extensions/amazon-lambda/common-deployment/src/main/java/io/quarkus/amazon/lambda/deployment/AmazonLambdaCommonProcessor.java @@ -42,7 +42,7 @@ void tmpdirs(BuildProducer systemProperty, LaunchModeBuildItem launchModeBuildItem) { LaunchMode mode = launchModeBuildItem.getLaunchMode(); if (mode.isDevOrTest()) { - return; // just in case we're on windows. + return; // just in case we're on Windows. } systemProperty.produce(new SystemPropertyBuildItem("java.io.tmpdir", "/tmp")); systemProperty.produce(new SystemPropertyBuildItem("vertx.cacheDirBase", "/tmp/vertx")); diff --git a/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AbstractLambdaPollLoop.java b/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AbstractLambdaPollLoop.java index d856507bf9526..140e82c3e49a5 100644 --- a/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AbstractLambdaPollLoop.java +++ b/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AbstractLambdaPollLoop.java @@ -172,7 +172,7 @@ public void run() { } catch (Exception ex) { log.error("Failed to report init error (" + launchMode + ")", ex); } finally { - // our main loop is done, time to shutdown + // our main loop is done, time to shut down Application app = Application.currentApplication(); if (app != null) { log.error("Shutting down Quarkus application because of error (" + launchMode + ")"); @@ -274,7 +274,7 @@ protected HttpURLConnection responseStream(URL url) throws IOException { } boolean abortGracefully(Exception ex) { - // if we are running in test mode, or native mode outside of the lambda container, then don't output stack trace for socket errors + // if we are running in test mode, or native mode outside the lambda container, then don't output stack trace for socket errors boolean lambdaEnv = System.getenv("AWS_LAMBDA_RUNTIME_API") != null; boolean testOrDevEnv = LaunchMode.current() == LaunchMode.TEST || LaunchMode.current() == LaunchMode.DEVELOPMENT; diff --git a/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AmazonLambdaApi.java b/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AmazonLambdaApi.java index 5b04cbf30bde5..3220c509567e9 100644 --- a/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AmazonLambdaApi.java +++ b/extensions/amazon-lambda/common-runtime/src/main/java/io/quarkus/amazon/lambda/runtime/AmazonLambdaApi.java @@ -32,7 +32,7 @@ public class AmazonLambdaApi { public static final String API_PATH_ERROR = "/error"; public static final String API_PATH_RESPONSE = "/response"; - // Only available in dev/test mode and points to path for mock even tserver + // Only available in dev/test mode and points to path for mock EventServer public static final String API_BASE_PATH_TEST = "/_lambda_"; public static final String POST_EVENT = API_BASE_PATH_TEST; diff --git a/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/ReflectiveBeanClassBuildItem.java b/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/ReflectiveBeanClassBuildItem.java index f3a1c331a1398..8c49e43c9f446 100644 --- a/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/ReflectiveBeanClassBuildItem.java +++ b/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/ReflectiveBeanClassBuildItem.java @@ -7,7 +7,7 @@ import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; /** - * This build item instructs ArC to produce a {@link ReflectiveClassBuildItem} for a client proxy and intercepred + * This build item instructs ArC to produce a {@link ReflectiveClassBuildItem} for a client proxy and intercepted * subclass generated for the given bean class. */ public final class ReflectiveBeanClassBuildItem extends MultiBuildItem { diff --git a/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/staticmethods/InterceptedStaticMethodsProcessor.java b/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/staticmethods/InterceptedStaticMethodsProcessor.java index 7685913a1a6e6..370682bcf50f2 100644 --- a/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/staticmethods/InterceptedStaticMethodsProcessor.java +++ b/extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/staticmethods/InterceptedStaticMethodsProcessor.java @@ -90,7 +90,7 @@ void collectInterceptedStaticMethods(BeanArchiveIndexBuildItem beanArchiveIndex, InterceptorResolverBuildItem interceptorResolver, TransformedAnnotationsBuildItem transformedAnnotations, BuildProducer unremovableBeans) { - // In this step we collect all intercepted static methods, ie. static methods annotated with interceptor bindings + // In this step we collect all intercepted static methods, i.e. static methods annotated with interceptor bindings Set interceptorBindings = interceptorResolver.getInterceptorBindings(); for (ClassInfo clazz : beanArchiveIndex.getIndex().getKnownClasses()) { diff --git a/extensions/arc/deployment/src/test/java/io/quarkus/arc/test/autoscope/AutoScopeBuildItemTest.java b/extensions/arc/deployment/src/test/java/io/quarkus/arc/test/autoscope/AutoScopeBuildItemTest.java index 9e53e084f0227..d96ceaebe26d2 100644 --- a/extensions/arc/deployment/src/test/java/io/quarkus/arc/test/autoscope/AutoScopeBuildItemTest.java +++ b/extensions/arc/deployment/src/test/java/io/quarkus/arc/test/autoscope/AutoScopeBuildItemTest.java @@ -33,7 +33,7 @@ public void execute(BuildContext context) { return clazz.name().toString().equals(SimpleBean.class.getName()); }).defaultScope(BuiltinScope.DEPENDENT) .scopeAlreadyAdded((scope, reason) -> { - // We cant's pass the state directly to AutoScopeBuildItemTest because it's loaded by a different classloader + // We can't pass the state directly to AutoScopeBuildItemTest because it's loaded by a different classloader Logger.getLogger("AutoScopeBuildItemTest").info(scope + ":" + reason); }).build()); } diff --git a/extensions/arc/runtime/src/main/java/io/quarkus/arc/config/ConfigProperties.java b/extensions/arc/runtime/src/main/java/io/quarkus/arc/config/ConfigProperties.java index bb18437984caf..6774bc3bc6ecd 100644 --- a/extensions/arc/runtime/src/main/java/io/quarkus/arc/config/ConfigProperties.java +++ b/extensions/arc/runtime/src/main/java/io/quarkus/arc/config/ConfigProperties.java @@ -55,7 +55,7 @@ NamingStrategy namingStrategy() default NamingStrategy.FROM_CONFIG; /** - * Whether or not to fail when a non-public field of a class doesn't have a corresponding setter + * Whether to fail when a non-public field of a class doesn't have a corresponding setter */ boolean failOnMismatchingMember() default DEFAULT_FAIL_ON_MISMATCHING_MEMBER; diff --git a/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupIfProperty.java b/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupIfProperty.java index 1e6eb27436ef1..d35ca529779c9 100644 --- a/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupIfProperty.java +++ b/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupIfProperty.java @@ -11,7 +11,7 @@ /** * Indicates that a bean should only be obtained by programmatic lookup if the property matches the provided value. *

- * This annotation is repeatable. A bean will be included if all of the conditions defined by the {@link LookupIfProperty} and + * This annotation is repeatable. A bean will be included if all the conditions defined by the {@link LookupIfProperty} and * {@link LookupUnlessProperty} annotations are satisfied. * *

diff --git a/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupUnlessProperty.java b/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupUnlessProperty.java
index 4daf26db2a1dd..229288f7dba8b 100644
--- a/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupUnlessProperty.java
+++ b/extensions/arc/runtime/src/main/java/io/quarkus/arc/lookup/LookupUnlessProperty.java
@@ -11,7 +11,7 @@
 /**
  * Indicates that a bean should only be obtained by programmatic lookup if the property does not match the provided value.
  * 

- * This annotation is repeatable. A bean will be included if all of the conditions defined by the {@link LookupUnlessProperty} + * This annotation is repeatable. A bean will be included if all the conditions defined by the {@link LookupUnlessProperty} * and {@link LookupIfProperty} annotations are satisfied. * *

diff --git a/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/IfBuildProperty.java b/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/IfBuildProperty.java
index 7b809be7e4390..850c6e85490e7 100644
--- a/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/IfBuildProperty.java
+++ b/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/IfBuildProperty.java
@@ -13,7 +13,7 @@
  * By default, the bean is not enabled when the build time property is not defined at all, but this behavior is configurable
  * via the {#code enableIfMissing} property.
  * 

- * This annotation is repeatable. A bean will only be enabled if all of the conditions defined by the {@link IfBuildProperty} + * This annotation is repeatable. A bean will only be enabled if all the conditions defined by the {@link IfBuildProperty} * annotations are satisfied. */ @Repeatable(IfBuildProperty.List.class) diff --git a/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/UnlessBuildProperty.java b/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/UnlessBuildProperty.java index 97aa116c69c1f..2999c5c1d83ee 100644 --- a/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/UnlessBuildProperty.java +++ b/extensions/arc/runtime/src/main/java/io/quarkus/arc/properties/UnlessBuildProperty.java @@ -13,7 +13,7 @@ * By default, the bean is not enabled when the build time property is not defined at all, but this behavior is configurable * via the {#code enableIfMissing} property. *

- * This annotation is repeatable. A bean will only be enabled if all of the conditions defined by the + * This annotation is repeatable. A bean will only be enabled if all the conditions defined by the * {@link UnlessBuildProperty} annotations are satisfied. */ @Repeatable(UnlessBuildProperty.List.class) diff --git a/extensions/container-image/spi/src/main/java/io/quarkus/container/spi/ContainerImageBuilderBuildItem.java b/extensions/container-image/spi/src/main/java/io/quarkus/container/spi/ContainerImageBuilderBuildItem.java index 645fe50e6ef8f..b2bd6fed023b7 100644 --- a/extensions/container-image/spi/src/main/java/io/quarkus/container/spi/ContainerImageBuilderBuildItem.java +++ b/extensions/container-image/spi/src/main/java/io/quarkus/container/spi/ContainerImageBuilderBuildItem.java @@ -6,7 +6,7 @@ /** * A {@link MultiBuildItem} that represents eligible container image builders. * Some extension have a dependency on external services (e.g. openshift and s2i). - * So, the persennce of the extension alone is not enough to let the build system know that extension is usable. + * So, the presence of the extension alone is not enough to let the build system know that extension is usable. * This build item is produced only when all environment requirements are met. */ public final class ContainerImageBuilderBuildItem extends MultiBuildItem { diff --git a/extensions/datasource/deployment/src/main/java/io/quarkus/datasource/deployment/devservices/DevServicesDatasourceProcessor.java b/extensions/datasource/deployment/src/main/java/io/quarkus/datasource/deployment/devservices/DevServicesDatasourceProcessor.java index 951e655cc28d9..5929968447e30 100644 --- a/extensions/datasource/deployment/src/main/java/io/quarkus/datasource/deployment/devservices/DevServicesDatasourceProcessor.java +++ b/extensions/datasource/deployment/src/main/java/io/quarkus/datasource/deployment/devservices/DevServicesDatasourceProcessor.java @@ -86,7 +86,7 @@ DevServicesDatasourceResultBuildItem launchDatabases(CurateOutcomeBuildItem cura for (RunningDevService database : databases) { devServicesResultBuildItemBuildProducer.produce(database.toBuildItem()); } - // keep the previous behaviour of producing DevServicesDatasourceResultBuildItem only when the devservice first starts. + // keep the previous behaviour of producing DevServicesDatasourceResultBuildItem only when the devservices first starts. return null; } for (Closeable i : databases) { diff --git a/extensions/elasticsearch-rest-client/deployment/src/main/java/io/quarkus/elasticsearch/restclient/lowlevel/deployment/ElasticsearchBuildTimeConfig.java b/extensions/elasticsearch-rest-client/deployment/src/main/java/io/quarkus/elasticsearch/restclient/lowlevel/deployment/ElasticsearchBuildTimeConfig.java index d12b79edc995f..9889677867ad5 100644 --- a/extensions/elasticsearch-rest-client/deployment/src/main/java/io/quarkus/elasticsearch/restclient/lowlevel/deployment/ElasticsearchBuildTimeConfig.java +++ b/extensions/elasticsearch-rest-client/deployment/src/main/java/io/quarkus/elasticsearch/restclient/lowlevel/deployment/ElasticsearchBuildTimeConfig.java @@ -7,7 +7,7 @@ @ConfigRoot(name = "elasticsearch", phase = ConfigPhase.BUILD_TIME) public class ElasticsearchBuildTimeConfig { /** - * Whether or not an health check is published in case the smallrye-health extension is present. + * Whether a health check is published in case the smallrye-health extension is present. */ @ConfigItem(name = "health.enabled", defaultValue = "true") public boolean healthEnabled; diff --git a/extensions/elytron-security-common/deployment/src/main/java/io/quarkus/elytron/security/common/deployment/QuarkusSecurityCommonProcessor.java b/extensions/elytron-security-common/deployment/src/main/java/io/quarkus/elytron/security/common/deployment/QuarkusSecurityCommonProcessor.java index e672358f10c50..9b66baa74c96b 100644 --- a/extensions/elytron-security-common/deployment/src/main/java/io/quarkus/elytron/security/common/deployment/QuarkusSecurityCommonProcessor.java +++ b/extensions/elytron-security-common/deployment/src/main/java/io/quarkus/elytron/security/common/deployment/QuarkusSecurityCommonProcessor.java @@ -23,7 +23,7 @@ public void registerPasswordProvider(ElytronCommonRecorder recorder) { } /** - * Graal VM now seems to loose providers registered at static init + * Graal VM now seems to lose providers registered at static init * * We re-register at runtime (which is a no-op in JVM mode) */ diff --git a/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/BcryptUtil.java b/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/BcryptUtil.java index 84a9f13800574..1c43ea54b2ff1 100644 --- a/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/BcryptUtil.java +++ b/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/BcryptUtil.java @@ -97,7 +97,7 @@ public static String bcryptHash(String password, int iterationCount, byte[] salt * * @param plainText the plain text string to check * @param passwordHash the Modular Crypt Format bcrypt hash to compare against - * @return the boolean result of whether or not the plain text matches the decoded Modular Crypt Format bcrypt hash + * @return the boolean result of whether the plain text matches the decoded Modular Crypt Format bcrypt hash * @throws NullPointerException if the plainText password or passwordHash is null */ public static boolean matches(String plainText, String passwordHash) { diff --git a/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/runtime/ElytronCommonRecorder.java b/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/runtime/ElytronCommonRecorder.java index 46465585c93d5..faaf6149642d6 100644 --- a/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/runtime/ElytronCommonRecorder.java +++ b/extensions/elytron-security-common/runtime/src/main/java/io/quarkus/elytron/security/common/runtime/ElytronCommonRecorder.java @@ -14,7 +14,7 @@ public class ElytronCommonRecorder { public void registerPasswordProvider() { //we don't remove this, as there is no correct place where it can be removed - //as continuous testing can be running along side the dev mode app, but there is + //as continuous testing can be running alongside the dev mode app, but there is //only ever one provider WildFlyElytronPasswordProvider provider = new WildFlyElytronPasswordProvider(); if (Security.getProvider(provider.getName()) == null) { diff --git a/extensions/elytron-security-ldap/deployment/src/main/java/io/quarkus/elytron/security/ldap/deployment/config/LdapSecurityRealmBuildTimeConfig.java b/extensions/elytron-security-ldap/deployment/src/main/java/io/quarkus/elytron/security/ldap/deployment/config/LdapSecurityRealmBuildTimeConfig.java index cde9faf29fa6b..336941bc789ca 100644 --- a/extensions/elytron-security-ldap/deployment/src/main/java/io/quarkus/elytron/security/ldap/deployment/config/LdapSecurityRealmBuildTimeConfig.java +++ b/extensions/elytron-security-ldap/deployment/src/main/java/io/quarkus/elytron/security/ldap/deployment/config/LdapSecurityRealmBuildTimeConfig.java @@ -5,7 +5,7 @@ import io.quarkus.runtime.annotations.ConfigRoot; /** - * A configuration object for a LDAP based realm configuration, + * A configuration object for an LDAP based realm configuration, * {@linkplain org.wildfly.security.auth.realm.ldap.LdapSecurityRealm} */ @ConfigRoot(name = "security.ldap", phase = ConfigPhase.BUILD_TIME) diff --git a/extensions/elytron-security-ldap/runtime/src/main/java/io/quarkus/elytron/security/ldap/config/LdapSecurityRealmRuntimeConfig.java b/extensions/elytron-security-ldap/runtime/src/main/java/io/quarkus/elytron/security/ldap/config/LdapSecurityRealmRuntimeConfig.java index b04cc291d65d3..f1e1e351d5269 100644 --- a/extensions/elytron-security-ldap/runtime/src/main/java/io/quarkus/elytron/security/ldap/config/LdapSecurityRealmRuntimeConfig.java +++ b/extensions/elytron-security-ldap/runtime/src/main/java/io/quarkus/elytron/security/ldap/config/LdapSecurityRealmRuntimeConfig.java @@ -5,7 +5,7 @@ import io.quarkus.runtime.annotations.ConfigRoot; /** - * Runtime configuration object for a LDAP based realm configuration, + * Runtime configuration object for an LDAP based realm configuration, * {@linkplain org.wildfly.security.auth.realm.ldap.LdapSecurityRealm} */ @ConfigRoot(name = "security.ldap", phase = ConfigPhase.RUN_TIME) diff --git a/extensions/elytron-security/deployment/src/main/java/io/quarkus/elytron/security/deployment/SecurityDomainBuildItem.java b/extensions/elytron-security/deployment/src/main/java/io/quarkus/elytron/security/deployment/SecurityDomainBuildItem.java index aabcd560a7315..d4347f6b43a10 100644 --- a/extensions/elytron-security/deployment/src/main/java/io/quarkus/elytron/security/deployment/SecurityDomainBuildItem.java +++ b/extensions/elytron-security/deployment/src/main/java/io/quarkus/elytron/security/deployment/SecurityDomainBuildItem.java @@ -6,7 +6,7 @@ import io.quarkus.runtime.RuntimeValue; /** - * This represent a {@linkplain SecurityDomain} instance output by a build step. + * This represents a {@linkplain SecurityDomain} instance output by a build step. */ public final class SecurityDomainBuildItem extends SimpleBuildItem { diff --git a/extensions/elytron-security/runtime/src/main/java/io/quarkus/elytron/security/runtime/ElytronRecorder.java b/extensions/elytron-security/runtime/src/main/java/io/quarkus/elytron/security/runtime/ElytronRecorder.java index 3c005442adc63..1285e779a7486 100644 --- a/extensions/elytron-security/runtime/src/main/java/io/quarkus/elytron/security/runtime/ElytronRecorder.java +++ b/extensions/elytron-security/runtime/src/main/java/io/quarkus/elytron/security/runtime/ElytronRecorder.java @@ -77,7 +77,7 @@ public boolean implies(Permission permission) { } /** - * Called to add an additional realm to the {@linkplain SecurityDomain} being built + * Called to add a realm to the {@linkplain SecurityDomain} being built * * @param builder - runtime value for SecurityDomain.Builder created by * {@linkplain #configureDomainBuilder(String, RuntimeValue)} diff --git a/extensions/flyway/deployment/src/main/java/io/quarkus/flyway/FlywayProcessor.java b/extensions/flyway/deployment/src/main/java/io/quarkus/flyway/FlywayProcessor.java index d337794dca8cc..f622167746dba 100644 --- a/extensions/flyway/deployment/src/main/java/io/quarkus/flyway/FlywayProcessor.java +++ b/extensions/flyway/deployment/src/main/java/io/quarkus/flyway/FlywayProcessor.java @@ -162,7 +162,7 @@ ServiceStartBuildItem createBeansAndStartActions(FlywayRecorder recorder, // make a FlywayContainerProducer bean additionalBeans.produce(AdditionalBeanBuildItem.builder().addBeanClasses(FlywayContainerProducer.class).setUnremovable() .setDefaultScope(DotNames.SINGLETON).build()); - // add the @FlywayDataSource class otherwise it won't registered as a qualifier + // add the @FlywayDataSource class otherwise it won't be registered as a qualifier additionalBeans.produce(AdditionalBeanBuildItem.builder().addBeanClass(FlywayDataSource.class).build()); recorder.resetFlywayContainers(); diff --git a/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayDevModeTest.java b/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayDevModeTest.java index db139875a57c7..b899cea2a7ac3 100644 --- a/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayDevModeTest.java +++ b/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayDevModeTest.java @@ -12,7 +12,7 @@ /** * Flyway needs a datasource to work. * This tests assures, that an error occurs, - * as soon as the default flyway configuration points to an missing default datasource. + * as soon as the default flyway configuration points to a missing default datasource. */ public class FlywayDevModeTest { diff --git a/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigEmptyTest.java b/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigEmptyTest.java index ac1eaa444fcd8..ad2021699d337 100644 --- a/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigEmptyTest.java +++ b/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigEmptyTest.java @@ -16,7 +16,7 @@ /** * Flyway needs a datasource to work. * This tests assures, that an error occurs, - * as soon as the default flyway configuration points to an missing default datasource. + * as soon as the default flyway configuration points to a missing default datasource. */ public class FlywayExtensionConfigEmptyTest { diff --git a/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigMissingNamedDataSourceTest.java b/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigMissingNamedDataSourceTest.java index 48fa3f519c6e5..c02d987a45335 100644 --- a/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigMissingNamedDataSourceTest.java +++ b/extensions/flyway/deployment/src/test/java/io/quarkus/flyway/test/FlywayExtensionConfigMissingNamedDataSourceTest.java @@ -16,7 +16,7 @@ /** * Flyway needs a datasource to work. - * This tests assures, that an error occurs, as soon as a named flyway configuration points to an missing datasource. + * This tests assures that an error occurs as soon as a named flyway configuration points to a missing datasource. */ public class FlywayExtensionConfigMissingNamedDataSourceTest { diff --git a/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayContainerProducer.java b/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayContainerProducer.java index 03ea79b3ec8fa..b5ddabd5cb530 100644 --- a/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayContainerProducer.java +++ b/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayContainerProducer.java @@ -14,7 +14,7 @@ * and it's {@code createFlyway} method is called at runtime in order to produce * the actual {@code Flyway} objects. * - * CDI scopes and qualifiers are setup at build-time, which is why this class is devoid of + * CDI scopes and qualifiers are set up at build-time, which is why this class is devoid of * any CDI annotations * */ diff --git a/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceBuildTimeConfig.java b/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceBuildTimeConfig.java index 2da069dc480ba..7668f4ca191ea 100644 --- a/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceBuildTimeConfig.java +++ b/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceBuildTimeConfig.java @@ -30,7 +30,7 @@ public final class FlywayDataSourceBuildTimeConfig { /** * Comma-separated list of fully qualified class names of Callback implementations * to use to hook into the Flyway lifecycle. - * The {@link org.flywaydb.core.api.callback.Callback} sub-class must have a no-args constructor and must not be abstract. + * The {@link org.flywaydb.core.api.callback.Callback} subclass must have a no-args constructor and must not be abstract. * These classes must also not have any fields that hold state (unless that state is initialized in the constructor). */ @ConfigItem diff --git a/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceRuntimeConfig.java b/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceRuntimeConfig.java index 35c41221c1b02..1b49e442622c0 100644 --- a/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceRuntimeConfig.java +++ b/extensions/flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceRuntimeConfig.java @@ -53,7 +53,7 @@ public static final FlywayDataSourceRuntimeConfig defaultConfig() { /** * The name of Flyway's schema history table. - * By default (single-schema mode) the schema history table is placed in the default schema for the connection provided by + * By default (single-schema mode), the schema history table is placed in the default schema for the connection provided by * the datasource. * When the flyway.schemas property is set (multi-schema mode), the schema history table is placed in the first schema of * the list. diff --git a/extensions/funqy/funqy-google-cloud-functions/deployment/src/main/java/io/quarkus/funqy/gcp/functions/deployment/bindings/CloudFunctionsDeploymentBuildStep.java b/extensions/funqy/funqy-google-cloud-functions/deployment/src/main/java/io/quarkus/funqy/gcp/functions/deployment/bindings/CloudFunctionsDeploymentBuildStep.java index d68595654ebcf..8ffcc6d0ca0ee 100644 --- a/extensions/funqy/funqy-google-cloud-functions/deployment/src/main/java/io/quarkus/funqy/gcp/functions/deployment/bindings/CloudFunctionsDeploymentBuildStep.java +++ b/extensions/funqy/funqy-google-cloud-functions/deployment/src/main/java/io/quarkus/funqy/gcp/functions/deployment/bindings/CloudFunctionsDeploymentBuildStep.java @@ -24,7 +24,7 @@ public UberJarRequiredBuildItem forceUberJar() { /** * Creates a target/deployment dir and copy the uber jar in it. - * This facilitates the usage of the 'glcoud' command. + * This facilitates the usage of the 'gcloud' command. */ @BuildStep(onlyIf = IsNormal.class, onlyIfNot = NativeBuild.class) public ArtifactResultBuildItem functionDeployment(OutputTargetBuildItem target, JarBuildItem jar) diff --git a/extensions/funqy/funqy-google-cloud-functions/runtime/src/main/java/io/quarkus/funqy/gcp/functions/event/PubsubMessage.java b/extensions/funqy/funqy-google-cloud-functions/runtime/src/main/java/io/quarkus/funqy/gcp/functions/event/PubsubMessage.java index b863c388e18f7..133e578a5b300 100644 --- a/extensions/funqy/funqy-google-cloud-functions/runtime/src/main/java/io/quarkus/funqy/gcp/functions/event/PubsubMessage.java +++ b/extensions/funqy/funqy-google-cloud-functions/runtime/src/main/java/io/quarkus/funqy/gcp/functions/event/PubsubMessage.java @@ -3,7 +3,7 @@ import java.util.Map; /** - * Background function event for Pubsub + * Background function event for Pub/Sub. * * @see PubsubMessage */ diff --git a/extensions/google-cloud-functions-http/deployment/src/main/java/io/quarkus/gcp/functions/http/deployment/CloudFunctionsDeploymentBuildStep.java b/extensions/google-cloud-functions-http/deployment/src/main/java/io/quarkus/gcp/functions/http/deployment/CloudFunctionsDeploymentBuildStep.java index 2c28c2cdc3a39..25dbcf3ce150c 100644 --- a/extensions/google-cloud-functions-http/deployment/src/main/java/io/quarkus/gcp/functions/http/deployment/CloudFunctionsDeploymentBuildStep.java +++ b/extensions/google-cloud-functions-http/deployment/src/main/java/io/quarkus/gcp/functions/http/deployment/CloudFunctionsDeploymentBuildStep.java @@ -23,7 +23,7 @@ public UberJarRequiredBuildItem forceUberJar() { /** * Creates a target/deployment dir and copy the uber jar in it. - * This facilitates the usage of the 'glcoud' command. + * This facilitates the usage of the 'gcloud' command. */ @BuildStep(onlyIf = IsNormal.class, onlyIfNot = NativeBuild.class) public ArtifactResultBuildItem functionDeployment(OutputTargetBuildItem target, JarBuildItem jar) diff --git a/extensions/google-cloud-functions/deployment/src/main/java/io/quarkus/gcp/functions/deployment/CloudFunctionDeploymentBuildStep.java b/extensions/google-cloud-functions/deployment/src/main/java/io/quarkus/gcp/functions/deployment/CloudFunctionDeploymentBuildStep.java index 33a6a141652b9..7e44844e743f8 100644 --- a/extensions/google-cloud-functions/deployment/src/main/java/io/quarkus/gcp/functions/deployment/CloudFunctionDeploymentBuildStep.java +++ b/extensions/google-cloud-functions/deployment/src/main/java/io/quarkus/gcp/functions/deployment/CloudFunctionDeploymentBuildStep.java @@ -31,7 +31,7 @@ void failForNativeSources(BuildProducer artifactResultP /** * Creates a target/deployment dir and copy the uber jar in it. - * This facilitates the usage of the 'glcoud' command. + * This facilitates the usage of the 'gcloud' command. */ @BuildStep(onlyIf = IsNormal.class, onlyIfNot = NativeBuild.class) public ArtifactResultBuildItem functionDeployment(OutputTargetBuildItem target, JarBuildItem jar) diff --git a/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcClientConfiguration.java b/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcClientConfiguration.java index b1652d9dbf5c5..7631ed0c0953d 100644 --- a/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcClientConfiguration.java +++ b/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcClientConfiguration.java @@ -63,7 +63,7 @@ public class GrpcClientConfiguration { public Optional idleTimeout; /** - * The amount of time the sender of of a keep alive ping waits for an acknowledgement. + * The amount of time the sender of a keep alive ping waits for an acknowledgement. */ @ConfigItem public Optional keepAliveTimeout; @@ -147,8 +147,8 @@ public class GrpcClientConfiguration { /** * Use a custom load balancing policy. - * Accepted values are: {@code pick_value}, {@code round_robin}, {@code grpclb} - * This value is ignored if name-resolver is set to 'stork' + * Accepted values are: {@code pick_value}, {@code round_robin}, {@code grpclb}. + * This value is ignored if name-resolver is set to 'stork'. */ @ConfigItem(defaultValue = "pick_first") public String loadBalancingPolicy; diff --git a/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcCodeGenConfig.java b/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcCodeGenConfig.java index 1746ececfdcd0..6b9060a572db5 100644 --- a/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcCodeGenConfig.java +++ b/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcCodeGenConfig.java @@ -33,7 +33,7 @@ public class GrpcCodeGenConfig { *

  • all - scan all dependencies
  • * * - * By default: com.google.protobuf:protobuf-java + * By default, com.google.protobuf:protobuf-java. */ @ConfigItem(defaultValue = "com.google.protobuf:protobuf-java") public String scanForImports; diff --git a/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcServerBuildTimeConfig.java b/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcServerBuildTimeConfig.java index 8b044d8683ea1..a03f3be8a152b 100644 --- a/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcServerBuildTimeConfig.java +++ b/extensions/grpc/runtime/src/main/java/io/quarkus/grpc/runtime/config/GrpcServerBuildTimeConfig.java @@ -7,13 +7,13 @@ @ConfigRoot(phase = ConfigPhase.BUILD_TIME, name = "grpc.server") public class GrpcServerBuildTimeConfig { /** - * Whether or not a health check on gRPC status is published in case the smallrye-health extension is present. + * Whether a health check on gRPC status is published in case the smallrye-health extension is present. */ @ConfigItem(name = "health.enabled", defaultValue = "true") public boolean mpHealthEnabled; /** - * Whether or not the gRPC health check is exposed. + * Whether the gRPC health check is exposed. */ @ConfigItem(name = "grpc-health.enabled", defaultValue = "true") public boolean grpcHealthEnabled; diff --git a/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateEntityEnhancer.java b/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateEntityEnhancer.java index 639b5f4ce261b..bcd85314097e3 100644 --- a/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateEntityEnhancer.java +++ b/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateEntityEnhancer.java @@ -72,7 +72,7 @@ public ClassLoader getLoadingClassLoader() { @Override public void visitEnd() { super.visitEnd(); - final ClassWriter writer = (ClassWriter) this.cv; //safe cast: cv is the the ClassWriter instance we passed to the super constructor + final ClassWriter writer = (ClassWriter) this.cv; //safe cast: cv is the ClassWriter instance we passed to the super constructor //We need to convert the nice Visitor chain into a plain byte array to adapt to the Hibernate ORM //enhancement API: final byte[] inputBytes = writer.toByteArray(); diff --git a/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateOrmConfig.java b/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateOrmConfig.java index a4e5e390499a6..cff9da039e318 100644 --- a/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateOrmConfig.java +++ b/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateOrmConfig.java @@ -52,7 +52,7 @@ public class HibernateOrmConfig { public Optional logSessionMetrics; /** - * Whether or not metrics are published if a metrics extension is enabled. + * Whether metrics are published if a metrics extension is enabled. */ @ConfigItem(name = "metrics.enabled") public boolean metricsEnabled; diff --git a/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/devconsole/MyEntityWithFailingDDLGeneration.java b/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/devconsole/MyEntityWithFailingDDLGeneration.java index 97f3eba7ca57a..f698f9d7eff99 100644 --- a/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/devconsole/MyEntityWithFailingDDLGeneration.java +++ b/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/devconsole/MyEntityWithFailingDDLGeneration.java @@ -20,7 +20,7 @@ public class MyEntityWithFailingDDLGeneration { @GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "myEntitySeq") public Long id; - // The goal of this cusstom type is to trigger an error during DDL generation + // The goal of this custom type is to trigger an error during DDL generation @Type(type = "typeWithUnsupportedSqlCode") public String name; diff --git a/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessFieldTypesTest.java b/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessFieldTypesTest.java index 877eac326e4ec..80435a245ddce 100644 --- a/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessFieldTypesTest.java +++ b/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessFieldTypesTest.java @@ -45,7 +45,7 @@ public class PublicFieldAccessFieldTypesTest { public void testFieldAccess() throws SystemException, NotSupportedException, HeuristicRollbackException, HeuristicMixedException, RollbackException { - // Ideally we'd write a @ParamaterizedTest and pass the delegates as parameters, + // Ideally we'd write a @ParameterizedTest and pass the delegates as parameters, // but we cannot do that due to JUnit using a different classloader than the test. for (FieldAccessEnhancedDelegate delegate : FieldAccessEnhancedDelegate.values()) { doTestFieldAccess(delegate); diff --git a/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessInheritanceTest.java b/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessInheritanceTest.java index f43f146b887ff..a2c6ab66ec783 100644 --- a/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessInheritanceTest.java +++ b/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/publicfields/PublicFieldAccessInheritanceTest.java @@ -46,7 +46,7 @@ public class PublicFieldAccessInheritanceTest { public void testFieldAccess() throws SystemException, NotSupportedException, HeuristicRollbackException, HeuristicMixedException, RollbackException { - // Ideally we'd write a @ParamaterizedTest and pass the delegates as parameters, + // Ideally we'd write a @ParameterizedTest and pass the delegates as parameters, // but we cannot do that due to JUnit using a different classloader than the test. for (FieldAccessEnhancedDelegate delegate : FieldAccessEnhancedDelegate.values()) { doTestFieldAccess(delegate); diff --git a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/JPAConfig.java b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/JPAConfig.java index 705d5ec3c0465..d1186090e1f1b 100644 --- a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/JPAConfig.java +++ b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/JPAConfig.java @@ -98,7 +98,7 @@ public Set getPersistenceUnits() { } /** - * Need to shutdown all instances of Hibernate ORM before the actual destroy event, + * Need to shut down all instances of Hibernate ORM before the actual destroy event, * as it might need to use the datasources during shutdown. * * @param event ignored diff --git a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/FastBootMetadataBuilder.java b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/FastBootMetadataBuilder.java index b4685c525b117..804f0f260bce4 100644 --- a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/FastBootMetadataBuilder.java +++ b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/FastBootMetadataBuilder.java @@ -155,7 +155,7 @@ public FastBootMetadataBuilder(final QuarkusPersistenceUnitDefinition puDefiniti * initializes the EnversService and produces some additional mapping documents. * 4. After that point the EnversService appears to be fully functional. * - * The following trick uses the aforementioned steps to setup the EnversService and then turns it into + * The following trick uses the aforementioned steps to set up the EnversService and then turns it into * a ProvidedService so that it is not necessary to repeat all these complex steps during the reactivation * of the destroyed service registry in PreconfiguredServiceRegistryBuilder. * @@ -239,7 +239,7 @@ private MergedSettings mergeSettings(QuarkusPersistenceUnitDefinition puDefiniti cfg.put("hibernate.temp.use_jdbc_metadata_defaults", "false"); - //This shouldn't be encouraged but sometimes it's really useful - and it used to be the default + //This shouldn't be encouraged, but sometimes it's really useful - and it used to be the default //in Hibernate ORM before the JPA spec would require to change this. //At this time of transitioning we'll only expose it as a global system property, so to allow usage //for special circumstances and yet not encourage this. diff --git a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/QuarkusPersistenceUnitDefinition.java b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/QuarkusPersistenceUnitDefinition.java index 5774438c23ef5..fe82c2d5a8659 100644 --- a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/QuarkusPersistenceUnitDefinition.java +++ b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/QuarkusPersistenceUnitDefinition.java @@ -17,7 +17,7 @@ import io.quarkus.runtime.ObjectSubstitution; /** - * This represent the fully specified configuration of a Persistence Unit, + * This represents the fully specified configuration of a Persistence Unit, * in a format which is compatible with the bytecode recorder. */ public final class QuarkusPersistenceUnitDefinition { diff --git a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/registry/PreconfiguredServiceRegistryBuilder.java b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/registry/PreconfiguredServiceRegistryBuilder.java index 772e3e61345cd..be0f31cc1b3cf 100644 --- a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/registry/PreconfiguredServiceRegistryBuilder.java +++ b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/boot/registry/PreconfiguredServiceRegistryBuilder.java @@ -99,7 +99,7 @@ public StandardServiceRegistryImpl buildNewServiceRegistry() { // Can skip, it's only deprecated stuff: // applyServiceContributingIntegrators( bootstrapServiceRegistry ); - // This is NOT deprecated stuff.. yet they will at best contribute stuff we + // This is NOT deprecated stuff, yet they will at best contribute stuff we // already recorded as part of #applyIntegrator, #addInitiator, #addService // applyServiceContributors( bootstrapServiceRegistry ); diff --git a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/proxies/ProxyDefinitions.java b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/proxies/ProxyDefinitions.java index b44d9cb114afb..7a85c3d2abe90 100644 --- a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/proxies/ProxyDefinitions.java +++ b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/proxies/ProxyDefinitions.java @@ -28,7 +28,7 @@ * class is meant to be created at build time and hold onto those class definitions. * * Implementors of a custom {@link org.hibernate.bytecode.spi.ProxyFactoryFactory} are - * then able to lookup such class definitions at runtime to create new instances of the + * then able to look up such class definitions at runtime to create new instances of the * required enhanced proxies. * * Failure to generate such a proxy is not critical, but it implies that Hibernate ORM @@ -39,7 +39,7 @@ * Default constructors are required beyond proxy generation, so a lack of such a constructor * will have us abort the bootstrap process with a critical error. * On the other hand, having the entities marked as "final" is handled gracefully, as we - * can simply fallback to not use the enhanced proxy for the specific entity, and because + * can simply fall back to not use the enhanced proxy for the specific entity, and because * it's a common case when writing entities in Kotlin. */ public final class ProxyDefinitions { diff --git a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/schema/SchemaManagementIntegrator.java b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/schema/SchemaManagementIntegrator.java index 89045cb129e07..e75f1b40afd4d 100644 --- a/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/schema/SchemaManagementIntegrator.java +++ b/extensions/hibernate-orm/runtime/src/main/java/io/quarkus/hibernate/orm/runtime/schema/SchemaManagementIntegrator.java @@ -167,7 +167,7 @@ public void accept(String command) { public void resetDatabase(String dbName) { String name = datasourceToPuMap.get(dbName); if (name == null) { - //not a hibernate DS + //not an hibernate DS return; } recreateDatabase(name); diff --git a/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessFieldTypesTest.java b/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessFieldTypesTest.java index c562c026f9a8b..d5eefc6786237 100644 --- a/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessFieldTypesTest.java +++ b/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessFieldTypesTest.java @@ -36,7 +36,7 @@ public class PublicFieldAccessFieldTypesTest { @Test @RunOnVertxContext public void testFieldAccess(UniAsserter asserter) { - // Ideally we'd write a @ParamaterizedTest and pass the delegates as parameters, + // Ideally we'd write a @ParameterizedTest and pass the delegates as parameters, // but we cannot do that due to JUnit using a different classloader than the test. for (FieldAccessEnhancedDelegate delegate : FieldAccessEnhancedDelegate.values()) { doTestFieldAccess(delegate, asserter); diff --git a/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessInheritanceTest.java b/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessInheritanceTest.java index d736762a619d4..83c7bb07cf70e 100644 --- a/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessInheritanceTest.java +++ b/extensions/hibernate-reactive/deployment/src/test/java/io/quarkus/hibernate/reactive/publicfields/PublicFieldAccessInheritanceTest.java @@ -37,7 +37,7 @@ public class PublicFieldAccessInheritanceTest { @Test @RunOnVertxContext public void testFieldAccess(final UniAsserter asserter) { - // Ideally we'd write a @ParamaterizedTest and pass the delegates as parameters, + // Ideally we'd write a @ParameterizedTest and pass the delegates as parameters, // but we cannot do that due to JUnit using a different classloader than the test. for (FieldAccessEnhancedDelegate delegate : FieldAccessEnhancedDelegate .values()) { diff --git a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/FastBootHibernateReactivePersistenceProvider.java b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/FastBootHibernateReactivePersistenceProvider.java index 0aad81c4e70c9..2357a59c73c7f 100644 --- a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/FastBootHibernateReactivePersistenceProvider.java +++ b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/FastBootHibernateReactivePersistenceProvider.java @@ -246,7 +246,7 @@ private void registerVertxAndPool(String persistenceUnitName, return; } - // for now we only support one pool but this will change + // for now, we only support one pool but this will change InstanceHandle poolHandle = Arc.container().instance(Pool.class); if (!poolHandle.isAvailable()) { throw new IllegalStateException("No pool has been defined for persistence unit " + persistenceUnitName); diff --git a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/boot/registry/PreconfiguredReactiveServiceRegistryBuilder.java b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/boot/registry/PreconfiguredReactiveServiceRegistryBuilder.java index 75e5755fb7b88..11b203c98e3a1 100644 --- a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/boot/registry/PreconfiguredReactiveServiceRegistryBuilder.java +++ b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/boot/registry/PreconfiguredReactiveServiceRegistryBuilder.java @@ -98,7 +98,7 @@ public StandardServiceRegistryImpl buildNewServiceRegistry() { // Can skip, it's only deprecated stuff: // applyServiceContributingIntegrators( bootstrapServiceRegistry ); - // This is NOT deprecated stuff.. yet they will at best contribute stuff we + // This is NOT deprecated stuff, yet they will at best contribute stuff we // already recorded as part of #applyIntegrator, #addInitiator, #addService // applyServiceContributors( bootstrapServiceRegistry ); diff --git a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/CheckingVertxContext.java b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/CheckingVertxContext.java index 8380cc51efc56..4a45a0a6dd6c2 100644 --- a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/CheckingVertxContext.java +++ b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/CheckingVertxContext.java @@ -8,7 +8,7 @@ * The {@link VertxContext} in Hibernate Reactive is accessing the * Vert.x context directly, assuming this is the correct context as * intended by the developer, as Hibernate Reactive has no opinion in - * regards to how Vert.x is integrated with other components. + * regard to how Vert.x is integrated with other components. * The precise definition of "correct context" will most likely depend * on the runtime model and how other components are integrated with Vert.x; * in particular the lifecycle of the context needs to be specified. diff --git a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/QuarkusReactiveConnectionPoolInitiator.java b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/QuarkusReactiveConnectionPoolInitiator.java index d36b9296c3ad0..6a709b48bae05 100644 --- a/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/QuarkusReactiveConnectionPoolInitiator.java +++ b/extensions/hibernate-reactive/runtime/src/main/java/io/quarkus/hibernate/reactive/runtime/customized/QuarkusReactiveConnectionPoolInitiator.java @@ -65,7 +65,7 @@ protected SqlStatementLogger getSqlStatementLogger() { * Since this Service implementation does not implement @{@link org.hibernate.service.spi.Stoppable} * and we're only adapting an externally provided pool, we will not actually close such provided pool * when Hibernate ORM is shutdown (it doesn't own the lifecycle of this external component). - * Therefore there is no need to wait for its shutdown and this method returns an already + * Therefore, there is no need to wait for its shutdown and this method returns an already * successfully completed CompletionStage. * * @return diff --git a/extensions/infinispan-client/README.MD b/extensions/infinispan-client/README.MD index c9d052455a4e8..759d975aba3ad 100644 --- a/extensions/infinispan-client/README.MD +++ b/extensions/infinispan-client/README.MD @@ -2,7 +2,7 @@ ## Status -The RemoteCache by default only supports byte[] key and values. However a user can plug in a Marshaller to workaround +The RemoteCache by default only supports byte[] key and values. However, a user can plug in a Marshaller to work around that. Please see the next few sections about features that currently work and those that don't ## Things that work @@ -60,7 +60,7 @@ that the programmatic way via @Produces will overwrite any same named files (ie. ### CDI RemoteCache and RemoteCacheManager can both be injected via @Inject. The former allows named cache injection by -adding the Remote annotation, specifying the name of the cache. Currently the RemoteCacheManager is configured via the +adding the Remote annotation, specifying the name of the cache. Currently, the RemoteCacheManager is configured via the hotrod-client.properties and/or microprofile-config.properties (with micro profile config replacing properties if both are present). diff --git a/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java b/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java index 3404733cc1ac4..0e0f4625866f3 100644 --- a/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java +++ b/extensions/infinispan-client/deployment/src/main/java/io/quarkus/infinispan/client/deployment/InfinispanClientProcessor.java @@ -121,7 +121,7 @@ InfinispanPropertiesBuildItem setup(ApplicationArchivesBuildItem applicationArch IndexView index = applicationIndexBuildItem.getIndex(); - // This is always non null + // This is always non-null Object marshaller = properties.get(ConfigurationProperties.MARSHALLER); if (marshaller instanceof ProtoStreamMarshaller) { diff --git a/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientProducer.java b/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientProducer.java index c7795db2d17aa..2f84f57f21c88 100644 --- a/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientProducer.java +++ b/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientProducer.java @@ -256,7 +256,7 @@ private static void handleProtoStreamMarshaller(ProtoStreamMarshaller marshaller FileDescriptorSource fds = (FileDescriptorSource) beanManager.getReference(bean, FileDescriptorSource.class, ctx); serializationContext.registerProtoFiles(fds); - // Register all of the fds so they can be queried + // Register all the fds so they can be queried for (Map.Entry fdEntry : fds.getFileDescriptors().entrySet()) { properties.put(PROTOBUF_FILE_PREFIX + fdEntry.getKey(), new String(fdEntry.getValue())); } diff --git a/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientRuntimeConfig.java b/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientRuntimeConfig.java index 147fedb7ad3d2..2e52778ac9fbf 100644 --- a/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientRuntimeConfig.java +++ b/extensions/infinispan-client/runtime/src/main/java/io/quarkus/infinispan/client/runtime/InfinispanClientRuntimeConfig.java @@ -36,7 +36,7 @@ public class InfinispanClientRuntimeConfig { /** * Sets client intelligence used by authentication * Available values: - * * `BASIC` - Means that the client doesn't handle server topology changes and therefore will only used the list + * * `BASIC` - Means that the client doesn't handle server topology changes and therefore will only use the list * of servers supplied at configuration time. * * `TOPOLOGY_AWARE` - Use this provider if you don't want the client to present any certificates to the * remote TLS host. @@ -50,7 +50,7 @@ public class InfinispanClientRuntimeConfig { // @formatter:off /** - * Enables or disables authentication. Set it to false when connecting to a Infinispan Server without authentication. + * Enables or disables authentication. Set it to false when connecting to an Infinispan Server without authentication. * deployments. Default is 'true'. */ // @formatter:on @@ -58,7 +58,7 @@ public class InfinispanClientRuntimeConfig { Optional useAuth; /** - * Sets user name used by authentication. + * Sets username used by authentication. */ @ConfigItem Optional authUsername; diff --git a/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerBuildTimeConfig.java b/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerBuildTimeConfig.java index 62b66a53d21f3..e63255fc5d5c2 100644 --- a/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerBuildTimeConfig.java +++ b/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerBuildTimeConfig.java @@ -15,7 +15,7 @@ public class JaegerBuildTimeConfig { public boolean enabled; /** - * Whether or not metrics are published in case a metrics extension is present. + * Whether metrics are published in case a metrics extension is present. */ @ConfigItem(name = "metrics.enabled", defaultValue = "false") public boolean metricsEnabled; diff --git a/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerConfig.java b/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerConfig.java index 724c968ec52b9..8982c215804ae 100644 --- a/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerConfig.java +++ b/extensions/jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/JaegerConfig.java @@ -119,7 +119,7 @@ public class JaegerConfig { public Boolean logTraceContext; /** - * Whether or not the registration of tracer as the global tracer should be disabled. + * Whether the registration of tracer as the global tracer should be disabled. * This setting should only be turned on in tests that need to install a mock tracer. */ @ConfigItem(defaultValue = "false") diff --git a/extensions/jdbc/jdbc-derby/deployment/src/test/java/io/quarkus/jdbc/derby/deployment/DevServicesDerbyDatasourceTestCase.java b/extensions/jdbc/jdbc-derby/deployment/src/test/java/io/quarkus/jdbc/derby/deployment/DevServicesDerbyDatasourceTestCase.java index cefdaad778cb0..c9848ae168f78 100644 --- a/extensions/jdbc/jdbc-derby/deployment/src/test/java/io/quarkus/jdbc/derby/deployment/DevServicesDerbyDatasourceTestCase.java +++ b/extensions/jdbc/jdbc-derby/deployment/src/test/java/io/quarkus/jdbc/derby/deployment/DevServicesDerbyDatasourceTestCase.java @@ -44,7 +44,7 @@ public void testDatasource() throws Exception { try { configuration = dataSource.getConfiguration().connectionPoolConfiguration(); } catch (NullPointerException e) { - // we catch the NPE here as we have a proxycd and we can't test dataSource directly + // we catch the NPE here as we have a proxycd and we can't test dataSource directly fail("Datasource should not be null"); } assertTrue(configuration.connectionFactoryConfiguration().jdbcUrl().contains("jdbc:derby:")); diff --git a/extensions/jdbc/jdbc-mariadb/deployment/src/test/java/io/quarkus/jdbc/mariadb/deployment/DevServicesMariaDBDatasourceTestCase.java b/extensions/jdbc/jdbc-mariadb/deployment/src/test/java/io/quarkus/jdbc/mariadb/deployment/DevServicesMariaDBDatasourceTestCase.java index cf871a848903a..7dc1d6265b979 100644 --- a/extensions/jdbc/jdbc-mariadb/deployment/src/test/java/io/quarkus/jdbc/mariadb/deployment/DevServicesMariaDBDatasourceTestCase.java +++ b/extensions/jdbc/jdbc-mariadb/deployment/src/test/java/io/quarkus/jdbc/mariadb/deployment/DevServicesMariaDBDatasourceTestCase.java @@ -44,7 +44,7 @@ public void testDatasource() throws Exception { try { configuration = dataSource.getConfiguration().connectionPoolConfiguration(); } catch (NullPointerException e) { - // we catch the NPE here as we have a proxycd and we can't test dataSource directly + // we catch the NPE here as we have a proxycd and we can't test dataSource directly fail("Datasource should not be null"); } assertTrue(configuration.connectionFactoryConfiguration().jdbcUrl().contains("jdbc:mariadb:")); diff --git a/extensions/jdbc/jdbc-mssql/runtime/pom.xml b/extensions/jdbc/jdbc-mssql/runtime/pom.xml index e762fe270a6fd..bac87d358b766 100644 --- a/extensions/jdbc/jdbc-mssql/runtime/pom.xml +++ b/extensions/jdbc/jdbc-mssql/runtime/pom.xml @@ -28,7 +28,7 @@ mssql-jdbc com.microsoft.azure diff --git a/extensions/jdbc/jdbc-mysql/deployment/src/test/java/io/quarkus/jdbc/mysql/deployment/DevServicesMySQLDatasourceTestCase.java b/extensions/jdbc/jdbc-mysql/deployment/src/test/java/io/quarkus/jdbc/mysql/deployment/DevServicesMySQLDatasourceTestCase.java index 11eb452ffedb8..d3e3cd9834e72 100644 --- a/extensions/jdbc/jdbc-mysql/deployment/src/test/java/io/quarkus/jdbc/mysql/deployment/DevServicesMySQLDatasourceTestCase.java +++ b/extensions/jdbc/jdbc-mysql/deployment/src/test/java/io/quarkus/jdbc/mysql/deployment/DevServicesMySQLDatasourceTestCase.java @@ -44,7 +44,7 @@ public void testDatasource() throws Exception { try { configuration = dataSource.getConfiguration().connectionPoolConfiguration(); } catch (NullPointerException e) { - // we catch the NPE here as we have a proxycd and we can't test dataSource directly + // we catch the NPE here as we have a proxycd and we can't test dataSource directly fail("Datasource should not be null"); } assertTrue(configuration.connectionFactoryConfiguration().jdbcUrl().contains("jdbc:mysql:")); diff --git a/extensions/jdbc/jdbc-oracle/deployment/src/main/java/io/quarkus/jdbc/oracle/deployment/OracleMetadataOverrides.java b/extensions/jdbc/jdbc-oracle/deployment/src/main/java/io/quarkus/jdbc/oracle/deployment/OracleMetadataOverrides.java index 1ca65afd34c76..632df697aec2c 100644 --- a/extensions/jdbc/jdbc-oracle/deployment/src/main/java/io/quarkus/jdbc/oracle/deployment/OracleMetadataOverrides.java +++ b/extensions/jdbc/jdbc-oracle/deployment/src/main/java/io/quarkus/jdbc/oracle/deployment/OracleMetadataOverrides.java @@ -40,7 +40,7 @@ public final class OracleMetadataOverrides { /** * Should match the contents of {@literal reflect-config.json} * - * @param reflectiveClass builItem producer + * @param reflectiveClass buildItem producer */ @BuildStep void build(BuildProducer reflectiveClass) { @@ -75,7 +75,7 @@ void runtimeInitializeDriver(BuildProducer run // runtimeInitialized.produce(new RuntimeInitializedClassBuildItem("oracle.jdbc.driver.OracleDriver")); // The Oracle driver's metadata hints to require java.sql.DriverManager to be initialized at runtime, but: - // A) I disagree with the fact that a driver makes changes outside of its scope (java.sql in this case) + // A) I disagree with the fact that a driver makes changes outside its scope (java.sql in this case) // B) It does actually not compile if you have other JDBC drivers, as other implementations need this class initialized during build // C) This metadata is expected to get improved in the next public release of the Oracle JDBC driver // runtimeInitialized.produce(new RuntimeInitializedClassBuildItem("java.sql.DriverManager")); diff --git a/extensions/jdbc/jdbc-oracle/deployment/src/test/java/io/quarkus/jdbc/oracle/deployment/DevServicesOracleDatasourceTestCase.java b/extensions/jdbc/jdbc-oracle/deployment/src/test/java/io/quarkus/jdbc/oracle/deployment/DevServicesOracleDatasourceTestCase.java index 6fa5878ccb134..e2c59af38e454 100644 --- a/extensions/jdbc/jdbc-oracle/deployment/src/test/java/io/quarkus/jdbc/oracle/deployment/DevServicesOracleDatasourceTestCase.java +++ b/extensions/jdbc/jdbc-oracle/deployment/src/test/java/io/quarkus/jdbc/oracle/deployment/DevServicesOracleDatasourceTestCase.java @@ -44,7 +44,7 @@ public void testDatasource() throws Exception { try { configuration = dataSource.getConfiguration().connectionPoolConfiguration(); } catch (NullPointerException e) { - // we catch the NPE here as we have a proxycd and we can't test dataSource directly + // we catch the NPE here as we have a proxycd and we can't test dataSource directly fail("Datasource should not be null"); } assertTrue(configuration.connectionFactoryConfiguration().jdbcUrl().contains("jdbc:oracle:")); diff --git a/extensions/jdbc/jdbc-postgresql/deployment/src/test/java/io/quarkus/jdbc/postgresql/deployment/DevServicesPostgresqlDatasourceTestCase.java b/extensions/jdbc/jdbc-postgresql/deployment/src/test/java/io/quarkus/jdbc/postgresql/deployment/DevServicesPostgresqlDatasourceTestCase.java index 3041f45030ec0..a013ea3714e3d 100644 --- a/extensions/jdbc/jdbc-postgresql/deployment/src/test/java/io/quarkus/jdbc/postgresql/deployment/DevServicesPostgresqlDatasourceTestCase.java +++ b/extensions/jdbc/jdbc-postgresql/deployment/src/test/java/io/quarkus/jdbc/postgresql/deployment/DevServicesPostgresqlDatasourceTestCase.java @@ -44,7 +44,7 @@ public void testDatasource() throws Exception { try { configuration = dataSource.getConfiguration().connectionPoolConfiguration(); } catch (NullPointerException e) { - // we catch the NPE here as we have a proxycd and we can't test dataSource directly + // we catch the NPE here as we have a proxycd and we can't test dataSource directly fail("Datasource should not be null"); } assertTrue(configuration.connectionFactoryConfiguration().jdbcUrl().contains("jdbc:postgresql:")); diff --git a/extensions/jdbc/jdbc-postgresql/runtime/src/main/java/io/quarkus/jdbc/postgresql/runtime/graal/PgSQLXML.java b/extensions/jdbc/jdbc-postgresql/runtime/src/main/java/io/quarkus/jdbc/postgresql/runtime/graal/PgSQLXML.java index b0333866f2b75..5a1890b6dd771 100644 --- a/extensions/jdbc/jdbc-postgresql/runtime/src/main/java/io/quarkus/jdbc/postgresql/runtime/graal/PgSQLXML.java +++ b/extensions/jdbc/jdbc-postgresql/runtime/src/main/java/io/quarkus/jdbc/postgresql/runtime/graal/PgSQLXML.java @@ -54,9 +54,9 @@ * triggered via {@see java.sql.PreparedStatement#setObject}, * which is widely used for other purposes as well. * - * Considering that the efficiency costs of including all of + * Considering that the efficiency costs of including all * resources and reflective registrations necessary to support the - * XML parsers is non trivial, we apply here a trick to help the + * XML parsers is non-trivial, we apply here a trick to help the * GraalVM native image compiler avoid this inclusion unless this * feature is actually is reachable, by isolating the construction of * the parsers and their supporting other types via a reflective call. @@ -309,7 +309,7 @@ private void ensureInitialized() throws SQLException { } } else if (stringWriter != null) { // This is also handling the work for Stream, SAX, and StAX Results - // as they will use the same underlying stringwriter variable. + // as they will use the same underlying stringWriter variable. // data = stringWriter.toString(); stringWriter = null; diff --git a/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaBuildTimeConfig.java b/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaBuildTimeConfig.java index 2ef777a7c5ad1..b975b460e869b 100644 --- a/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaBuildTimeConfig.java +++ b/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaBuildTimeConfig.java @@ -7,7 +7,7 @@ @ConfigRoot(name = "kafka", phase = ConfigPhase.BUILD_TIME) public class KafkaBuildTimeConfig { /** - * Whether or not an health check is published in case the smallrye-health extension is present. + * Whether a health check is published in case the smallrye-health extension is present. *

    * If you enable the health check, you must specify the `kafka.bootstrap.servers` property. */ @@ -15,7 +15,7 @@ public class KafkaBuildTimeConfig { public boolean healthEnabled; /** - * Whether or not to enable Snappy in native mode. + * Whether to enable Snappy in native mode. *

    * Note that Snappy requires GraalVM 21+ and embeds a native library in the native executable. * This library is unpacked and loaded when the application starts. diff --git a/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaDevServicesBuildTimeConfig.java b/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaDevServicesBuildTimeConfig.java index 937bb5a87e90e..af79c55869f7b 100644 --- a/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaDevServicesBuildTimeConfig.java +++ b/extensions/kafka-client/deployment/src/main/java/io/quarkus/kafka/client/deployment/KafkaDevServicesBuildTimeConfig.java @@ -67,7 +67,7 @@ public class KafkaDevServicesBuildTimeConfig { * This property is used when {@code shared} is set to {@code true}. * In this case, before starting a container, Dev Services for Kafka looks for a container with the * {@code quarkus-dev-service-kafka} label - * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise it + * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise, it * starts a new container with the {@code quarkus-dev-service-kafka} label set to the specified value. *

    * This property is used when you need multiple shared Kafka brokers. diff --git a/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsBuildTimeConfig.java b/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsBuildTimeConfig.java index f3aeaa8fc76d7..ef449bef60cfd 100644 --- a/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsBuildTimeConfig.java +++ b/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsBuildTimeConfig.java @@ -8,7 +8,7 @@ public class KafkaStreamsBuildTimeConfig { /** - * Whether or not a health check is published in case the smallrye-health extension is present (defaults to true). + * Whether a health check is published in case the smallrye-health extension is present (defaults to true). */ @ConfigItem(name = "health.enabled", defaultValue = "true") public boolean healthEnabled; diff --git a/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsProcessor.java b/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsProcessor.java index 00b6d746a29b9..fef5d141a804a 100644 --- a/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsProcessor.java +++ b/extensions/kafka-streams/deployment/src/main/java/io/quarkus/kafka/streams/deployment/KafkaStreamsProcessor.java @@ -161,7 +161,7 @@ void processBuildTimeConfig(KafkaStreamsRecorder recorder, LaunchModeBuildItem l .supplier(recorder.kafkaStreamsSupportSupplier(kafkaStreamsProperties)) .done()); - // make the producer an unremoveable bean + // make the producer an unremovable bean additionalBeans .produce(AdditionalBeanBuildItem.builder().addBeanClasses(KafkaStreamsProducer.class).setUnremovable().build()); } diff --git a/extensions/keycloak-authorization/runtime/src/main/java/io/quarkus/keycloak/pep/runtime/KeycloakPolicyEnforcerTenantConfig.java b/extensions/keycloak-authorization/runtime/src/main/java/io/quarkus/keycloak/pep/runtime/KeycloakPolicyEnforcerTenantConfig.java index ba5e5e74e7937..bacdca9a5f291 100644 --- a/extensions/keycloak-authorization/runtime/src/main/java/io/quarkus/keycloak/pep/runtime/KeycloakPolicyEnforcerTenantConfig.java +++ b/extensions/keycloak-authorization/runtime/src/main/java/io/quarkus/keycloak/pep/runtime/KeycloakPolicyEnforcerTenantConfig.java @@ -67,7 +67,7 @@ public static class KeycloakConfigPolicyEnforcer { /** * Specifies how scopes should be mapped to HTTP methods. If set to true, the policy enforcer will use the HTTP method * from - * the current request to check whether or not access should be granted + * the current request to check whether access should be granted */ @ConfigItem public boolean httpMethodAsScope; diff --git a/extensions/kotlin/deployment/src/main/java/io/quarkus/kotlin/deployment/KotlinProcessor.java b/extensions/kotlin/deployment/src/main/java/io/quarkus/kotlin/deployment/KotlinProcessor.java index e031aaa262d1c..32114a9d8adc3 100644 --- a/extensions/kotlin/deployment/src/main/java/io/quarkus/kotlin/deployment/KotlinProcessor.java +++ b/extensions/kotlin/deployment/src/main/java/io/quarkus/kotlin/deployment/KotlinProcessor.java @@ -37,7 +37,7 @@ void registerKotlinJacksonModule(BuildProducer /** * Kotlin data classes that have multiple constructors need to have their final fields writable, - * otherwise creating a instance of them with default values, fails in native mode + * otherwise creating an instance of them with default values fails in native mode. */ @BuildStep ReflectiveClassFinalFieldsWritablePredicateBuildItem dataClassPredicate() { diff --git a/extensions/kubernetes-client/runtime-internal/src/main/java/io/quarkus/kubernetes/client/runtime/KubernetesClientBuildConfig.java b/extensions/kubernetes-client/runtime-internal/src/main/java/io/quarkus/kubernetes/client/runtime/KubernetesClientBuildConfig.java index 48a5cee31b3cc..9b8da5f8ae55b 100644 --- a/extensions/kubernetes-client/runtime-internal/src/main/java/io/quarkus/kubernetes/client/runtime/KubernetesClientBuildConfig.java +++ b/extensions/kubernetes-client/runtime-internal/src/main/java/io/quarkus/kubernetes/client/runtime/KubernetesClientBuildConfig.java @@ -11,7 +11,7 @@ public class KubernetesClientBuildConfig { /** - * Whether or not the client should trust a self signed certificate if so presented by the API server + * Whether the client should trust a self-signed certificate if so presented by the API server */ @ConfigItem public Optional trustCerts = Optional.empty(); diff --git a/extensions/kubernetes-config/deployment/src/main/java/io/quarkus/kubernetes/config/deployment/KubernetesConfigProcessor.java b/extensions/kubernetes-config/deployment/src/main/java/io/quarkus/kubernetes/config/deployment/KubernetesConfigProcessor.java index 7b1ebb0aff3ad..7a86348c5bb36 100644 --- a/extensions/kubernetes-config/deployment/src/main/java/io/quarkus/kubernetes/config/deployment/KubernetesConfigProcessor.java +++ b/extensions/kubernetes-config/deployment/src/main/java/io/quarkus/kubernetes/config/deployment/KubernetesConfigProcessor.java @@ -51,7 +51,7 @@ public void handleAccessToSecrets(KubernetesConfigSourceConfig config, } // done in order to ensure that http logs aren't shown by default which happens because of the interplay between - // not yet setup logging (as the bootstrap config runs before logging is setup) and the configuration + // not yet setup logging (as the bootstrap config runs before logging is set up) and the configuration // of the okhttp3.logging.HttpLoggingInterceptor by io.fabric8.kubernetes.client.utils.HttpClientUtils @BuildStep public void produceLoggingCategories(BuildProducer categories) { diff --git a/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/AbstractKubernetesConfigSourceUtil.java b/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/AbstractKubernetesConfigSourceUtil.java index 54c291333a995..e3dd7fb96d59f 100644 --- a/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/AbstractKubernetesConfigSourceUtil.java +++ b/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/AbstractKubernetesConfigSourceUtil.java @@ -92,9 +92,9 @@ List toConfigSources(ObjectMeta metadata, Map kube private int getOrdinal(int ordinalOffset) { final OrdinalData ordinalData = ordinalData(); /* - * We don't want a large list of sources to cause an "overflow" into an Ordinal of another ConfigSource + * We don't want a large list of sources to cause an "overflow" into an Ordinal of another ConfigSource, * so we just let the last ones all use the max ordinal - * this is not fool proof, but it's very unlikely that an application will need to define + * this is not foolproof, but it's very unlikely that an application will need to define * a list with 10+ sources... */ return Math.min(ordinalData.getBase() + ordinalOffset, ordinalData.getMax()); diff --git a/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigBuildTimeConfig.java b/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigBuildTimeConfig.java index 79c5b6b5576c0..cfe9f11d85a73 100644 --- a/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigBuildTimeConfig.java +++ b/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigBuildTimeConfig.java @@ -7,7 +7,7 @@ @ConfigRoot(name = "kubernetes-config", phase = ConfigPhase.BUILD_TIME) public class KubernetesConfigBuildTimeConfig { /** - * Whether or not configuration can be read from secrets. + * Whether configuration can be read from secrets. * If set to {@code true}, Kubernetes resources allowing access to secrets (role and role binding) will be generated. */ @ConfigItem(name = "secrets.enabled", defaultValue = "false") diff --git a/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigSourceConfig.java b/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigSourceConfig.java index a8931756eb06d..015ced56a7c62 100644 --- a/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigSourceConfig.java +++ b/extensions/kubernetes-config/runtime/src/main/java/io/quarkus/kubernetes/config/runtime/KubernetesConfigSourceConfig.java @@ -26,7 +26,7 @@ public class KubernetesConfigSourceConfig { * ConfigMaps to look for in the namespace that the Kubernetes Client has been configured for. * ConfigMaps defined later in this list have a higher priority that ConfigMaps defined earlier * in this list. - * Furthermore any Secrets defined in {@code secrets}, will have higher priorities than all ConfigMaps. + * Furthermore, any Secrets defined in {@code secrets}, will have higher priorities than all ConfigMaps. */ @ConfigItem public Optional> configMaps; @@ -36,7 +36,7 @@ public class KubernetesConfigSourceConfig { * If you use this, you probably want to enable {@code quarkus.kubernetes-config.secrets.enabled}. * Secrets defined later in this list have a higher priority that ConfigMaps defined earlier * in this list. - * Furthermore these Secrets have a higher priorities than all ConfigMaps defined in {@code configMaps}. + * Furthermore, these Secrets have a higher priorities than all ConfigMaps defined in {@code configMaps}. */ @ConfigItem public Optional> secrets; diff --git a/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/KubernetesServiceBindingConfig.java b/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/KubernetesServiceBindingConfig.java index 7d8a979caa75a..6095d1d14f238 100644 --- a/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/KubernetesServiceBindingConfig.java +++ b/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/KubernetesServiceBindingConfig.java @@ -12,7 +12,7 @@ public class KubernetesServiceBindingConfig { /** * A list of explicitly configured services. - * The configured value will be used in order to generate the `ServiceBindig` resource (in case that kubernetes resource + * The configured value will be used in order to generate the `ServiceBinding` resource (in case that kubernetes resource * generation is enabled). */ @ConfigItem @@ -26,7 +26,7 @@ public class KubernetesServiceBindingConfig { /** * Determines if binding should be created as files or env vars. - * Set this value to false to bind as env vars.` + * Set this value to {@code false} to bind as env vars.` */ @ConfigItem(defaultValue = "true") public Boolean bindAsFiles; diff --git a/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/ServiceBindingProcessor.java b/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/ServiceBindingProcessor.java index 62e91744d8c0b..4385baada90b6 100644 --- a/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/ServiceBindingProcessor.java +++ b/extensions/kubernetes-service-binding/deployment/src/main/java/io/quarkus/kubernetes/service/binding/buildtime/ServiceBindingProcessor.java @@ -36,16 +36,16 @@ * * Notes: * - * The following pairs are equivallent: + * The following pairs are equivalent: * - * quarkus.kubernetes-service-binding.services.prostresql-default.xxx=yyy - * quarkus.kubernetes-service-binding.services.prostresql.xxx=yyy + * quarkus.kubernetes-service-binding.services.postgresql-default.xxx=yyy + * quarkus.kubernetes-service-binding.services.postgresql.xxx=yyy * - * quarkus.kubernetes-service-binding.services.prostresql-persondb.xxx=yyy + * quarkus.kubernetes-service-binding.services.postgresql-persondb.xxx=yyy * quarkus.kubernetes-service-binding.services.persondb.xxx=yyy * * When service are auto bound the minimal from (e.g. persondb) will be used. - * Users are still able to to tune things using the [kind]-[name] combo to avoid naming clashes. + * Users are still able to tune things using the [kind]-[name] combo to avoid naming clashes. * */ public class ServiceBindingProcessor { @@ -115,7 +115,7 @@ public List createServiceBindingDecorators(KubernetesService * @param applicationName The name of the application. * @param serviceId The key of the service. * @param config The config instance. - * @return The servicebinding requirement that corresponds to the matching entry, or empty if no service with serviceId was + * @return The service binding requirement that corresponds to the matching entry, or empty if no service with serviceId was * found in config. */ protected static Optional createRequirementFromConfig(String applicationName, @@ -136,7 +136,7 @@ protected static Optional createRequirementF * @param serviceId The key of the service. * @param resourceName The name of the target resource. * @param config The config instance. - * @return The servicebinding requirement that corresponds to the matching entry, or empty if no service with serviceId was + * @return The service binding requirement that corresponds to the matching entry, or empty if no service with serviceId was * found in config. */ protected static Optional createRequirementFromConfig(String applicationName, @@ -162,7 +162,7 @@ protected static Optional createRequirementF * @param applicationName The name of the application. * @param config The config instance. * @param qualifier The qualifier that will be converted to a requirement. - * @return The servicebinding requirement that corresponds to the config entry that matches the qualifier, or the defaults + * @return The service binding requirement that corresponds to the config entry that matches the qualifier, or the defaults * for the qualifier. Returns empty if none of the above was found. */ protected static Optional createRequirementFromQualifier(String applicationName, diff --git a/extensions/kubernetes/spi/src/main/java/io/quarkus/kubernetes/spi/KubernetesOptionalResourceDefinitionBuildItem.java b/extensions/kubernetes/spi/src/main/java/io/quarkus/kubernetes/spi/KubernetesOptionalResourceDefinitionBuildItem.java index f3cc5cee883b8..738764996d789 100644 --- a/extensions/kubernetes/spi/src/main/java/io/quarkus/kubernetes/spi/KubernetesOptionalResourceDefinitionBuildItem.java +++ b/extensions/kubernetes/spi/src/main/java/io/quarkus/kubernetes/spi/KubernetesOptionalResourceDefinitionBuildItem.java @@ -5,7 +5,7 @@ /** * A build item that extension can use to mark a specific resource definition as optional. - * A an optional resource, when fails to get deployed, will not cause the entire deployment process to fail, + * An optional resource, when fails to get deployed, will not cause the entire deployment process to fail, * but will log a warning instead. */ public final class KubernetesOptionalResourceDefinitionBuildItem extends MultiBuildItem { diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/AutoScalingConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/AutoScalingConfig.java index 7a6b9b1633f80..303eb62c0a330 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/AutoScalingConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/AutoScalingConfig.java @@ -37,7 +37,7 @@ public class AutoScalingConfig { * The exact amount of requests allowed to the replica at a time. * Its default value is “0”, which means an unlimited number of requests are allowed to flow into the replica. * - * @return the container concurrenct or zero if its not bound. + * @return the container concurrency, or zero if it is not bound. */ Optional containerConcurrency; diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GitRepoVolumeConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GitRepoVolumeConfig.java index cc1663a01617f..40a567eba7ef3 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GitRepoVolumeConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GitRepoVolumeConfig.java @@ -10,7 +10,7 @@ public class GitRepoVolumeConfig { /** - * Git repoistory URL. + * Git repository URL. */ @ConfigItem String repository; diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GlobalAutoScalingConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GlobalAutoScalingConfig.java index 499b94f421351..1ad74874d6a0b 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GlobalAutoScalingConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/GlobalAutoScalingConfig.java @@ -23,7 +23,7 @@ public class GlobalAutoScalingConfig { * * @see Knative Knative: Configuring * concurrency: Hard Limit - * @return the container concurrenct or zero if its not bound. + * @return the container concurrency, or zero if it is not bound. */ Optional containerConcurrency; diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeConfig.java index cab1095a06df9..e791b0f5c45de 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeConfig.java @@ -56,7 +56,7 @@ public class KnativeConfig implements PlatformConfiguration { Map annotations; /** - * Whether or not to add the build timestamp to the Kubernetes annotations + * Whether to add the build timestamp to the Kubernetes annotations * This is a very useful way to have manifests of successive builds of the same * application differ - thus ensuring that Kubernetes will apply the updated resources */ @@ -386,7 +386,7 @@ public EnvVarsConfig getEnv() { } /** - * Whether or not this service is cluster-local. + * Whether this service is cluster-local. * Cluster local services are not exposed to the outside world. * More information in this link. */ @@ -395,7 +395,7 @@ public EnvVarsConfig getEnv() { /** * This value controls the minimum number of replicas each revision should have. - * Knative will attempt to never have less than this number of replicas at any one point in time. + * Knative will attempt to never have less than this number of replicas at any point in time. */ @ConfigItem Optional minScale; @@ -403,7 +403,7 @@ public EnvVarsConfig getEnv() { /** * This value controls the maximum number of replicas each revision should have. * Knative will attempt to never have more than this number of replicas running, or in the process of being created, at any - * one point in time. + * point in time. **/ @ConfigItem Optional maxScale; @@ -441,7 +441,7 @@ public EnvVarsConfig getEnv() { Optional appSecret; /** - * If set, the config amp will mounted to the application container and its contents will be used for application + * If set, the config map will be mounted to the application container and its contents will be used for application * configuration. */ @ConfigItem diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeProcessor.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeProcessor.java index 0497e2a89279d..0d47692d9e713 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeProcessor.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KnativeProcessor.java @@ -64,7 +64,7 @@ public class KnativeProcessor { private static final String LATEST_REVISION = "latest"; /** - * The following properties must be set to workaround + * The following properties must be set to work around * this Dekorate issue: https://github.com/dekorateio/dekorate/issues/869. * Once this issue is fixed, we can get rid of these properties and use the Dekorate knative decorators. */ diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesConfig.java index 4d29e06f77f46..3af344c89d93d 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesConfig.java @@ -78,7 +78,7 @@ public enum DeploymentResourceKind { Map annotations; /** - * Whether or not to add the build timestamp to the Kubernetes annotations + * Whether to add the build timestamp to the Kubernetes annotations * This is a very useful way to have manifests of successive builds of the same * application differ - thus ensuring that Kubernetes will apply the updated resources */ @@ -235,9 +235,8 @@ public enum DeploymentResourceKind { /** * The target deployment platform. - * Defaults to kubernetes. Can be kubernetes, openshift, knative, minikube etc, or any combination of the above as comma - * separated - * list. + * Defaults to kubernetes. Can be kubernetes, openshift, knative, minikube etc., or any combination of the above as comma + * separated list. */ @ConfigItem Optional> deploymentTarget; diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeploy.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeploy.java index f8e312eec9829..765ccfcee79f2 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeploy.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeploy.java @@ -26,11 +26,10 @@ private KubernetesDeploy() { } /** - * @return {@code true} iff @{code quarkus.kubernetes.deploy=true} AND the target Kubernetes API server is reachable + * @return {@code true} if @{code quarkus.kubernetes.deploy=true} AND the target Kubernetes API server is reachable, + * {@code false} otherwise * - * It follows that this can only return {@code false} if the if @{code quarkus.kubernetes.deploy=false} - * - * @throws RuntimeException if communication to the Kubernetes API server errored + * @throws RuntimeException if there was an error while communicating with the Kubernetes API server */ public boolean check() { Result result = doCheck(); @@ -43,11 +42,8 @@ public boolean check() { } /** - * @return {@code true} iff @{code quarkus.kubernetes.deploy=true} AND the target Kubernetes API server is reachable - * - * Never throws an exception even in the face of a communication error with the API server, just returns - * {@code false} - * in that case + * @return {@code true} if @{code quarkus.kubernetes.deploy=true} AND the target Kubernetes API server is reachable + * {@code false} otherwise or if there was an error while communicating with the Kubernetes API server */ public boolean checkSilently() { return doCheck().isAllowed(); diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeployer.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeployer.java index 2e7b7da9364f4..cc5c1dee815e8 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeployer.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/KubernetesDeployer.java @@ -70,7 +70,7 @@ public void selectDeploymentTarget(ContainerImageInfoBuildItem containerImageInf .getActiveContainerImageCapability(capabilities); if (activeContainerImageCapability.isEmpty()) { - // we can't thrown an exception here, because it could prevent the Kubernetes resources from being generated + // we can't throw an exception here, because it could prevent the Kubernetes resources from being generated return; } @@ -135,10 +135,9 @@ public void deploy(KubernetesClientBuildItem kubernetesClient, * The selection is done as follows: * * If there is no target deployment at all, just use vanilla Kubernetes. This will happen in cases where the user does not - * select - * a deployment target and no extensions that specify one are are present + * select a deployment target and no extension that specify one is present. * - * If the user specifies deployment targets, pick the first one + * If the user specifies deployment targets, pick the first one. */ private DeploymentTargetEntry determineDeploymentTarget( ContainerImageInfoBuildItem containerImageInfo, diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/MinikubeManifestGenerator.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/MinikubeManifestGenerator.java index 657c8d9aa2f7b..8c12db2e1558a 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/MinikubeManifestGenerator.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/MinikubeManifestGenerator.java @@ -152,7 +152,7 @@ public LabelSelector createSelector(KubernetesConfig config) { /** * Creates a {@link PodTemplateSpec} for the {@link KubernetesConfig}. * - * @param appConfig The sesssion. + * @param appConfig The session. * @return The pod template specification. */ public static PodTemplateSpec createPodTemplateSpec(KubernetesConfig appConfig, ImageConfiguration imageConfig) { @@ -167,7 +167,7 @@ public static PodTemplateSpec createPodTemplateSpec(KubernetesConfig appConfig, /** * Creates a {@link PodSpec} for the {@link KubernetesConfig}. * - * @param imageConfig The sesssion. + * @param imageConfig The session. * @return The pod specification. */ public static PodSpec createPodSpec(KubernetesConfig appConfig, ImageConfiguration imageConfig) { diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftConfig.java index a44104a17d82a..aca680fde2e4d 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftConfig.java @@ -58,7 +58,7 @@ public static enum DeploymentResourceKind { /** * The kind of the deployment resource to use. - * Supported values are 'Deployment' and 'DeploymentConfig' defaulting to the later. + * Supported values are 'Deployment' and 'DeploymentConfig' defaulting to the latter. */ @ConfigItem Optional deploymentKind; @@ -71,7 +71,7 @@ public static enum DeploymentResourceKind { /** * The name of the application. This value will be used for naming Kubernetes - * resources like: - Deployment - Service and so on ... + * resources like: 'Deployment', 'Service' and so on... */ @ConfigItem(defaultValue = "${quarkus.container-image.name}") Optional name; @@ -502,7 +502,7 @@ public EnvVarsConfig getEnv() { Optional appSecret; /** - * If set, the config amp will mounted to the application container and its contents will be used for application + * If set, the config amp will be mounted to the application container and its contents will be used for application * configuration. */ @ConfigItem diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftProcessor.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftProcessor.java index deb446fbca881..6b1bead78aea8 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftProcessor.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/OpenshiftProcessor.java @@ -95,7 +95,7 @@ public void populateInternalRegistry(OpenshiftConfig openshiftConfig, ContainerI DeploymentResourceKind deploymentResourceKind = openshiftConfig.getDeploymentResourceKind(); if (deploymentResourceKind != DeploymentResourceKind.DeploymentConfig) { if (openshiftConfig.isOpenshiftBuildEnabled(containerImageConfig, capabilities)) { - // Images stored in internal openshift registry use the following patttern: + // Images stored in internal openshift registry use the following pattern: // 'image-registry.openshift-image-registry.svc:5000/{{ project name}}/{{ image name }}: {{image version }}. // So, we need warn users if group does not match currently selected project. containerImageRegistry.produce(new FallbackContainerImageRegistryBuildItem(OPENSHIFT_INTERNAL_REGISTRY)); diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/ProbeConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/ProbeConfig.java index afa2f764e90de..0ce8a0e956dd7 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/ProbeConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/ProbeConfig.java @@ -15,7 +15,7 @@ public class ProbeConfig { * needs to be set * * Assuming the container port has been set (as per above comment), if - * execAction or tcpSocketAction are not set, an http probe will be used + * execAction or tcpSocketAction are not set, an HTTP probe will be used * automatically even if no path is set (which will result in the root path * being used) */ diff --git a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/TrafficConfig.java b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/TrafficConfig.java index 676c54c2e4265..e772badcc6cfb 100644 --- a/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/TrafficConfig.java +++ b/extensions/kubernetes/vanilla/deployment/src/main/java/io/quarkus/kubernetes/deployment/TrafficConfig.java @@ -32,7 +32,7 @@ public class TrafficConfig { /** * Percent indicates that percentage based routing should be used and - * the value indicates the percent of traffic that is be routed to this + * the value indicates the percent of traffic that is to be routed to this * Revision or Configuration. `0` (zero) mean no traffic, `100` means all * traffic. */ diff --git a/extensions/liquibase/deployment/src/main/java/io/quarkus/liquibase/deployment/LiquibaseProcessor.java b/extensions/liquibase/deployment/src/main/java/io/quarkus/liquibase/deployment/LiquibaseProcessor.java index 354ee11506d3f..087e86b82cbca 100644 --- a/extensions/liquibase/deployment/src/main/java/io/quarkus/liquibase/deployment/LiquibaseProcessor.java +++ b/extensions/liquibase/deployment/src/main/java/io/quarkus/liquibase/deployment/LiquibaseProcessor.java @@ -265,7 +265,7 @@ void createBeans(LiquibaseRecorder recorder, additionalBeans .produce(AdditionalBeanBuildItem.builder().addBeanClasses(LiquibaseFactoryProducer.class).setUnremovable() .setDefaultScope(DotNames.SINGLETON).build()); - // add the @LiquibaseDataSource class otherwise it won't registered as a qualifier + // add the @LiquibaseDataSource class otherwise it won't be registered as a qualifier additionalBeans.produce(AdditionalBeanBuildItem.builder().addBeanClass(LiquibaseDataSource.class).build()); Collection dataSourceNames = getDataSourceNames(jdbcDataSourceBuildItems); diff --git a/extensions/liquibase/deployment/src/test/java/io/quarkus/liquibase/test/LiquibaseExtensionConfigMissingNamedDataSourceTest.java b/extensions/liquibase/deployment/src/test/java/io/quarkus/liquibase/test/LiquibaseExtensionConfigMissingNamedDataSourceTest.java index 432b51a820f1f..fbd63777cbc34 100644 --- a/extensions/liquibase/deployment/src/test/java/io/quarkus/liquibase/test/LiquibaseExtensionConfigMissingNamedDataSourceTest.java +++ b/extensions/liquibase/deployment/src/test/java/io/quarkus/liquibase/test/LiquibaseExtensionConfigMissingNamedDataSourceTest.java @@ -16,7 +16,7 @@ /** * Liquibase needs a datasource to work. - * This tests assures, that an error occurs, as soon as a named liquibase configuration points to an missing datasource. + * This tests assures, that an error occurs, as soon as a named liquibase configuration points to a missing datasource. */ public class LiquibaseExtensionConfigMissingNamedDataSourceTest { diff --git a/extensions/liquibase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseFactoryProducer.java b/extensions/liquibase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseFactoryProducer.java index 5984c98c90aed..3d5c71c857b76 100644 --- a/extensions/liquibase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseFactoryProducer.java +++ b/extensions/liquibase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseFactoryProducer.java @@ -12,7 +12,7 @@ * and it's {@code createLiquibaseFactory} method is called at runtime in order to produce * the actual {@code LiquibaseFactory} objects. * - * CDI scopes and qualifiers are setup at build-time, which is why this class is devoid of + * CDI scopes and qualifiers are set up at build-time, which is why this class is devoid of * any CDI annotations * */ diff --git a/extensions/mailer/deployment/src/main/java/io/quarkus/mailer/deployment/MailerProcessor.java b/extensions/mailer/deployment/src/main/java/io/quarkus/mailer/deployment/MailerProcessor.java index 26aa613c50028..1ff2b45ef4179 100644 --- a/extensions/mailer/deployment/src/main/java/io/quarkus/mailer/deployment/MailerProcessor.java +++ b/extensions/mailer/deployment/src/main/java/io/quarkus/mailer/deployment/MailerProcessor.java @@ -108,7 +108,7 @@ void validateMailTemplates( filePath = filePath.replace(File.separatorChar, '/'); } if (filePath.endsWith("html") || filePath.endsWith("htm") || filePath.endsWith("txt")) { - // For e-mails we only consider html and txt templates + // For emails, we only consider html and txt templates filePaths.add(filePath); int idx = filePath.lastIndexOf('.'); if (idx != -1) { diff --git a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Attachment.java b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Attachment.java index 74e83058da8e9..bc3e31c329600 100644 --- a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Attachment.java +++ b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Attachment.java @@ -203,7 +203,7 @@ public Attachment setData(byte[] data) { return this; } - // And the fun begins, we cannot use fromArray on an byte[] as the boxing does not work + // And the fun begins, we cannot use fromArray on a byte[] as the boxing does not work // we cannot use Arrays.stream as it's limited to int, long and double... // so, let's use the good old method creating an iterator for the array. At least it avoids duplicating // the array. diff --git a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Mail.java b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Mail.java index 6b5d223c2616c..07e094bcf5b6f 100644 --- a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Mail.java +++ b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/Mail.java @@ -349,7 +349,7 @@ public Mail setHeaders(Map> headers) { * @param name the name of the attachment, generally a file name. * @param file the file to be attached. Note that the file will be read asynchronously. * @param contentType the content type - * @param contentId the content id. It must follows the {@code } syntax. Then the HTML + * @param contentId the content id. It must follow the {@code } syntax. Then the HTML * content can reference this attachment using {@code src="cid:some-id@some-domain"}. * @return the current {@link Mail} */ @@ -403,7 +403,7 @@ public Mail addAttachment(String name, Publisher data, String contentType) * @param name the name of the attachment, generally a file name. * @param data the binary data to be attached * @param contentType the content type - * @param contentId the content id. It must follows the {@code } syntax. Then the HTML + * @param contentId the content id. It must follow the {@code } syntax. Then the HTML * content can reference this attachment using {@code src="cid:some-id@some-domain"}. * @return the current {@link Mail} */ @@ -418,7 +418,7 @@ public Mail addInlineAttachment(String name, byte[] data, String contentType, St * @param name the name of the attachment, generally a file name. * @param data the binary data to be attached * @param contentType the content type - * @param contentId the content id. It must follows the {@code } syntax. Then the HTML + * @param contentId the content id. It must follow the {@code } syntax. Then the HTML * content can reference this attachment using {@code src="cid:some-id@some-domain"}. * @return the current {@link Mail} */ diff --git a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/DkimSignOptionsConfig.java b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/DkimSignOptionsConfig.java index b52cb67aa2420..a9a7d240a50f8 100644 --- a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/DkimSignOptionsConfig.java +++ b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/DkimSignOptionsConfig.java @@ -30,7 +30,7 @@ public class DkimSignOptionsConfig { public Optional privateKeyPath; /** - * Configures the Agent or User Identifier(AUID). + * Configures the Agent or User Identifier (AUID). */ @ConfigItem public Optional auid; @@ -42,7 +42,7 @@ public class DkimSignOptionsConfig { public Optional selector; /** - * Configures the Signing Domain Identifier. + * Configures the Signing Domain Identifier (SDID). */ @ConfigItem public Optional sdid; diff --git a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/MailConfig.java b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/MailConfig.java index 9bb604fcc9004..f60ec03354383 100644 --- a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/MailConfig.java +++ b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/MailConfig.java @@ -171,7 +171,7 @@ public class MailConfig { public TrustStoreConfig truststore; /** - * Whether or not the mail should always been sent as multipart even if they don't have attachments. + * Whether the mail should always been sent as multipart even if they don't have attachments. * When sets to true, the mail message will be encoded as multipart even for simple mails without attachments. */ @ConfigItem(defaultValue = "false") diff --git a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/TrustStoreConfig.java b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/TrustStoreConfig.java index 348472631e6a9..e3bbc723ca6e4 100644 --- a/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/TrustStoreConfig.java +++ b/extensions/mailer/runtime/src/main/java/io/quarkus/mailer/runtime/TrustStoreConfig.java @@ -33,9 +33,9 @@ public class TrustStoreConfig { * Sets the trust store type. * By default, it guesses the type from the file name extension. * For instance, {@code truststore.pem} will be seen as a PEM file, while {@code truststore.jks} will be seen as a - * JKS file. {@code truststore.p12} and {@code truststore.pfx} will both be seen as PCK#12 files. + * JKS file. {@code truststore.p12} and {@code truststore.pfx} will both be seen as PKCS#12 files. * - * Accepted values are: {@code JKS}, {@code PEM}, {@code PCKS}. + * Accepted values are: {@code JKS}, {@code PEM}, {@code PKCS}. */ @ConfigItem Optional type; diff --git a/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/MicrometerProcessor.java b/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/MicrometerProcessor.java index e3e2e6e8076e2..175f6c7f3bb01 100644 --- a/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/MicrometerProcessor.java +++ b/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/MicrometerProcessor.java @@ -76,7 +76,7 @@ public boolean getAsBoolean() { MicrometerConfig mConfig; /** - * config objects are beans, but they are not unremoveable by default + * config objects are beans, but they are not unremovable by default */ @BuildStep UnremovableBeanBuildItem mpConfigAsBean() { diff --git a/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/binder/HttpBinderProcessor.java b/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/binder/HttpBinderProcessor.java index 892dd13c6aa82..9566c1b775d9f 100644 --- a/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/binder/HttpBinderProcessor.java +++ b/extensions/micrometer/deployment/src/main/java/io/quarkus/micrometer/deployment/binder/HttpBinderProcessor.java @@ -78,7 +78,7 @@ SyntheticBeanBuildItem enableHttpBinders(MicrometerRecorder recorder, createAdditionalBean(additionalBeans, HTTP_METER_FILTER_CONFIGURATION); } - // Other things use this bean to test whether or not http server/client metrics are enabled + // Other things use this bean to test whether http server/client metrics are enabled return SyntheticBeanBuildItem .configure(HttpBinderConfiguration.class) .scope(Singleton.class) diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/CompositeRegistryCreator.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/CompositeRegistryCreator.java index abfd7896af0a0..ae33b75cb6754 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/CompositeRegistryCreator.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/CompositeRegistryCreator.java @@ -10,7 +10,7 @@ import io.quarkus.arc.AlternativePriority; /** - * @return the single resolveable "root" MeterRegistry + * @return the single resolvable "root" MeterRegistry */ public class CompositeRegistryCreator { @Produces diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/MicrometerRecorder.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/MicrometerRecorder.java index b8a0ba82eb6b9..4ed9d324b4dc1 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/MicrometerRecorder.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/MicrometerRecorder.java @@ -125,7 +125,7 @@ public void configureRegistries(MicrometerConfig config, new FileDescriptorMetrics().bindTo(Metrics.globalRegistry); } - // Discover and bind MeterBinders (includes annotated gauges, etc) + // Discover and bind MeterBinders (includes annotated gauges, etc.) // This must be done at runtime. If done before backend registries are // configured, some measurements may be missed. Instance allBinders = beanManager.createInstance() diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/RequestMetricInfo.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/RequestMetricInfo.java index 2bede135ed28c..86a504703b215 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/RequestMetricInfo.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/RequestMetricInfo.java @@ -51,7 +51,7 @@ protected String getNormalizedUriPath(Map matchPatterns, List

    serviceInterface, RestClientBuilder builder) { return; } if (prepClientMetrics()) { - // This must run AFTER the OpenTelmetry client request filter + // This must run AFTER the OpenTelemetry client request filter builder.register(this.clientRequestFilter, Priorities.HEADER_DECORATOR + 1); - // This must run Before the OpenTelmetry client response filter + // This must run Before the OpenTelemetry client response filter builder.register(this.clientResponseFilter, Priorities.HEADER_DECORATOR + 1); } } diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxHttpServerMetrics.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxHttpServerMetrics.java index dd220d6f661f9..31e527a130030 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxHttpServerMetrics.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxHttpServerMetrics.java @@ -47,7 +47,7 @@ public class VertxHttpServerMetrics extends VertxTcpMetrics } /** - * Called when an http server response is pushed. + * Called when an HTTP server response is pushed. * * @param socketMetric a Map for socket metric context or null * @param method the pushed response method @@ -84,7 +84,7 @@ public void requestRouted(HttpRequestMetric requestMetric, String route) { } /** - * Called when an http server request begins. Vert.x will invoke + * Called when an HTTP server request begins. Vert.x will invoke * {@link #responseEnd} when the response has ended or {@link #requestReset} if * the request/response has failed before. * @@ -128,7 +128,7 @@ public void requestReset(HttpRequestMetric requestMetric) { } /** - * Called when an http server response has ended. + * Called when an HTTP server response has ended. * * @param requestMetric a RequestMetricContext or null * @param response the http server response diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxMetricsTags.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxMetricsTags.java index 0cff2413f7d2b..6382a95e604c0 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxMetricsTags.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxMetricsTags.java @@ -14,7 +14,7 @@ public class VertxMetricsTags { /** * Creates a {@code method} tag based on the {@link HttpServerRequest#method()} - * method} of the given {@code request}. + * method of the given {@code request}. * * @param method the request method * @return the method tag whose value is a capitalized method (e.g. GET). diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxTcpMetrics.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxTcpMetrics.java index b9bf30a35b56f..1317e0b565592 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxTcpMetrics.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/binder/vertx/VertxTcpMetrics.java @@ -24,7 +24,7 @@ public class VertxTcpMetrics extends VertxNetworkMetrics * Called when a client has connected, which is applicable for TCP connections. *

    * The remote name of the client is a best effort to provide the name of the - * remote host, i.e if the name is specified at creation time, this name will be + * remote host, i.e. if the name is specified at creation time, this name will be * used otherwise it will be the remote address. * * @param remoteAddress the remote address of the client diff --git a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/config/MicrometerConfig.java b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/config/MicrometerConfig.java index efca8bcd170ce..666d8eea8ec08 100644 --- a/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/config/MicrometerConfig.java +++ b/extensions/micrometer/runtime/src/main/java/io/quarkus/micrometer/runtime/config/MicrometerConfig.java @@ -47,7 +47,7 @@ public final class MicrometerConfig { /** * For MeterRegistry configurations with optional 'enabled' attributes, - * determine whether or not the registry is enabled using {@link #registryEnabledDefault} + * determine whether the registry is enabled using {@link #registryEnabledDefault} * as the default value. */ public boolean checkRegistryEnabledWithDefault(CapabilityEnabled config) { @@ -64,7 +64,7 @@ public boolean checkRegistryEnabledWithDefault(CapabilityEnabled config) { /** * For MeterBinder configurations with optional 'enabled' attributes, - * determine whether or not the binder is enabled using {@link #binderEnabledDefault} + * determine whether the binder is enabled using {@link #binderEnabledDefault} * as the default value. */ public boolean checkBinderEnabledWithDefault(CapabilityEnabled config) { diff --git a/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientBuildTimeConfig.java b/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientBuildTimeConfig.java index c06426b9d99b4..1d4ea4fe1a8bd 100644 --- a/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientBuildTimeConfig.java +++ b/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientBuildTimeConfig.java @@ -7,19 +7,19 @@ @ConfigRoot(name = "mongodb", phase = ConfigPhase.BUILD_TIME) public class MongoClientBuildTimeConfig { /** - * Whether or not an health check is published in case the smallrye-health extension is present. + * Whether a health check is published in case the smallrye-health extension is present. */ @ConfigItem(name = "health.enabled", defaultValue = "true") public boolean healthEnabled; /** - * Whether or not metrics are published in case a metrics extension is present. + * Whether metrics are published in case a metrics extension is present. */ @ConfigItem(name = "metrics.enabled") public boolean metricsEnabled; /** - * Whether or not tracing spans of driver commands are sent in case the smallrye-opentracing extension is present. + * Whether tracing spans of driver commands are sent in case the smallrye-opentracing extension is present. */ @ConfigItem(name = "tracing.enabled") public boolean tracingEnabled; diff --git a/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientProcessor.java b/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientProcessor.java index cac6579a1ecd6..89a4d1a6dd992 100644 --- a/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientProcessor.java +++ b/extensions/mongodb-client/deployment/src/main/java/io/quarkus/mongodb/deployment/MongoClientProcessor.java @@ -207,9 +207,9 @@ MongoConnectionPoolListenerBuildItem setupMetrics( @BuildStep void additionalBeans(BuildProducer additionalBeans) { - // add the @MongoClientName class otherwise it won't registered as a qualifier + // add the @MongoClientName class otherwise it won't be registered as a qualifier additionalBeans.produce(AdditionalBeanBuildItem.builder().addBeanClass(MongoClientName.class).build()); - // make MongoClients an unremoveable bean + // make MongoClients an unremovable bean additionalBeans.produce(AdditionalBeanBuildItem.builder().addBeanClasses(MongoClients.class).setUnremovable().build()); } @@ -328,7 +328,7 @@ private SyntheticBeanBuildItem createBlockingSyntheticBean(MongoClientRecorder r .configure(MongoClient.class) .scope(ApplicationScoped.class) // pass the runtime config into the recorder to ensure that the DataSource related beans - // are created after runtime configuration has been setup + // are created after runtime configuration has been set up .supplier(recorder.mongoClientSupplier(clientName, mongodbConfig)) .setRuntimeInit(); @@ -342,7 +342,7 @@ private SyntheticBeanBuildItem createReactiveSyntheticBean(MongoClientRecorder r .configure(ReactiveMongoClient.class) .scope(ApplicationScoped.class) // pass the runtime config into the recorder to ensure that the DataSource related beans - // are created after runtime configuration has been setup + // are created after runtime configuration has been set up .supplier(recorder.reactiveMongoClientSupplier(clientName, mongodbConfig)) .setRuntimeInit(); diff --git a/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoMetricsTest.java b/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoMetricsTest.java index 55b84c94426d1..10792e0392a2a 100644 --- a/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoMetricsTest.java +++ b/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoMetricsTest.java @@ -48,7 +48,7 @@ void testMetricsInitialization() { assertThat(getGaugeValueOrNull("mongodb.connection-pool.size", getTags())).isNull(); assertThat(getGaugeValueOrNull("mongodb.connection-pool.checked-out-count", getTags())).isNull(); - // Just need to execute something so that an connection is opened + // Just need to execute something so that a connection is opened String name = client.listDatabaseNames().first(); assertEquals(1L, getGaugeValueOrNull("mongodb.connection-pool.size", getTags())); diff --git a/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoTestBase.java b/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoTestBase.java index cfb83a1df13f1..5bb4d9529688b 100644 --- a/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoTestBase.java +++ b/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoTestBase.java @@ -84,7 +84,7 @@ private static MongodExecutable getMongodExecutable(MongodConfig config) { try { return doGetExecutable(config); } catch (Exception e) { - // sometimes the download process can timeout so just sleep and try again + // sometimes the download process can time out so just sleep and try again try { Thread.sleep(1000); } catch (InterruptedException ignored) { diff --git a/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoWithReplicasTestBase.java b/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoWithReplicasTestBase.java index c0261221cb1f0..e1dfc17fdb10a 100644 --- a/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoWithReplicasTestBase.java +++ b/extensions/mongodb-client/deployment/src/test/java/io/quarkus/mongodb/MongoWithReplicasTestBase.java @@ -78,7 +78,7 @@ private static MongodExecutable getMongodExecutable(MongodConfig config) { try { return doGetExecutable(config); } catch (Exception e) { - // sometimes the download process can timeout so just sleep and try again + // sometimes the download process can time out so just sleep and try again try { Thread.sleep(1000); } catch (InterruptedException ignored) { diff --git a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/reactive/ReactiveMongoCollection.java b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/reactive/ReactiveMongoCollection.java index 090aefe622987..65ecdb58f72a4 100644 --- a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/reactive/ReactiveMongoCollection.java +++ b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/reactive/ReactiveMongoCollection.java @@ -844,7 +844,7 @@ Uni insertMany(ClientSession clientSession, List * Removes at most one document from the collection that matches the given filter. * If no documents match, the collection is not modified. * - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. */ @@ -854,7 +854,7 @@ Uni insertMany(ClientSession clientSession, List * Removes at most one document from the collection that matches the given filter. * If no documents match, the collection is not modified. * - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @param options the options to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. @@ -866,7 +866,7 @@ Uni insertMany(ClientSession clientSession, List * If no documents match, the collection is not modified. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. */ @@ -877,7 +877,7 @@ Uni insertMany(ClientSession clientSession, List * If no documents match, the collection is not modified. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @param options the options to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. @@ -888,7 +888,7 @@ Uni insertMany(ClientSession clientSession, List * Removes all documents from the collection that match the given query filter. If no documents match, the * collection is not modified. * - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. */ @@ -898,7 +898,7 @@ Uni insertMany(ClientSession clientSession, List * Removes all documents from the collection that match the given query filter. If no documents match, the * collection is not modified. * - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @param options the options to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. @@ -910,7 +910,7 @@ Uni insertMany(ClientSession clientSession, List * collection is not modified. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. */ @@ -921,7 +921,7 @@ Uni insertMany(ClientSession clientSession, List * collection is not modified. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the delete operation + * @param filter the query filter to apply to the delete operation * @param options the options to apply to the delete operation * @return a {@link Uni} receiving the {@link DeleteResult}, or propagating a {@link com.mongodb.MongoException} on * failure. @@ -931,7 +931,7 @@ Uni insertMany(ClientSession clientSession, List /** * Replace a document in the collection according to the specified arguments. * - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @return a {@link Uni} receiving the {@link UpdateResult} */ @@ -940,7 +940,7 @@ Uni insertMany(ClientSession clientSession, List /** * Replace a document in the collection according to the specified arguments. * - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @param options the options to apply to the replace operation * @return a {@link Uni} receiving the {@link UpdateResult} @@ -951,7 +951,7 @@ Uni insertMany(ClientSession clientSession, List * Replace a document in the collection according to the specified arguments. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @return a {@link Uni} receiving the {@link UpdateResult} */ @@ -961,7 +961,7 @@ Uni insertMany(ClientSession clientSession, List * Replace a document in the collection according to the specified arguments. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @param options the options to apply to the replace operation * @return a {@link Uni} receiving the {@link UpdateResult} @@ -1102,7 +1102,7 @@ Uni updateMany(ClientSession clientSession, Bson filter, Bson upda /** * Atomically find a document and replace it. * - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @return a {@link Uni} completed with the document that was replaced. Depending on the value of the * {@code returnOriginal} @@ -1115,7 +1115,7 @@ Uni updateMany(ClientSession clientSession, Bson filter, Bson upda /** * Atomically find a document and replace it. * - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @param options the options to apply to the operation * @return a {@link Uni} completed with the document that was replaced. Depending on the value of the @@ -1130,7 +1130,7 @@ Uni updateMany(ClientSession clientSession, Bson filter, Bson upda * Atomically find a document and replace it. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @return a {@link Uni} completed with the document that was replaced. Depending on the value of the * {@code returnOriginal} @@ -1144,7 +1144,7 @@ Uni updateMany(ClientSession clientSession, Bson filter, Bson upda * Atomically find a document and replace it. * * @param clientSession the client session with which to associate this operation - * @param filter the query filter to apply the the replace operation + * @param filter the query filter to apply to the replace operation * @param replacement the replacement document * @param options the options to apply to the operation * @return a {@link Uni} completed with the document that was replaced. Depending on the value of the diff --git a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/CredentialConfig.java b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/CredentialConfig.java index e4567377683ab..0f5b5cdf4756e 100644 --- a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/CredentialConfig.java +++ b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/CredentialConfig.java @@ -36,7 +36,7 @@ public class CredentialConfig { /** * Configures the source of the authentication credentials. * This is typically the database that the credentials have been created. The value defaults to the database - * specified in the path portion of the connection string or in the 'database' configuration property.. + * specified in the path portion of the connection string or in the 'database' configuration property. * If the database is specified in neither place, the default value is {@code admin}. This option is only * respected when using the MONGO-CR mechanism (the default). */ diff --git a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongoClientConfig.java b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongoClientConfig.java index 1dea7a021af7f..acc9c7761fa68 100644 --- a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongoClientConfig.java +++ b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongoClientConfig.java @@ -19,7 +19,7 @@ public class MongoClientConfig { *

    * {@code mongodb://} is a required prefix to identify that this is a string in the standard connection format. *

    - * {@code username:password@} are optional. If given, the driver will attempt to login to a database after + * {@code username:password@} are optional. If given, the driver will attempt to log in to a database after * connecting to a database server. For some authentication mechanisms, only the username is specified and the * password is not, in which case the ":" after the username is left off as well. *

    @@ -27,7 +27,7 @@ public class MongoClientConfig { *

    * {@code :portX} is optional and defaults to :27017 if not provided. *

    - * {@code /database} is the name of the database to login to and thus is only relevant if the + * {@code /database} is the name of the database to log in to and thus is only relevant if the * {@code username:password@} syntax is used. If not specified the {@code admin} database will be used by default. *

    * {@code ?options} are connection options. Note that if {@code database} is absent there is still a {@code /} @@ -99,7 +99,7 @@ public class MongoClientConfig { public Optional maxConnectionIdleTime; /** - * Maximum life time of a pooled connection. A connection that exceeds this limit will be closed. + * Maximum lifetime of a pooled connection. A connection that exceeds this limit will be closed. */ @ConfigItem public Optional maxConnectionLifeTime; diff --git a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongodbConfig.java b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongodbConfig.java index 09f0ca79fd171..c0ec6cd6d685f 100644 --- a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongodbConfig.java +++ b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/MongodbConfig.java @@ -50,7 +50,7 @@ public class MongodbConfig { * The default DNS resolver used to handle {@code mongo+srv://} urls cannot be used in a native executable. * This option enables a fallback to use Vert.x to resolve the server names instead of JNDI. * - * IMPORTANT: The resolution may be different in JVM mode (using the default (JNDI-based) DNS resolver, + * IMPORTANT: The resolution may be different in JVM mode using the default (JNDI-based) DNS resolver, * and in native mode. This feature is experimental. */ @ConfigItem(name = "native.dns.use-vertx-dns-resolver", defaultValue = "false") diff --git a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/graal/MongoClientSubstitutions.java b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/graal/MongoClientSubstitutions.java index 2186a4c9384fe..8e13043e610de 100644 --- a/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/graal/MongoClientSubstitutions.java +++ b/extensions/mongodb-client/runtime/src/main/java/io/quarkus/mongodb/runtime/graal/MongoClientSubstitutions.java @@ -153,7 +153,7 @@ final class DefaultDnsResolverSubstitution { * 0 5 5060 example.com. * The priority and weight are ignored, and we just concatenate the host (after removing the ending '.') and port with a * ':' in between, as expected by ServerAddress. - * It's required that the srvHost has at least three parts (e.g. foo.bar.baz) and that all of the resolved hosts have a + * It's required that the srvHost has at least three parts (e.g. foo.bar.baz) and that all the resolved hosts have a * parent * domain equal to the domain of the srvHost. */ diff --git a/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoTestBase.java b/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoTestBase.java index d10b00bafd933..a9d5426ca3485 100644 --- a/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoTestBase.java +++ b/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoTestBase.java @@ -97,7 +97,7 @@ private static MongodExecutable getMongodExecutable(MongodConfig config) { try { return doGetExecutable(config); } catch (Exception e) { - // sometimes the download process can timeout so just sleep and try again + // sometimes the download process can time out so just sleep and try again try { Thread.sleep(1000); } catch (InterruptedException ignored) { diff --git a/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoWithReplicasTestBase.java b/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoWithReplicasTestBase.java index 2d3b0e1e0d420..dce8211bd8b3f 100644 --- a/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoWithReplicasTestBase.java +++ b/extensions/mongodb-client/runtime/src/test/java/io/quarkus/mongodb/reactive/MongoWithReplicasTestBase.java @@ -79,7 +79,7 @@ private static MongodExecutable getMongodExecutable(MongodConfig config) { try { return doGetExecutable(config); } catch (Exception e) { - // sometimes the download process can timeout so just sleep and try again + // sometimes the download process can time out so just sleep and try again try { Thread.sleep(1000); } catch (InterruptedException ignored) { diff --git a/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/RunOptions.java b/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/RunOptions.java index b9f66679bbf4a..bb35f06cb7256 100644 --- a/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/RunOptions.java +++ b/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/RunOptions.java @@ -47,7 +47,7 @@ public RunOptions semantic(Semantic semantic) { * This exception will still be propagated to the caller, so this method should not log or perform any other actions other * than determine what should happen to the current transaction. *

    - * By default the exception is always rolled back. + * By default, the exception is always rolled back. * * @param handler The exception handler * @return This builder diff --git a/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TestTransactionInterceptor.java b/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TestTransactionInterceptor.java index dceb2e2e28352..6b52c3df48da7 100644 --- a/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TestTransactionInterceptor.java +++ b/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TestTransactionInterceptor.java @@ -35,7 +35,7 @@ public Object intercept(InvocationContext context) throws Exception { return context.proceed(); } - // an exception from proceed() has to be captured to avoid shadowing it in finally() with a exception from rollback() + // an exception from proceed() has to be captured to avoid shadowing it in finally() with an exception from rollback() Throwable caught = null; try { userTransaction.begin(); diff --git a/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TransactionalInterceptorBase.java b/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TransactionalInterceptorBase.java index 847648280e981..cb3d8c49bf8ef 100644 --- a/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TransactionalInterceptorBase.java +++ b/extensions/narayana-jta/runtime/src/main/java/io/quarkus/narayana/jta/runtime/interceptor/TransactionalInterceptorBase.java @@ -391,7 +391,7 @@ protected void resetUserTransactionAvailability(boolean previousUserTransactionA } /** - * An utility method to throw any exception as a {@link RuntimeException}. + * A utility method to throw any exception as a {@link RuntimeException}. * We may throw a checked exception (subtype of {@code Throwable} or {@code Exception}) as un-checked exception. * This considers the Java 8 inference rule that states that a {@code throws E} is inferred as {@code RuntimeException}. *

    diff --git a/extensions/narayana-lra/runtime/src/main/java/io/narayana/lra/client/internal/proxy/nonjaxrs/LRAParticipant.java b/extensions/narayana-lra/runtime/src/main/java/io/narayana/lra/client/internal/proxy/nonjaxrs/LRAParticipant.java index 25898d22859d7..18f4c5d389520 100644 --- a/extensions/narayana-lra/runtime/src/main/java/io/narayana/lra/client/internal/proxy/nonjaxrs/LRAParticipant.java +++ b/extensions/narayana-lra/runtime/src/main/java/io/narayana/lra/client/internal/proxy/nonjaxrs/LRAParticipant.java @@ -39,7 +39,7 @@ import io.narayana.lra.logging.LRALogger; /** - * Keeps references to individual non-JAX-RS paraticipant methods in + * Keeps references to individual non-JAX-RS participant methods in * single LRA participant class. */ public class LRAParticipant { @@ -229,7 +229,7 @@ private Response processCompletionStageResult(Method method, URI lraId, URI pare return processResult(result, participantResult.getType(), type); } else { - // participant is still compensating / compeleting + // participant is still compensating / completing return Response.accepted().build(); } } diff --git a/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/graal/NettySubstitutions.java b/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/graal/NettySubstitutions.java index 6b993ea6e23e8..9d94773c2b3e1 100644 --- a/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/graal/NettySubstitutions.java +++ b/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/graal/NettySubstitutions.java @@ -346,7 +346,7 @@ final ChannelFuture initAndRegister() { // channel can be null if newChannel crashed (eg SocketException("too many open files")) channel.unsafe().closeForcibly(); } - // as the Channel is not registered yet we need to force the usage of the GlobalEventExecutor + // as the Channel is not registered yet, we need to force the usage of the GlobalEventExecutor return new DefaultChannelPromise(channel, GlobalEventExecutor.INSTANCE).setFailure(t); } @@ -496,8 +496,8 @@ public long memoryAddress() { } -// We need to delete this class but we let GraalVM dead code elimination do it for us. -// Otherwise it causes a problem when --report-unsupported-elements-at-runtime is enabled: +// We need to delete this class, but we let GraalVM dead code elimination do it for us. +// Otherwise, it causes a problem when --report-unsupported-elements-at-runtime is enabled: // when trying to delete the class, GraalVM throws a java.lang.NoClassDefFoundError: Lcom/aayushatharva/brotli4j/decoder/DecoderJNI$Wrapper; // While we recommend not using this option, some extensions out there are using it. //@TargetClass(className = "io.netty.handler.codec.compression.BrotliDecoder") diff --git a/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualChannel.java b/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualChannel.java index 781abf7e07924..49ad8346b253e 100644 --- a/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualChannel.java +++ b/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualChannel.java @@ -44,7 +44,7 @@ import io.netty.util.internal.logging.InternalLoggerFactory; /** - * A {@link Channel} for the local transport. This is a bit different than a LocalChannel in regular Netty + * A {@link Channel} for the local transport. This is a bit different from a LocalChannel in regular Netty * as it does not require a client event loop and exposes the client inbound queue directly. * * The queue exposed is a blocking queue so that local virtual clients can block on this queue and obtain @@ -218,7 +218,7 @@ protected void doClose() throws Exception { if (oldState != null && oldState != State.CLOSED) { // We need to release all the buffers that may be put into our inbound queue since we closed the Channel // to ensure we not leak any memory. This is fine as it basically gives the same guarantees as TCP which - // means even if the promise was notified before its not really guaranteed that the "remote peer" will + // means even if the promise was notified before, it's not really guaranteed that the "remote peer" will // see the buffer at all. releaseInboundBuffers(); } diff --git a/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualServerChannel.java b/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualServerChannel.java index ce894e11cf7cb..6b8f878842fff 100644 --- a/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualServerChannel.java +++ b/extensions/netty/runtime/src/main/java/io/quarkus/netty/runtime/virtual/VirtualServerChannel.java @@ -33,7 +33,7 @@ /** * A {@link ServerChannel} for the local transport which allows in VM communication. * - * This is a bit different than a LocalServerChannel in regular Netty + * This is a bit different from a LocalServerChannel in regular Netty * as it does not require a client event loop and exposes the client inbound queue directly. * */ diff --git a/extensions/oidc-client/runtime/src/main/java/io/quarkus/oidc/client/runtime/OidcClientImpl.java b/extensions/oidc-client/runtime/src/main/java/io/quarkus/oidc/client/runtime/OidcClientImpl.java index 2f798015f5e62..1641510fa8b99 100644 --- a/extensions/oidc-client/runtime/src/main/java/io/quarkus/oidc/client/runtime/OidcClientImpl.java +++ b/extensions/oidc-client/runtime/src/main/java/io/quarkus/oidc/client/runtime/OidcClientImpl.java @@ -117,7 +117,7 @@ public Uni get() { body.add(entry.getKey(), entry.getValue()); } } - // Retry up to three times with a one second delay between the retries if the connection is closed + // Retry up to three times with a one-second delay between the retries if the connection is closed Uni> response = request.sendBuffer(OidcCommonUtils.encodeForm(body)) .onFailure(ConnectException.class) .retry() diff --git a/extensions/oidc-common/runtime/src/main/java/io/quarkus/oidc/common/runtime/OidcCommonConfig.java b/extensions/oidc-common/runtime/src/main/java/io/quarkus/oidc/common/runtime/OidcCommonConfig.java index 12a109c3e9f45..f06ff3754b26b 100644 --- a/extensions/oidc-common/runtime/src/main/java/io/quarkus/oidc/common/runtime/OidcCommonConfig.java +++ b/extensions/oidc-common/runtime/src/main/java/io/quarkus/oidc/common/runtime/OidcCommonConfig.java @@ -270,7 +270,7 @@ public static class Jwt { /** * JWT audience ('aud') claim value. - * By default the audience is set to the address of the OpenId Connect Provider's token endpoint. + * By default, the audience is set to the address of the OpenId Connect Provider's token endpoint. */ @ConfigItem public Optional audience = Optional.empty(); diff --git a/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/DevUiConfig.java b/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/DevUiConfig.java index 074d1279336dc..8e6d9a99337fd 100644 --- a/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/DevUiConfig.java +++ b/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/DevUiConfig.java @@ -15,7 +15,7 @@ public class DevUiConfig { * * For example: if the grant type is 'code' then an authorization code will be returned directly to Dev UI which will use a * code - * handler to acquire the tokens while a user name and password will have to be entered to request a token using a + * handler to acquire the tokens while a username and password will have to be entered to request a token using a * 'password' grant. */ public Grant grant = new Grant(); diff --git a/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/devservices/keycloak/DevServicesConfig.java b/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/devservices/keycloak/DevServicesConfig.java index df8a0dad18dd1..0163d9d8c2c0f 100644 --- a/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/devservices/keycloak/DevServicesConfig.java +++ b/extensions/oidc/deployment/src/main/java/io/quarkus/oidc/deployment/devservices/keycloak/DevServicesConfig.java @@ -70,7 +70,7 @@ public class DevServicesConfig { * This property is used when {@code shared} is set to {@code true}. * In this case, before starting a container, Dev Services for Keycloak looks for a container with the * {@code quarkus-dev-service-keycloak} label - * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise it + * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise, it * starts a new container with the {@code quarkus-dev-service-keycloak} label set to the specified value. *

    * Container sharing is only used in dev mode. @@ -112,7 +112,7 @@ public class DevServicesConfig { public boolean createRealm; /** - * The Keycloak users map containing the user name and password pairs. + * The Keycloak users map containing the username and password pairs. * If this map is empty then two users, 'alice' and 'bob' with the passwords matching their names will be created. * This property will be used to create the Keycloak users if the realm file pointed to by the 'realm-path' property does * not exist. diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/OidcTenantConfig.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/OidcTenantConfig.java index b6fc9b942519b..d5a0c01789e35 100644 --- a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/OidcTenantConfig.java +++ b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/OidcTenantConfig.java @@ -542,7 +542,7 @@ public enum ResponseMode { * If this property is set to 'true' then the original request URI which was used before * the authentication will be restored after the user has been redirected back to the application. * - * Note if `redirectPath` property is not set the the original request URI will be restored even if this property is + * Note if `redirectPath` property is not set, the original request URI will be restored even if this property is * disabled. */ @ConfigItem(defaultValue = "false") @@ -561,7 +561,7 @@ public enum ResponseMode { * 'error_description' * parameters, instead of the expected authorization 'code'. * - * If this property is set then the user will be redirected to the endpoint which can return a user friendly + * If this property is set then the user will be redirected to the endpoint which can return a user-friendly * error description page. It has to start from a forward slash and will be appended to the request URI's host and port. * For example, if it is set as '/error' and the current request URI is * 'https://localhost:8080/callback?error=invalid_scope' @@ -935,7 +935,7 @@ public static Token fromAudience(String... audience) { * must have a valid ('exp') expiry claim value. The only exception where setting this property relaxes the requirement * is when a logout token is sent with a back-channel logout request since the current * OpenId Connect Back-Channel specification does not explicitly require the logout tokens to contain an 'exp' claim. - * However even if the current logout token is allowed to have no 'exp' claim, the `exp` claim will be still verified + * However, even if the current logout token is allowed to have no 'exp' claim, the `exp` claim will be still verified * if the logout token contains it. */ @ConfigItem diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/TokenStateManager.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/TokenStateManager.java index 40d2a8edd169d..d6e67d8f39b90 100644 --- a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/TokenStateManager.java +++ b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/TokenStateManager.java @@ -9,7 +9,7 @@ * for OIDC Code AuthenticationMechanism to keep it as a session cookie. * * For example, default TokenStateManager concatenates all 3 tokens into a single String but does not persist it. - * Custom TokenStateManager may choose to keep the tokens in the external storage (DB, file system, etc) and return + * Custom TokenStateManager may choose to keep the tokens in the external storage (DB, file system, etc.) and return * a reference to this storage. */ public interface TokenStateManager { diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/DefaultTenantConfigResolver.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/DefaultTenantConfigResolver.java index b72658eead883..b092b3eaed83f 100644 --- a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/DefaultTenantConfigResolver.java +++ b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/DefaultTenantConfigResolver.java @@ -109,7 +109,7 @@ public Uni apply(TenantConfigContext tenantConfig TenantConfigContext tenantContext = getStaticTenantContext(context); if (tenantContext != null && !tenantContext.ready) { - // check if it the connection has already been created + // check if the connection has already been created TenantConfigContext readyTenantContext = tenantConfigBean.getDynamicTenantsConfig() .get(tenantContext.oidcConfig.tenantId.get()); if (readyTenantContext == null) { diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcAuthenticationMechanism.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcAuthenticationMechanism.java index 9b10f8f09d77b..23aada5a97aa7 100644 --- a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcAuthenticationMechanism.java +++ b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcAuthenticationMechanism.java @@ -124,7 +124,7 @@ private static void setTenantIdAttribute(RoutingContext context, String cookiePr context.put(OidcUtils.TENANT_ID_ATTRIBUTE, OidcUtils.DEFAULT_TENANT_ID); } else { String suffix = cookieName.substring(cookiePrefix.length() + 1); - // It can be either a tenant_id or tenand_id and cookie suffix property, example, q_session_github or q_session_github_test + // It can be either a tenant_id, or a tenant_id and cookie suffix property, example, q_session_github or q_session_github_test int index = suffix.indexOf("_"); String tenantId = index == -1 ? suffix : suffix.substring(0, index); context.put(OidcUtils.TENANT_ID_ATTRIBUTE, tenantId); diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcProviderClient.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcProviderClient.java index f509a3613bb24..5aab55d763619 100644 --- a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcProviderClient.java +++ b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcProviderClient.java @@ -128,7 +128,7 @@ private UniOnItem> getHttpResponse(String uri, MultiMap for formBody.add(OidcConstants.CLIENT_ID, oidcConfig.clientId.get()); } LOG.debugf("Get token on: %s params: %s headers: %s", metadata.getTokenUri(), formBody, request.headers()); - // Retry up to three times with a one second delay between the retries if the connection is closed. + // Retry up to three times with a one-second delay between the retries if the connection is closed. Uni> response = request.sendBuffer(OidcCommonUtils.encodeForm(formBody)) .onFailure(ConnectException.class) .retry() diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcUtils.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcUtils.java index 7c3cf23dd2221..23ca7aade9a8a 100644 --- a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcUtils.java +++ b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/runtime/OidcUtils.java @@ -90,7 +90,7 @@ public static JsonObject decodeJwtContent(String jwt) { // part 2: token content String encodedContent = tokens.nextToken(); - // lets check only 1 more signature part is available + // let's check only 1 more signature part is available if (tokens.countTokens() != 1) { return null; } diff --git a/extensions/opentelemetry/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/OpenTelemetryUtil.java b/extensions/opentelemetry/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/OpenTelemetryUtil.java index c406794ba449d..8f66651dc3eec 100644 --- a/extensions/opentelemetry/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/OpenTelemetryUtil.java +++ b/extensions/opentelemetry/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/OpenTelemetryUtil.java @@ -71,7 +71,7 @@ public static ContextPropagators mapPropagators(List propagators) { * In case of duplicate keys, the latest takes precedence. * * @param headers nullable list of "key=value" pairs - * @return non null map of key-value pairs + * @return non-null map of key-value pairs */ public static Map convertKeyValueListToMap(List headers) { if (headers == null) { diff --git a/extensions/panache/hibernate-orm-panache-kotlin/runtime/pom.xml b/extensions/panache/hibernate-orm-panache-kotlin/runtime/pom.xml index f2a22133bf6cd..66f4191d3f9b9 100644 --- a/extensions/panache/hibernate-orm-panache-kotlin/runtime/pom.xml +++ b/extensions/panache/hibernate-orm-panache-kotlin/runtime/pom.xml @@ -100,7 +100,7 @@ org.jetbrains.kotlin kotlin-maven-plugin - + compile compile diff --git a/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheQuery.kt b/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheQuery.kt index c0e4e12b00921..d773d19f1b809 100644 --- a/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheQuery.kt +++ b/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheQuery.kt @@ -156,7 +156,7 @@ interface PanacheQuery { * WARNING: setting filters can only be done on the underlying Hibernate [Session] and so this * will modify the session's filters for the duration of obtaining the results (not while building * the query). Enabled filters will be removed from the session afterwards, but no effort is made to - * preserve filters enabled on the session outside of this API. + * preserve filters enabled on the session outside this API. * * @param filterName The name of the filter to enable * @param parameters The set of parameters for the filter, if the filter requires parameters @@ -172,7 +172,7 @@ interface PanacheQuery { * WARNING: setting filters can only be done on the underlying Hibernate [Session] and so this * will modify the session's filters for the duration of obtaining the results (not while building * the query). Enabled filters will be removed from the session afterwards, but no effort is made to - * preserve filters enabled on the session outside of this API. + * preserve filters enabled on the session outside this API. * * @param filterName The name of the filter to enable * @param parameters The set of parameters for the filter, if the filter requires parameters @@ -188,7 +188,7 @@ interface PanacheQuery { * WARNING: setting filters can only be done on the underlying Hibernate [Session] and so this * will modify the session's filters for the duration of obtaining the results (not while building * the query). Enabled filters will be removed from the session afterwards, but no effort is made to - * preserve filters enabled on the session outside of this API. + * preserve filters enabled on the session outside this API. * * @param filterName The name of the filter to enable * @return this query, modified @@ -240,4 +240,4 @@ interface PanacheQuery { * @see [PanacheQuery.firstResult] */ fun singleResult(): Entity -} \ No newline at end of file +} diff --git a/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheRepositoryBase.kt b/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheRepositoryBase.kt index 5f8b8cc156801..68d86bd0c9c3a 100644 --- a/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheRepositoryBase.kt +++ b/extensions/panache/hibernate-orm-panache-kotlin/runtime/src/main/kotlin/io/quarkus/hibernate/orm/panache/kotlin/PanacheRepositoryBase.kt @@ -576,7 +576,7 @@ interface PanacheRepositoryBase { * Persist all given entities. * * @param entities the entities to persist - * @see [PanacfluheRepositoryBase.persist] + * @see [PanacheRepositoryBase.persist] */ fun persist(entities: Stream) { INSTANCE.persist(entities) diff --git a/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/AdditionalJpaOperations.java b/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/AdditionalJpaOperations.java index d4d7c80462c90..f09ba6149feda 100644 --- a/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/AdditionalJpaOperations.java +++ b/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/AdditionalJpaOperations.java @@ -24,7 +24,7 @@ import io.quarkus.panache.common.Parameters; import io.quarkus.panache.common.Sort; -//TODO this class is only needed by the Spring Data JPA module and would be placed there it it weren't for a dev-mode classloader issue +//TODO this class is only needed by the Spring Data JPA module and would not be placed there if it weren't for a dev-mode classloader issue // see https://github.com/quarkusio/quarkus/issues/6214 public class AdditionalJpaOperations { @@ -55,7 +55,7 @@ public static PanacheQuery find(AbstractJpaOperations jpaOperations, Class public static long deleteAllWithCascade(AbstractJpaOperations jpaOperations, Class entityClass) { EntityManager em = jpaOperations.getEntityManager(); - //detecting the case where there are cascade-delete associations, and do the the bulk delete query otherwise. + //detecting the case where there are cascade-delete associations, and do the bulk delete query otherwise. if (deleteOnCascadeDetected(jpaOperations, entityClass)) { int count = 0; List objects = jpaOperations.listAll(entityClass); diff --git a/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/CustomCountPanacheQuery.java b/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/CustomCountPanacheQuery.java index b23da33663d42..07a68ed4a617e 100644 --- a/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/CustomCountPanacheQuery.java +++ b/extensions/panache/hibernate-orm-panache/runtime/src/main/java/io/quarkus/hibernate/orm/panache/runtime/CustomCountPanacheQuery.java @@ -7,7 +7,7 @@ import io.quarkus.hibernate.orm.panache.common.runtime.CommonPanacheQueryImpl; -//TODO this class is only needed by the Spring Data JPA module and would be placed there it it weren't for a dev-mode classloader issue +//TODO this class is only needed by the Spring Data JPA module and would not be placed there if it weren't for a dev-mode classloader issue // see https://github.com/quarkusio/quarkus/issues/6214 public class CustomCountPanacheQuery extends PanacheQueryImpl { diff --git a/extensions/panache/hibernate-reactive-panache/runtime/src/main/java/io/quarkus/hibernate/reactive/panache/runtime/CustomCountPanacheQuery.java b/extensions/panache/hibernate-reactive-panache/runtime/src/main/java/io/quarkus/hibernate/reactive/panache/runtime/CustomCountPanacheQuery.java index 8e54bb9df36cb..0bbe7ca504f85 100644 --- a/extensions/panache/hibernate-reactive-panache/runtime/src/main/java/io/quarkus/hibernate/reactive/panache/runtime/CustomCountPanacheQuery.java +++ b/extensions/panache/hibernate-reactive-panache/runtime/src/main/java/io/quarkus/hibernate/reactive/panache/runtime/CustomCountPanacheQuery.java @@ -5,7 +5,7 @@ import io.quarkus.hibernate.reactive.panache.common.runtime.CommonPanacheQueryImpl; import io.smallrye.mutiny.Uni; -//TODO this class is only needed by the Spring Data JPA module and would be placed there it it weren't for a dev-mode classloader issue +//TODO this class is only needed by the Spring Data JPA module and would not be placed there if it weren't for a dev-mode classloader issue // see https://github.com/quarkusio/quarkus/issues/6214 public class CustomCountPanacheQuery extends PanacheQueryImpl { diff --git a/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/binder/MongoParserVisitor.java b/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/binder/MongoParserVisitor.java index 828c6a2f7fa24..7af3a4cfbc5d9 100644 --- a/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/binder/MongoParserVisitor.java +++ b/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/binder/MongoParserVisitor.java @@ -56,7 +56,7 @@ public String visitLessThanOrEqualPredicate(HqlParser.LessThanOrEqualPredicateCo public String visitLikePredicate(HqlParser.LikePredicateContext ctx) { String parameter = ctx.expression(1).accept(this); if (parameter.indexOf('/') == 1 && parameter.lastIndexOf('/') > 1) { - // In case we have something like '/.*/.*' we are in a JavaScript regex so we must unescape the parameter. + // In case we have something like '/.*/.*' we are in a JavaScript regex, so we must unescape the parameter. // We do this here instead of inside visitParameterExpression to avoid unescaping for non-regex parameters. parameter = parameter.substring(1, parameter.length() - 1); } diff --git a/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/reactive/runtime/ReactiveMongoOperations.java b/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/reactive/runtime/ReactiveMongoOperations.java index bf433a4dd30f3..1bc547dd25f34 100644 --- a/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/reactive/runtime/ReactiveMongoOperations.java +++ b/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/reactive/runtime/ReactiveMongoOperations.java @@ -275,7 +275,7 @@ private Uni persistOrUpdate(ReactiveMongoCollection collection, Object ent } private Uni persistOrUpdate(ReactiveMongoCollection collection, List entities) { - //this will be an ordered bulk: it's less performant than a unordered one but will fail at the first failed write + //this will be an ordered bulk: it's less performant than an unordered one but will fail at the first failed write List bulk = new ArrayList<>(); for (Object entity : entities) { //we transform the entity as a document first diff --git a/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/runtime/MongoOperations.java b/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/runtime/MongoOperations.java index d6f446e88447c..bca0e045f31fb 100644 --- a/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/runtime/MongoOperations.java +++ b/extensions/panache/mongodb-panache-common/runtime/src/main/java/io/quarkus/mongodb/panache/common/runtime/MongoOperations.java @@ -283,7 +283,7 @@ private void persistOrUpdate(List entities) { MongoCollection collection = mongoCollection(firstEntity); ClientSession session = getSession(firstEntity); - //this will be an ordered bulk: it's less performant than a unordered one but will fail at the first failed write + //this will be an ordered bulk: it's less performant than an unordered one but will fail at the first failed write List bulk = new ArrayList<>(); for (Object entity : entities) { //we transform the entity as a document first diff --git a/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/PanacheMongoRepositoryBase.kt b/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/PanacheMongoRepositoryBase.kt index e8c9c15339691..1fcb6306e72a3 100644 --- a/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/PanacheMongoRepositoryBase.kt +++ b/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/PanacheMongoRepositoryBase.kt @@ -24,7 +24,7 @@ import io.quarkus.mongodb.panache.kotlin.runtime.KotlinMongoOperations.Companion interface PanacheMongoRepositoryBase { /** * Persist the given entity in the database. - * This will set it's ID field if not already set. + * This will set its ID field if not already set. * * @param entity the entity to insert. * @see [persist] @@ -40,7 +40,7 @@ interface PanacheMongoRepositoryBase { fun update(entity: Entity) = INSTANCE.update(entity) /** - * Persist the given entity in the database or update it if it already exist. + * Persist the given entity in the database or update it if it already exists. * * @param entity the entity to update. * @see [persistOrUpdate] diff --git a/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoCompanion.kt b/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoCompanion.kt index 00d68029ee6ee..662dc14692748 100644 --- a/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoCompanion.kt +++ b/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoCompanion.kt @@ -144,7 +144,7 @@ interface ReactivePanacheMongoCompanionBase = throw INSTANCE.implementationInjectionMissing() /** - * Find entities using a a BSON query and a BSON sort. + * Find entities using a BSON query and a BSON sort. * * @param query a [Document] query * @param sort the [Document] sort @@ -296,7 +296,7 @@ interface ReactivePanacheMongoCompanionBase> = throw INSTANCE.implementationInjectionMissing() /** - * Find entities using a a BSON query and a BSON sort. + * Find entities using a BSON query and a BSON sort. * This method is a shortcut for `find(query, sort).list()`. * * @param query a [Document] query @@ -451,7 +451,7 @@ interface ReactivePanacheMongoCompanionBase = throw INSTANCE.implementationInjectionMissing() /** - * Find entities using a a BSON query and a BSON sort. + * Find entities using a BSON query and a BSON sort. * This method is a shortcut for `find(query, sort).stream()`. * * @param query a [Document] query @@ -780,4 +780,4 @@ interface ReactivePanacheMongoCompanionBase update(): Uni = INSTANCE.update(this).map { this as T } /** - * Persist this entity in the database or update it if it already exist. + * Persist this entity in the database or update it if it already exists. * * @see [persistOrUpdate] */ @@ -41,4 +41,4 @@ abstract class ReactivePanacheMongoEntityBase { * @see [ReactivePanacheMongoCompanionBase.deleteAll] */ fun delete(): Uni = INSTANCE.delete(this) -} \ No newline at end of file +} diff --git a/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoRepositoryBase.kt b/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoRepositoryBase.kt index 8a0d87d46a987..d3ab1cb0f1aa3 100644 --- a/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoRepositoryBase.kt +++ b/extensions/panache/mongodb-panache-kotlin/runtime/src/main/kotlin/io/quarkus/mongodb/panache/kotlin/reactive/ReactivePanacheMongoRepositoryBase.kt @@ -26,7 +26,7 @@ import java.util.stream.Stream interface ReactivePanacheMongoRepositoryBase { /** * Persist the given entity in the database. - * This will set it's ID field if not already set. + * This will set its ID field if not already set. * * @param entity the entity to insert. * @see [persist] @@ -42,7 +42,7 @@ interface ReactivePanacheMongoRepositoryBase { fun update(entity: Entity): Uni = INSTANCE.update(entity).map { entity } /** - * Persist the given entity in the database or update it if it already exist. + * Persist the given entity in the database or update it if it already exists. * * @param entity the entity to update. * @see [persistOrUpdate] @@ -167,7 +167,7 @@ interface ReactivePanacheMongoRepositoryBase { fun find(query: Document): ReactivePanacheQuery = throw INSTANCE.implementationInjectionMissing() /** - * Find entities using a a BSON query and a BSON sort. + * Find entities using a BSON query and a BSON sort. * * @param query a [Document] query * @param sort the [Document] sort @@ -307,7 +307,7 @@ interface ReactivePanacheMongoRepositoryBase { fun list(query: Document): Uni> = throw INSTANCE.implementationInjectionMissing() /** - * Find entities using a a BSON query and a BSON sort. + * Find entities using a BSON query and a BSON sort. * This method is a shortcut for `find(query, sort).list()`. * * @param query a [Document] query @@ -450,7 +450,7 @@ interface ReactivePanacheMongoRepositoryBase { fun stream(query: Document): Multi = throw INSTANCE.implementationInjectionMissing() /** - * Find entities using a a BSON query and a BSON sort. + * Find entities using a BSON query and a BSON sort. * This method is a shortcut for `find(query, sort).stream()`. * * @param query a [Document] query @@ -746,4 +746,4 @@ interface ReactivePanacheMongoRepositoryBase { */ @GenerateBridge fun mongoDatabase(): ReactiveMongoDatabase = throw INSTANCE.implementationInjectionMissing() -} \ No newline at end of file +} diff --git a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoEntityBase.java b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoEntityBase.java index d5c3d86490cb1..05b4269dcf41b 100644 --- a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoEntityBase.java +++ b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoEntityBase.java @@ -48,7 +48,7 @@ public void update() { } /** - * Persist this entity in the database or update it if it already exist. + * Persist this entity in the database or update it if it already exists. * * @see #persistOrUpdate(Iterable) * @see #persistOrUpdate(Stream) diff --git a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoRepositoryBase.java b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoRepositoryBase.java index aec13875a1674..93e9ff8c6f5bc 100644 --- a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoRepositoryBase.java +++ b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/PanacheMongoRepositoryBase.java @@ -30,7 +30,7 @@ public interface PanacheMongoRepositoryBase { /** * Persist the given entity in the database. - * This will set it's ID field if not already set. + * This will set its ID field if not already set. * * @param entity the entity to insert. * @see #persist(Iterable) @@ -54,7 +54,7 @@ default void update(Entity entity) { } /** - * Persist the given entity in the database or update it if it already exist. + * Persist the given entity in the database or update it if it already exists. * * @param entity the entity to update. * @see #persistOrUpdate(Iterable) diff --git a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoEntityBase.java b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoEntityBase.java index e022c93c93671..0b9311c671b8c 100644 --- a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoEntityBase.java +++ b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoEntityBase.java @@ -51,7 +51,7 @@ public Uni update() { } /** - * Persist this entity in the database or update it if it already exist. + * Persist this entity in the database or update it if it already exists. * * @see #persistOrUpdate(Iterable) * @see #persistOrUpdate(Stream) diff --git a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoRepositoryBase.java b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoRepositoryBase.java index 8d79c7126e116..d594a091fe173 100644 --- a/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoRepositoryBase.java +++ b/extensions/panache/mongodb-panache/runtime/src/main/java/io/quarkus/mongodb/panache/reactive/ReactivePanacheMongoRepositoryBase.java @@ -31,7 +31,7 @@ public interface ReactivePanacheMongoRepositoryBase { /** * Persist the given entity in the database. - * This will set it's ID field if not already set. + * This will set its ID field if not already set. * * @param entity the entity to insert. * @see #persist(Iterable) @@ -55,7 +55,7 @@ default Uni update(Entity entity) { } /** - * Persist the given entity in the database or update it if it already exist. + * Persist the given entity in the database or update it if it already exists. * * @param entity the entity to update. * @see #persistOrUpdate(Iterable) diff --git a/extensions/panache/mongodb-rest-data-panache/deployment/src/main/java/io/quarkus/mongodb/rest/data/panache/deployment/MongoPanacheRestProcessor.java b/extensions/panache/mongodb-rest-data-panache/deployment/src/main/java/io/quarkus/mongodb/rest/data/panache/deployment/MongoPanacheRestProcessor.java index 533a4caf7f317..92619434954b7 100644 --- a/extensions/panache/mongodb-rest-data-panache/deployment/src/main/java/io/quarkus/mongodb/rest/data/panache/deployment/MongoPanacheRestProcessor.java +++ b/extensions/panache/mongodb-rest-data-panache/deployment/src/main/java/io/quarkus/mongodb/rest/data/panache/deployment/MongoPanacheRestProcessor.java @@ -157,7 +157,7 @@ private List getGenericTypes(ClassInfo classInfo) { /** * Annotate Mongo entity ID fields with a RESTEasy links annotation. - * Otherwise RESTEasy will not be able to generate links that use ID as path parameter. + * Otherwise, RESTEasy will not be able to generate links that use ID as path parameter. */ private BytecodeTransformerBuildItem getEntityIdAnnotationTransformer(String entityClassName, String idFieldName) { return new BytecodeTransformerBuildItem(entityClassName, diff --git a/extensions/panache/panache-common/deployment/src/main/java/io/quarkus/panache/common/deployment/visitors/PanacheEntityClassAccessorGenerationVisitor.java b/extensions/panache/panache-common/deployment/src/main/java/io/quarkus/panache/common/deployment/visitors/PanacheEntityClassAccessorGenerationVisitor.java index b9b5623e9e26c..14c26d1215b68 100644 --- a/extensions/panache/panache-common/deployment/src/main/java/io/quarkus/panache/common/deployment/visitors/PanacheEntityClassAccessorGenerationVisitor.java +++ b/extensions/panache/panache-common/deployment/src/main/java/io/quarkus/panache/common/deployment/visitors/PanacheEntityClassAccessorGenerationVisitor.java @@ -86,7 +86,7 @@ public AnnotationVisitor visitAnnotation(String descriptor, boolean visible) { public void visitEnd() { // Add the @JaxbTransient property to the field so that JAXB prefers the generated getter (otherwise JAXB complains about // having a field and property both with the same name) - // JSONB will already use the getter so we're good + // JSONB will already use the getter, so we're good // Note: we don't need to check if we already have @XmlTransient in the descriptors because if we did, we moved it to the getter // so we can't have any duplicate super.visitAnnotation(JAXB_TRANSIENT_SIGNATURE, true); diff --git a/extensions/panache/panache-common/runtime/src/main/java/io/quarkus/panache/common/Sort.java b/extensions/panache/panache-common/runtime/src/main/java/io/quarkus/panache/common/Sort.java index d5f0d4238d5b9..4dfafee123bbb 100644 --- a/extensions/panache/panache-common/runtime/src/main/java/io/quarkus/panache/common/Sort.java +++ b/extensions/panache/panache-common/runtime/src/main/java/io/quarkus/panache/common/Sort.java @@ -35,7 +35,7 @@ public enum Direction { */ Ascending, /** - * Sort in descending order (opposite from the default). + * Sort in descending order (opposite to the default). */ Descending; } diff --git a/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/ListMethodImplementor.java b/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/ListMethodImplementor.java index 534dfa881d331..16248f32957b2 100644 --- a/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/ListMethodImplementor.java +++ b/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/ListMethodImplementor.java @@ -51,7 +51,7 @@ public ListMethodImplementor(boolean isResteasyClassic, boolean isReactivePanach * Generate JAX-RS GET method. * * The RESTEasy Classic version exposes {@link RestDataResource#list(Page, Sort)} - * and the generated pseudo-code with enabled pagination is shown below. If pagination is disabled pageIndex and pageSize + * and the generated pseudocode with enabled pagination is shown below. If pagination is disabled pageIndex and pageSize * query parameters are skipped and null {@link Page} instance is used. * *
    diff --git a/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/hal/ListHalMethodImplementor.java b/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/hal/ListHalMethodImplementor.java
    index 55b54691f3e61..6c888edd013db 100644
    --- a/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/hal/ListHalMethodImplementor.java
    +++ b/extensions/panache/rest-data-panache/deployment/src/main/java/io/quarkus/rest/data/panache/deployment/methods/hal/ListHalMethodImplementor.java
    @@ -51,7 +51,7 @@ public ListHalMethodImplementor(boolean isResteasyClassic, boolean isReactivePan
          * Generate HAL JAX-RS GET method.
          *
          * The RESTEasy Classic version exposes {@link RestDataResource#list(Page, Sort)} via HAL JAX-RS method.
    -     * Generated pseudo-code with enabled pagination is shown below. If pagination is disabled pageIndex and pageSize
    +     * Generated pseudocode with enabled pagination is shown below. If pagination is disabled pageIndex and pageSize
          * query parameters are skipped and null {@link Page} instance is used.
          *
          * 
    diff --git a/extensions/panache/rest-data-panache/runtime/src/main/java/io/quarkus/rest/data/panache/MethodProperties.java b/extensions/panache/rest-data-panache/runtime/src/main/java/io/quarkus/rest/data/panache/MethodProperties.java
    index 2010cd4172394..5c0c8bbc26ff7 100644
    --- a/extensions/panache/rest-data-panache/runtime/src/main/java/io/quarkus/rest/data/panache/MethodProperties.java
    +++ b/extensions/panache/rest-data-panache/runtime/src/main/java/io/quarkus/rest/data/panache/MethodProperties.java
    @@ -16,7 +16,7 @@
     public @interface MethodProperties {
     
         /**
    -     * Expose this operations as a JAX-RS endpoint.
    +     * Expose this operation as a JAX-RS endpoint.
          * 

    * Default: true */ diff --git a/extensions/picocli/deployment/src/main/java/io/quarkus/picocli/deployment/PicocliDeploymentConfiguration.java b/extensions/picocli/deployment/src/main/java/io/quarkus/picocli/deployment/PicocliDeploymentConfiguration.java index e8d9e43bbd5a6..b68fa90a57e44 100644 --- a/extensions/picocli/deployment/src/main/java/io/quarkus/picocli/deployment/PicocliDeploymentConfiguration.java +++ b/extensions/picocli/deployment/src/main/java/io/quarkus/picocli/deployment/PicocliDeploymentConfiguration.java @@ -12,7 +12,7 @@ class PicocliDeploymentConfiguration { *

    *

    * CAUTION: this will have serious build-time performance impact since - * this is run on every restart in devmode, use with care! + * this is run on every restart in dev mode, use with care! *

    *

    * This property is intended to be used only in cases where an incompatible change in the diff --git a/extensions/quartz/runtime/src/main/java/io/quarkus/quartz/runtime/QuartzScheduler.java b/extensions/quartz/runtime/src/main/java/io/quarkus/quartz/runtime/QuartzScheduler.java index 57bac3a268d46..17812724b7dae 100644 --- a/extensions/quartz/runtime/src/main/java/io/quarkus/quartz/runtime/QuartzScheduler.java +++ b/extensions/quartz/runtime/src/main/java/io/quarkus/quartz/runtime/QuartzScheduler.java @@ -470,7 +470,7 @@ void start(@Observes @Priority(Interceptor.Priority.PLATFORM_BEFORE) StartupEven } /** - * Need to gracefully shutdown the scheduler making sure that all triggers have been + * Need to gracefully shut down the scheduler making sure that all triggers have been * released before datasource shutdown. * * @param event ignored diff --git a/extensions/qute/deployment/src/main/java/io/quarkus/qute/deployment/QuteProcessor.java b/extensions/qute/deployment/src/main/java/io/quarkus/qute/deployment/QuteProcessor.java index ccfa3467b2868..761beebd9e427 100644 --- a/extensions/qute/deployment/src/main/java/io/quarkus/qute/deployment/QuteProcessor.java +++ b/extensions/qute/deployment/src/main/java/io/quarkus/qute/deployment/QuteProcessor.java @@ -463,7 +463,7 @@ public Optional getVariant() { @Override public void beforeParsing(ParserHelper parserHelper) { - // The template id may be the full path, e.g. "items.html" or a path without the suffic, e.g. "items" + // The template id may be the full path, e.g. "items.html" or a path without the suffix, e.g. "items" String templateId = parserHelper.getTemplateId(); if (filePaths.contains(templateId)) { diff --git a/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/CheckedTemplateRequireTypeSafeTest.java b/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/CheckedTemplateRequireTypeSafeTest.java index 273d3cb25dd19..9de2a781f37b3 100644 --- a/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/CheckedTemplateRequireTypeSafeTest.java +++ b/extensions/qute/deployment/src/test/java/io/quarkus/qute/deployment/typesafe/CheckedTemplateRequireTypeSafeTest.java @@ -28,7 +28,7 @@ public class CheckedTemplateRequireTypeSafeTest { + "{inject:fool.getJoke(null)} " + "{inject:fool.getJoke(identifier)} " + "{#each name.chars.iterator}" - // {it_index} is not considered an error because the binding is registered by the loop section !} + // {it_index} is not considered an error because the binding is registered by the loop section ! + "{it_index}." // however, {index} is an error + "{index}" diff --git a/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/EngineProducer.java b/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/EngineProducer.java index d7de6b8c4c07c..926314f1d4fa5 100644 --- a/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/EngineProducer.java +++ b/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/EngineProducer.java @@ -119,7 +119,7 @@ public EngineProducer(QuteContext context, QuteConfig config, QuteRuntimeConfig builder.strictRendering(true); } else { builder.strictRendering(false); - // If needed use a specific result mapper for the selected strategy + // If needed, use a specific result mapper for the selected strategy if (runtimeConfig.propertyNotFoundStrategy.isPresent()) { switch (runtimeConfig.propertyNotFoundStrategy.get()) { case THROW_EXCEPTION: @@ -187,7 +187,7 @@ public EngineProducer(QuteContext context, QuteConfig config, QuteRuntimeConfig builder.addLocator(this::locate); registerCustomLocators(builder, locators); - // Add a special parserk hook for Qute.fmt() methods + // Add a special parser hook for Qute.fmt() methods builder.addParserHook(new Qute.IndexedArgumentsParserHook()); // Add template initializers @@ -195,7 +195,7 @@ public EngineProducer(QuteContext context, QuteConfig config, QuteRuntimeConfig builder.addTemplateInstanceInitializer(createInitializer(initializerClass)); } - // Add a special initializer for templates that contain a inject/cdi namespace expressions + // Add a special initializer for templates that contain an inject/cdi namespace expressions Map discoveredInjectTemplates = new HashMap<>(); builder.addTemplateInstanceInitializer(new Initializer() { diff --git a/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/devmode/QuteDevConsoleRecorder.java b/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/devmode/QuteDevConsoleRecorder.java index 820c34bdffb81..264efb2b570ac 100644 --- a/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/devmode/QuteDevConsoleRecorder.java +++ b/extensions/qute/runtime/src/main/java/io/quarkus/qute/runtime/devmode/QuteDevConsoleRecorder.java @@ -13,7 +13,7 @@ public class QuteDevConsoleRecorder { public static final String RENDER_HANDLER = QuteDevConsoleRecorder.class.getName() + ".RENDER_HANDLER"; public void setupRenderer() { - //setup the render handler that is used to handle the template + //set up the render handler that is used to handle the template //this is invoked from the deployment side ClassLoader tccl = Thread.currentThread().getContextClassLoader(); DevConsoleManager.setGlobal(RENDER_HANDLER, new BiFunction() { diff --git a/extensions/reactive-db2-client/runtime/src/main/java/io/quarkus/reactive/db2/client/runtime/health/ReactiveDB2DataSourcesHealthCheck.java b/extensions/reactive-db2-client/runtime/src/main/java/io/quarkus/reactive/db2/client/runtime/health/ReactiveDB2DataSourcesHealthCheck.java index 214f018c200ca..17d18fb5c8a26 100644 --- a/extensions/reactive-db2-client/runtime/src/main/java/io/quarkus/reactive/db2/client/runtime/health/ReactiveDB2DataSourcesHealthCheck.java +++ b/extensions/reactive-db2-client/runtime/src/main/java/io/quarkus/reactive/db2/client/runtime/health/ReactiveDB2DataSourcesHealthCheck.java @@ -27,7 +27,7 @@ @Readiness @ApplicationScoped /** - * Implementation note: this healthcheck doesn't extend ReactiveDatasourceHealthCheck + * Implementation note: this health check doesn't extend ReactiveDatasourceHealthCheck * as a DB2Pool is based on Mutiny: does not extend io.vertx.sqlclient.Pool */ class ReactiveDB2DataSourcesHealthCheck implements HealthCheck { diff --git a/extensions/reactive-routes/deployment/src/test/java/io/quarkus/vertx/web/cors/package-info.java b/extensions/reactive-routes/deployment/src/test/java/io/quarkus/vertx/web/cors/package-info.java index a1a18f0717e08..a81b5bfb8b552 100644 --- a/extensions/reactive-routes/deployment/src/test/java/io/quarkus/vertx/web/cors/package-info.java +++ b/extensions/reactive-routes/deployment/src/test/java/io/quarkus/vertx/web/cors/package-info.java @@ -1,5 +1,5 @@ /** - * These tests checks that filter are correctly applied when using route automatically registered. - * The are the same as the CORS test from vertx-http, except that the route is registered using @Route. + * These tests check that filter are correctly applied when using route automatically registered. + * They are the same as the CORS test from vertx-http, except that the route is registered using @Route. */ -package io.quarkus.vertx.web.cors; \ No newline at end of file +package io.quarkus.vertx.web.cors; diff --git a/extensions/reactive-routes/runtime/src/main/java/io/quarkus/vertx/web/ReactiveRoutes.java b/extensions/reactive-routes/runtime/src/main/java/io/quarkus/vertx/web/ReactiveRoutes.java index 28a57b6462ead..e71f044ae5041 100644 --- a/extensions/reactive-routes/runtime/src/main/java/io/quarkus/vertx/web/ReactiveRoutes.java +++ b/extensions/reactive-routes/runtime/src/main/java/io/quarkus/vertx/web/ReactiveRoutes.java @@ -85,7 +85,7 @@ private ReactiveRoutes() { } /** - * Indicates the the given stream should be written as server-sent-event in the response. + * Indicates that the given stream should be written as server-sent-event in the response. * Returning a {@code multi} wrapped using this method produces a {@code text/event-stream} response. Each item * is written as an event in the response. The response automatically enables the chunked encoding and set the * content type. @@ -125,9 +125,9 @@ public static Multi asEventStream(Multi multi) { } /** - * Indicates the the given stream should be written as a Json stream in the response. + * Indicates that the given stream should be written as a Json stream in the response. * Returning a {@code multi} wrapped using this method produces a {@code application/x-ndjson} response. Each item - * is written as an serialized json on a new line in the response. The response automatically enables the chunked + * is written as a serialized json on a new line in the response. The response automatically enables the chunked * encoding and set the content type. *

    * If the item is a String, the content will be wrapped in quotes and written. @@ -165,7 +165,7 @@ public static Multi asJsonStream(Multi multi) { } /** - * Indicates the the given stream should be written as a chunked JSON array in the response. + * Indicates that the given stream should be written as a chunked JSON array in the response. * Returning a {@code multi} wrapped using this method produces a {@code application/json} response. Each item * is written as an JSON object in the response. The response automatically enables the chunked encoding and set the * content type. @@ -201,7 +201,7 @@ public static Multi asJsonArray(Multi multi) { } /** - * A class allowing to customized how the server sent events are written. + * A class allowing to customize how the server sent events are written. *

    * The {@code data} section of the resulting event is the JSON representation of the result from {@link #data()}. * If {@link #event()} does not return {@code null}, the {@code event} section is written with the result as value. diff --git a/extensions/reactive-streams-operators/smallrye-reactive-type-converters/deployment/src/main/java/io/quarkus/smallrye/typeconverters/deployment/SmallRyeReactiveTypeConvertersProcessor.java b/extensions/reactive-streams-operators/smallrye-reactive-type-converters/deployment/src/main/java/io/quarkus/smallrye/typeconverters/deployment/SmallRyeReactiveTypeConvertersProcessor.java index 64d3aa75e469e..e59f8254d44f8 100644 --- a/extensions/reactive-streams-operators/smallrye-reactive-type-converters/deployment/src/main/java/io/quarkus/smallrye/typeconverters/deployment/SmallRyeReactiveTypeConvertersProcessor.java +++ b/extensions/reactive-streams-operators/smallrye-reactive-type-converters/deployment/src/main/java/io/quarkus/smallrye/typeconverters/deployment/SmallRyeReactiveTypeConvertersProcessor.java @@ -15,7 +15,7 @@ /** * Searches for implementations of the {@link ReactiveTypeConverter} class and register them as SPI. So the result depends - * of the implementation added by the user in the build classpath (Maven dependencies). + * on the implementation added by the user in the build classpath (Maven dependencies). * * Note that if none are found, nothing is added - so declaring this augmentation is quite useless in this case. */ diff --git a/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/DevServicesConfig.java b/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/DevServicesConfig.java index 0a6fcb9c87206..42ceef8ce6b00 100644 --- a/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/DevServicesConfig.java +++ b/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/DevServicesConfig.java @@ -53,7 +53,7 @@ public class DevServicesConfig { * This property is used when {@code shared} is set to {@code true}. * In this case, before starting a container, Dev Services for Redis looks for a container with the * {@code quarkus-dev-service-redis} label - * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise it + * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise, it * starts a new container with the {@code quarkus-dev-service-redis} label set to the specified value. *

    * This property is used when you need multiple shared Redis servers. diff --git a/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/RedisBuildTimeConfig.java b/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/RedisBuildTimeConfig.java index 8750192b3170c..1cc335208f82b 100644 --- a/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/RedisBuildTimeConfig.java +++ b/extensions/redis-client/deployment/src/main/java/io/quarkus/redis/client/deployment/RedisBuildTimeConfig.java @@ -11,7 +11,7 @@ @ConfigRoot public class RedisBuildTimeConfig { /** - * Whether or not an health check is published in case the smallrye-health extension is present. + * Whether a health check is published in case the smallrye-health extension is present. */ @ConfigItem(name = "health.enabled", defaultValue = "true") public boolean healthEnabled; diff --git a/extensions/redis-client/runtime/src/main/java/io/quarkus/redis/client/runtime/RedisConfig.java b/extensions/redis-client/runtime/src/main/java/io/quarkus/redis/client/runtime/RedisConfig.java index f4b5f2ae74b29..18fefefb919df 100644 --- a/extensions/redis-client/runtime/src/main/java/io/quarkus/redis/client/runtime/RedisConfig.java +++ b/extensions/redis-client/runtime/src/main/java/io/quarkus/redis/client/runtime/RedisConfig.java @@ -106,7 +106,7 @@ public static class RedisConfiguration { public Optional role; /** - * Whether or not to use replicas nodes (only considered in Cluster mode). + * Whether to use replicas nodes (only considered in Cluster mode). */ @ConfigItem(defaultValueDocumentation = "never") public Optional replicas; @@ -146,7 +146,7 @@ public static class RedisConfiguration { public Duration poolRecycleTimeout; /** - * Sets how much handlers is the client willing to queue. + * Sets how many handlers is the client willing to queue. *

    * The client will always work on pipeline mode, this means that messages can start queueing. * Using this configuration option, you can control how much backlog you're willing to accept. diff --git a/extensions/resteasy-classic/rest-client/deployment/src/main/java/io/quarkus/restclient/deployment/RestClientProcessor.java b/extensions/resteasy-classic/rest-client/deployment/src/main/java/io/quarkus/restclient/deployment/RestClientProcessor.java index a8e322340206d..f188a04f08191 100644 --- a/extensions/resteasy-classic/rest-client/deployment/src/main/java/io/quarkus/restclient/deployment/RestClientProcessor.java +++ b/extensions/resteasy-classic/rest-client/deployment/src/main/java/io/quarkus/restclient/deployment/RestClientProcessor.java @@ -306,7 +306,7 @@ ExtensionSslNativeSupportBuildItem activateSslNativeSupport() { // currently default methods on a rest-client interface // that is annotated with ClientHeaderParam // leads to NPEs (see https://github.com/quarkusio/quarkus/issues/10249) - // so lets warn users about its use + // so let's warn users about its use private void warnAboutNotWorkingFeaturesInNative(PackageConfig packageConfig, Map interfaces) { if (!packageConfig.type.equalsIgnoreCase(PackageConfig.NATIVE)) { return; diff --git a/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/configuration/RestClientOverrideRuntimeConfigTest.java b/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/configuration/RestClientOverrideRuntimeConfigTest.java index 7b9ea2443da4f..73a64721c38d5 100644 --- a/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/configuration/RestClientOverrideRuntimeConfigTest.java +++ b/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/configuration/RestClientOverrideRuntimeConfigTest.java @@ -59,7 +59,7 @@ void overrideConfig() { .getConfigValue("quarkus.rest-client.\"io.quarkus.restclient.configuration.EchoClient\".url"); assertEquals(mpValue.getValue(), quarkusValue.getValue()); assertEquals(RestClientRunTimeConfigSource.class.getName(), quarkusValue.getConfigSourceName()); - // The MP name has priority over the Quarkus one, so it is the name we get (even when we lookup the quarkus one) + // The MP name has priority over the Quarkus one, so it is the name we get (even when we look up the quarkus one) assertEquals(mpValue.getName(), "io.quarkus.restclient.configuration.EchoClient/mp-rest/url"); assertEquals(quarkusValue.getName(), "io.quarkus.restclient.configuration.EchoClient/mp-rest/url"); diff --git a/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/registerprovider/ProviderClientRegistrationTest.java b/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/registerprovider/ProviderClientRegistrationTest.java index 35a7fdfc9a311..fa65edd513e12 100644 --- a/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/registerprovider/ProviderClientRegistrationTest.java +++ b/extensions/resteasy-classic/rest-client/deployment/src/test/java/io/quarkus/restclient/registerprovider/ProviderClientRegistrationTest.java @@ -80,7 +80,7 @@ void helloFoo() { assertEquals("Hello Foo", client.hello()); } - // The Client needs to be deployed with the Quarkus app, so it inherits all of the RESTEasy configuration. + // The Client needs to be deployed with the Quarkus app, so it inherits all the RESTEasy configuration. @ApplicationScoped public static class ClientBean { private WebTarget webTarget; diff --git a/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/QuarkusRestClientBuilder.java b/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/QuarkusRestClientBuilder.java index 3e1fdb2685a3f..5a8136fd83e28 100644 --- a/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/QuarkusRestClientBuilder.java +++ b/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/QuarkusRestClientBuilder.java @@ -368,7 +368,7 @@ private void configureTrustAll(ResteasyClientBuilder clientBuilder) { } /** - * Determines whether or not to default to using the URLConnection instead of the Apache HTTP Client. + * Determines whether to default to using the URLConnection instead of the Apache HTTP Client. * If the {@code org.jboss.resteasy.microprofile.defaultToURLConnectionHttpClient} system property is {@code true}, * then this method returns {@code true}. In all other cases it returns {@code false} */ diff --git a/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/graal/ClientHttpEngineBuilder43Replacement.java b/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/graal/ClientHttpEngineBuilder43Replacement.java index 0fe36db78502b..7fba4adb63931 100644 --- a/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/graal/ClientHttpEngineBuilder43Replacement.java +++ b/extensions/resteasy-classic/rest-client/runtime/src/main/java/io/quarkus/restclient/runtime/graal/ClientHttpEngineBuilder43Replacement.java @@ -12,7 +12,7 @@ /** * This class is used to make sure that only the default SSLContext is requested when no SSLContext has been provided. - * The reason this is necessary is because by default, the code requests + * The reason this is necessary because, by default, the code requests * {@code SSLContext.getInstance(SSLConnectionSocketFactory.TLS)} which will fail in native when the SSL has been disabled */ @TargetClass(className = "org.jboss.resteasy.client.jaxrs.engines.ClientHttpEngineBuilder43") diff --git a/extensions/resteasy-classic/resteasy-common/deployment/src/main/java/io/quarkus/resteasy/common/deployment/ResteasyCommonProcessor.java b/extensions/resteasy-classic/resteasy-common/deployment/src/main/java/io/quarkus/resteasy/common/deployment/ResteasyCommonProcessor.java index 82e134ca087c3..4473273519dd4 100644 --- a/extensions/resteasy-classic/resteasy-common/deployment/src/main/java/io/quarkus/resteasy/common/deployment/ResteasyCommonProcessor.java +++ b/extensions/resteasy-classic/resteasy-common/deployment/src/main/java/io/quarkus/resteasy/common/deployment/ResteasyCommonProcessor.java @@ -514,7 +514,7 @@ private static boolean collectDeclaredProviders(List restCl } } - // handle @PartType: we don't know if it's used for writing or reading so we register both + // handle @PartType: we don't know if it's used for writing or reading, so we register both for (AnnotationInstance partTypeAnnotation : index.getAnnotations(ResteasyDotNames.RESTEASY_PART_TYPE)) { try { MediaType partTypeMediaType = MediaType.valueOf(partTypeAnnotation.value().asString()); @@ -605,7 +605,7 @@ private static Collection collectInferredProviders(final MediaType media // if it is, then include a provider which can handle that element type. // - if no @SseElementType is present, check if the media type has the "element-type" parameter // and if it does then include the provider which can handle that element-type - // - if neither of the above specifies an element-type then we by fallback to including text/plain + // - if neither of the above specifies an element-type then we fall back to including text/plain // provider as a default if (matches(MediaType.SERVER_SENT_EVENTS_TYPE, mediaType)) { final Set additionalProvidersToRegister = new HashSet<>(); @@ -623,7 +623,7 @@ private static Collection collectInferredProviders(final MediaType media if (elementType != null) { additionalProvidersToRegister.addAll(categorizedProviders.getPossible(MediaType.valueOf(elementType))); } else { - // add text/plain provider as a fallback default for SSE mediatype + // add text/plain provider as a fallback default for SSE media-type additionalProvidersToRegister.addAll(categorizedProviders.getPossible(MediaType.TEXT_PLAIN_TYPE)); } return additionalProvidersToRegister; @@ -633,7 +633,7 @@ private static Collection collectInferredProviders(final MediaType media /** * Compares the {@link MediaType#getType() type} and the {@link MediaType#getSubtype() subtype} to see if they are - * equal (case insensitive). If they are equal, then this method returns {@code true}, else returns {@code false}. + * equal (case-insensitive). If they are equal, then this method returns {@code true}, else returns {@code false}. * Unlike the {@link MediaType#equals(Object)}, this method doesn't take into account the {@link MediaType#getParameters() * parameters} during the equality check * diff --git a/extensions/resteasy-classic/resteasy-links/runtime/pom.xml b/extensions/resteasy-classic/resteasy-links/runtime/pom.xml index 853a37416adb3..0174bba700346 100644 --- a/extensions/resteasy-classic/resteasy-links/runtime/pom.xml +++ b/extensions/resteasy-classic/resteasy-links/runtime/pom.xml @@ -24,7 +24,7 @@ resteasy-links diff --git a/extensions/resteasy-classic/resteasy-server-common/deployment/src/main/java/io/quarkus/resteasy/server/common/deployment/ResteasyServerCommonProcessor.java b/extensions/resteasy-classic/resteasy-server-common/deployment/src/main/java/io/quarkus/resteasy/server/common/deployment/ResteasyServerCommonProcessor.java index 28a25a2a379f6..c8e8cc34f4edc 100644 --- a/extensions/resteasy-classic/resteasy-server-common/deployment/src/main/java/io/quarkus/resteasy/server/common/deployment/ResteasyServerCommonProcessor.java +++ b/extensions/resteasy-classic/resteasy-server-common/deployment/src/main/java/io/quarkus/resteasy/server/common/deployment/ResteasyServerCommonProcessor.java @@ -153,7 +153,7 @@ static final class ResteasyConfig { String path; /** - * Whether or not detailed JAX-RS metrics should be enabled if the smallrye-metrics + * Whether detailed JAX-RS metrics should be enabled if the smallrye-metrics * extension is present. *

    * See getExcludedClasses(List * @param annotationInstance the annotation instance to test. * @return {@code true} if the enclosing class of the annotation is a concrete class and is part of the allowed * classes, or is an interface and at least one concrete implementation is included, or is an abstract class - * and at least one concrete sub class is included, or is not part of the excluded classes, {@code false} otherwise. + * and at least one concrete subclass is included, or is not part of the excluded classes, {@code false} otherwise. */ private static boolean keepAnnotation(IndexView index, Set allowedClasses, Set excludedClasses, AnnotationInstance annotationInstance) { @@ -934,7 +934,7 @@ private static boolean keepAnnotation(IndexView index, Set allowedClasse // Keep the enclosing class only if not excluded return !excludedClasses.contains(className); } else if (Modifier.isAbstract(classInfo.flags())) { - // Only keep the annotation if a concrete implementation or a sub class has been included + // Only keep the annotation if a concrete implementation or a subclass has been included return (Modifier.isInterface(classInfo.flags()) ? index.getAllKnownImplementors(classInfo.name()) : index.getAllKnownSubclasses(classInfo.name())) .stream() @@ -954,7 +954,7 @@ private static boolean keepAnnotation(IndexView index, Set allowedClasse * filtered * @return an instance of {@link JaxrsProvidersToRegisterBuildItem} that has been filtered to take into account * the classes returned by the methods {@link Application#getClasses()} and {@link Application#getSingletons()} - * if at least one of those methods return a non empty {@code Set}, the provided instance of + * if at least one of those methods return a non-empty {@code Set}, the provided instance of * {@link JaxrsProvidersToRegisterBuildItem} otherwise. */ private static JaxrsProvidersToRegisterBuildItem getFilteredJaxrsProvidersToRegisterBuildItem( diff --git a/extensions/resteasy-classic/resteasy-server-common/runtime/src/main/java/io/quarkus/resteasy/server/common/runtime/RESTEasyTestHttpProvider.java b/extensions/resteasy-classic/resteasy-server-common/runtime/src/main/java/io/quarkus/resteasy/server/common/runtime/RESTEasyTestHttpProvider.java index a0b405e4a721b..554830b905fb3 100644 --- a/extensions/resteasy-classic/resteasy-server-common/runtime/src/main/java/io/quarkus/resteasy/server/common/runtime/RESTEasyTestHttpProvider.java +++ b/extensions/resteasy-classic/resteasy-server-common/runtime/src/main/java/io/quarkus/resteasy/server/common/runtime/RESTEasyTestHttpProvider.java @@ -24,7 +24,7 @@ public String apply(Class aClass) { if (value.startsWith("/")) { value = value.substring(1); } - //TODO: there is not really any way to handle @ApplicationPath, we could do something for @QuarkusTest apps but we can't for + //TODO: there is not really any way to handle @ApplicationPath, we could do something for @QuarkusTest apps, but we can't for //native apps, so we just have to document the limitation String path = "/"; Optional appPath = ConfigProvider.getConfig().getOptionalValue("quarkus.resteasy.path", String.class); diff --git a/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/ReadTimeoutTestCase.java b/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/ReadTimeoutTestCase.java index 6b172f79597a7..4f304c03a4121 100644 --- a/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/ReadTimeoutTestCase.java +++ b/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/ReadTimeoutTestCase.java @@ -31,7 +31,7 @@ public void testReadTimesOut() throws Exception { PostEndpoint.invoked = false; //make sure incomplete writes do not block threads - //and that incoplete data is not delivered to the endpoint + //and that incomplete data is not delivered to the endpoint Socket socket = new Socket(url.getHost(), url.getPort()); socket.getOutputStream().write( "POST /post HTTP/1.1\r\nHost: localhost\r\nContent-length:9\r\n\r\n12345".getBytes(StandardCharsets.UTF_8)); @@ -51,7 +51,7 @@ public void testReadSuccess() throws Exception { PostEndpoint.invoked = false; //make sure incomplete writes do not block threads - //and that incoplete data is not delivered to the endpoint + //and that incomplete data is not delivered to the endpoint Socket socket = new Socket(url.getHost(), url.getPort()); socket.getOutputStream().write( "POST /post HTTP/1.1\r\nHost: localhost\r\nContent-length:9\r\n\r\n12345".getBytes(StandardCharsets.UTF_8)); diff --git a/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/files/StaticFileWithResourcesTest.java b/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/files/StaticFileWithResourcesTest.java index b97854e6c72a3..2fb04c90ed9f9 100644 --- a/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/files/StaticFileWithResourcesTest.java +++ b/extensions/resteasy-classic/resteasy/deployment/src/test/java/io/quarkus/resteasy/test/files/StaticFileWithResourcesTest.java @@ -66,7 +66,7 @@ public void testMultipleThreadedRequestWithIfModifiedSince() throws Exception { // RFC1123 date formatter final DateFormat dtf = new SimpleDateFormat("EEE, dd MMM yyyy HH:mm:ss zzz", Locale.ENGLISH); dtf.setTimeZone(TimeZone.getTimeZone("GMT")); - // date in past, so that we always get a 200 response, instead of 304 + // date in the past, so that we always get a 200 response, instead of 304 final Date fiveMinInPast = new Date(System.currentTimeMillis() - (5 * 60 * 1000)); final String modifiedSinceHeader = dtf.format(fiveMinInPast); final int numRequests = 10; diff --git a/extensions/resteasy-classic/resteasy/runtime/src/main/java/io/quarkus/resteasy/runtime/ContextUtil.java b/extensions/resteasy-classic/resteasy/runtime/src/main/java/io/quarkus/resteasy/runtime/ContextUtil.java index ab1d1e45b82ac..efe32e31b053e 100644 --- a/extensions/resteasy-classic/resteasy/runtime/src/main/java/io/quarkus/resteasy/runtime/ContextUtil.java +++ b/extensions/resteasy-classic/resteasy/runtime/src/main/java/io/quarkus/resteasy/runtime/ContextUtil.java @@ -11,7 +11,7 @@ public class ContextUtil { /** * Used to push context objects from virtual http plugins like AWS Lambda and Azure Functions. - * We keep this code here because its used in multiple places. + * We keep this code here because it is used in multiple places. * * @param routingContext */ diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-common/runtime/src/main/java/io/quarkus/resteasy/reactive/common/runtime/ResteasyReactiveConfig.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive-common/runtime/src/main/java/io/quarkus/resteasy/reactive/common/runtime/ResteasyReactiveConfig.java index 61ea3d720d2e6..f1f4f1369f1a0 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-common/runtime/src/main/java/io/quarkus/resteasy/reactive/common/runtime/ResteasyReactiveConfig.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-common/runtime/src/main/java/io/quarkus/resteasy/reactive/common/runtime/ResteasyReactiveConfig.java @@ -26,7 +26,7 @@ public class ResteasyReactiveConfig { public int outputBufferSize; /** - * By default we assume a default produced media type of "text/plain" + * By default, we assume a default produced media type of "text/plain" * for String endpoint return types. If this is disabled, the default * produced media type will be "[text/plain, */*]" which is more * expensive due to negotiation. @@ -45,14 +45,14 @@ public class ResteasyReactiveConfig { public boolean defaultProduces; /** - * Whether or not annotations such `@IfBuildTimeProfile`, `@IfBuildTimeProperty` and friends will be taken + * Whether annotations such `@IfBuildTimeProfile`, `@IfBuildTimeProperty` and friends will be taken * into account when used on JAX-RS classes. */ @ConfigItem(defaultValue = "true") public boolean buildTimeConditionAware; /** - * Whether or not duplicate endpoints should trigger error at startup + * Whether duplicate endpoints should trigger error at startup */ @ConfigItem(defaultValue = "true") public boolean failOnDuplicate; diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/deployment/src/main/java/io/quarkus/resteasy/reactive/jackson/deployment/processor/ResteasyReactiveJacksonProcessor.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/deployment/src/main/java/io/quarkus/resteasy/reactive/jackson/deployment/processor/ResteasyReactiveJacksonProcessor.java index 41f653aa5016c..9d4f70e359973 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/deployment/src/main/java/io/quarkus/resteasy/reactive/jackson/deployment/processor/ResteasyReactiveJacksonProcessor.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/deployment/src/main/java/io/quarkus/resteasy/reactive/jackson/deployment/processor/ResteasyReactiveJacksonProcessor.java @@ -116,7 +116,7 @@ AdditionalBeanBuildItem beans() { .addBeanClass(BasicServerJacksonMessageBodyWriter.class) // This will not be needed in most cases, but it will not be involved in serialization // just because it's a bean. - // Whether or not it is used in RESTEasy Reactive is determined elsewhere + // Whether it is used in RESTEasy Reactive is determined elsewhere .addBeanClass(FullyFeaturedServerJacksonMessageBodyWriter.class) .setUnremovable().build(); } diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/runtime/src/main/java/io/quarkus/resteasy/reactive/jackson/CustomSerialization.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/runtime/src/main/java/io/quarkus/resteasy/reactive/jackson/CustomSerialization.java index db8aa8243a9af..ee77dcba83901 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/runtime/src/main/java/io/quarkus/resteasy/reactive/jackson/CustomSerialization.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-jackson/runtime/src/main/java/io/quarkus/resteasy/reactive/jackson/CustomSerialization.java @@ -32,7 +32,7 @@ * method. * * The class MUST contain a no-args constructor and it is advisable that it contains no state that is updated outside - * of its constructor. + * its constructor. * Furthermore, the {@code ObjectMapper} should NEVER be changed any way as it is the global ObjectMapper that is * accessible to the entire Quarkus application. */ diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-jsonb/deployment/src/test/java/io/quarkus/resteasy/reactive/jsonb/deployment/test/sse/SseParserTest.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive-jsonb/deployment/src/test/java/io/quarkus/resteasy/reactive/jsonb/deployment/test/sse/SseParserTest.java index 9836e508e0ea7..cd018c190b115 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-jsonb/deployment/src/test/java/io/quarkus/resteasy/reactive/jsonb/deployment/test/sse/SseParserTest.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-jsonb/deployment/src/test/java/io/quarkus/resteasy/reactive/jsonb/deployment/test/sse/SseParserTest.java @@ -108,7 +108,7 @@ public void testParser() { testParser(Arrays.asList("data:f", "oo\n\n"), Collections.singletonList(new InboundSseEventImpl(null, null) .setData("foo"))); - // one event in two buffers within a utf-8 char + // one event in two buffers within a UTF-8 char testParserWithBytes( Arrays.asList(new byte[] { 'd', 'a', 't', 'a', ':', (byte) 0b11000010 }, new byte[] { (byte) 0b10100010, '\n', '\n' }), @@ -169,7 +169,7 @@ private void testParserWithBytes(List events, List expe for (byte[] event : events) { parser.handle(Buffer.buffer(event)); } - // this is really synchronous, so we can't timeout + // this is really synchronous, so we can't time out try { latch.await(1, TimeUnit.MILLISECONDS); } catch (InterruptedException e) { diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive-links/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksContainerFactory.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive-links/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksContainerFactory.java index b85bb99f3d4d6..452b527ccb1e1 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive-links/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksContainerFactory.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive-links/deployment/src/main/java/io/quarkus/resteasy/reactive/links/deployment/LinksContainerFactory.java @@ -39,7 +39,7 @@ final class LinksContainerFactory { private static final String ADD = "add"; /** - * Find the resource methods that are marked with a {@link RestLink} annotations and add them to a links container. + * Find the resource methods that are marked with a {@link RestLink} annotations and add them to a {@link LinksContainer}. */ LinksContainer getLinksContainer(List entries, IndexView index) { LinksContainer linksContainer = new LinksContainer(); diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/FormDataOutputMapperGenerator.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/FormDataOutputMapperGenerator.java index 4b893a64cffb7..b0ab3d49ff258 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/FormDataOutputMapperGenerator.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/FormDataOutputMapperGenerator.java @@ -183,7 +183,7 @@ static String generate(ClassInfo returnTypeClassInfo, ClassOutput classOutput, I formAttrName = formParamValue.asString(); } - // TODO: not sure this is correct, but it seems to be what RESTEasy does and it also makes most sense in the context of a POJO + // TODO: not sure if this is correct, but it seems to be what RESTEasy does and it also makes most sense in the context of a POJO String partType = MediaType.TEXT_PLAIN; AnnotationInstance partTypeInstance = field.annotation(ResteasyReactiveDotNames.PART_TYPE_NAME); if (partTypeInstance != null) { diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/ResteasyReactiveProcessor.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/ResteasyReactiveProcessor.java index 478f45e3e653e..f94d94f315a31 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/ResteasyReactiveProcessor.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/main/java/io/quarkus/resteasy/reactive/server/deployment/ResteasyReactiveProcessor.java @@ -618,7 +618,7 @@ public Status isJava19OrHigher() { if (endpoints.isPresent()) { subResourceClasses.add(endpoints.get()); } - //we need to also look for all sub classes and interfaces + //we need to also look for all subclasses and interfaces //they may have type variables that need to be handled toScan.addAll(index.getKnownDirectImplementors(classInfo.name())); toScan.addAll(index.getKnownDirectSubclasses(classInfo.name())); diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/resource/basic/DefaultMediaTypeTest.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/resource/basic/DefaultMediaTypeTest.java index 54e96271c21bd..aab4f7a61973b 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/resource/basic/DefaultMediaTypeTest.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/resource/basic/DefaultMediaTypeTest.java @@ -28,7 +28,7 @@ /** * @tpSubChapter Resources * @tpChapter Integration tests - * @tpTestCaseDetails Regression test for JBEAP-2847. DefaultTextPlain should be used, if produce annotation annotation is not + * @tpTestCaseDetails Regression test for JBEAP-2847. DefaultTextPlain should be used, if produce annotation is not * used in end-point. * @tpSince RESTEasy 3.0.16 */ diff --git a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/simple/TestUtil.java b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/simple/TestUtil.java index b93f39f6e2490..149a01aa04cd1 100644 --- a/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/simple/TestUtil.java +++ b/extensions/resteasy-reactive/quarkus-resteasy-reactive/deployment/src/test/java/io/quarkus/resteasy/reactive/server/test/simple/TestUtil.java @@ -55,8 +55,8 @@ public static WebArchive prepareArchiveWithApplication(String deploymentName, Cl /** * Finish preparing war deployment and deploy it. * - * Add classes in @resources to deployment. Also all sub-classes of classes in @resources are added to deployment. - * But only classes in @resources (not sub-classes of classes in @resources) can be used as resources + * Add classes in @resources to deployment. Also, all subclasses of classes in @resources are added to deployment. + * But only classes in @resources (not subclasses of classes in @resources) can be used as resources * (getClasses function of TestApplication class return only classes in @resources). * * @param resources classes used in deployment as resources @@ -69,8 +69,8 @@ public static Archive finishContainerPrepare(WebArchive war, Map * The value is parsed with {@link Duration#parse(CharSequence)}. However, if an expression starts with a digit, "PT" prefix * is added automatically, so for example, {@code 15s} can be used instead of {@code PT15S} and is parsed as "15 seconds". diff --git a/extensions/scheduler/deployment/src/test/java/io/quarkus/scheduler/test/metrics/MicrometerTimedTest.java b/extensions/scheduler/deployment/src/test/java/io/quarkus/scheduler/test/metrics/MicrometerTimedTest.java index fdc99f3621b0b..b38d4285121bb 100644 --- a/extensions/scheduler/deployment/src/test/java/io/quarkus/scheduler/test/metrics/MicrometerTimedTest.java +++ b/extensions/scheduler/deployment/src/test/java/io/quarkus/scheduler/test/metrics/MicrometerTimedTest.java @@ -59,8 +59,8 @@ void testTimedMethod() throws InterruptedException { .tag("exception", "none") .timer(); assertNotNull(timer1); - //the count is updated after the method is called - //so we need to use Awaitility as the metric may not be up to date immediately + //the count is updated after the method is called, + //so we need to use Awaitility as the metric may not be up-to-date immediately Awaitility.await().pollInterval(10, TimeUnit.MILLISECONDS) .atMost(2, TimeUnit.SECONDS) .untilAsserted(() -> assertTrue(timer1.count() > 0, "Count=" + timer1.count())); diff --git a/extensions/schema-registry/devservice/deployment/src/main/java/io/quarkus/apicurio/registry/devservice/ApicurioRegistryDevServicesBuildTimeConfig.java b/extensions/schema-registry/devservice/deployment/src/main/java/io/quarkus/apicurio/registry/devservice/ApicurioRegistryDevServicesBuildTimeConfig.java index 55ecf543606a0..215893702e397 100644 --- a/extensions/schema-registry/devservice/deployment/src/main/java/io/quarkus/apicurio/registry/devservice/ApicurioRegistryDevServicesBuildTimeConfig.java +++ b/extensions/schema-registry/devservice/deployment/src/main/java/io/quarkus/apicurio/registry/devservice/ApicurioRegistryDevServicesBuildTimeConfig.java @@ -52,7 +52,7 @@ public class ApicurioRegistryDevServicesBuildTimeConfig { * This property is used when {@code shared} is set to {@code true}. * In this case, before starting a container, Dev Services for Apicurio Registry looks for a container with the * {@code quarkus-dev-service-apicurio-registry} label - * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise it + * set to the configured value. If found, it will use this container instead of starting a new one. Otherwise, it * starts a new container with the {@code quarkus-dev-service-apicurio-registry} label set to the specified value. *

    * This property is used when you need multiple shared Apicurio Registry instances. diff --git a/extensions/security-jpa/deployment/src/test/java/io/quarkus/security/jpa/FormAuthJpaTestCase.java b/extensions/security-jpa/deployment/src/test/java/io/quarkus/security/jpa/FormAuthJpaTestCase.java index a69c617fe5161..55286ae92ec57 100644 --- a/extensions/security-jpa/deployment/src/test/java/io/quarkus/security/jpa/FormAuthJpaTestCase.java +++ b/extensions/security-jpa/deployment/src/test/java/io/quarkus/security/jpa/FormAuthJpaTestCase.java @@ -66,7 +66,7 @@ public void testFormBasedAuthSuccess() { .header("location", containsString("/login")) .cookie("quarkus-redirect-location", containsString("/servlet-secured")); - // test with a non existent user + // test with a non-existent user RestAssured .given() .filter(cookies) diff --git a/extensions/security-jpa/runtime/src/main/java/io/quarkus/security/jpa/Username.java b/extensions/security-jpa/runtime/src/main/java/io/quarkus/security/jpa/Username.java index 888bf28e399a3..d5a71c8f5c6be 100644 --- a/extensions/security-jpa/runtime/src/main/java/io/quarkus/security/jpa/Username.java +++ b/extensions/security-jpa/runtime/src/main/java/io/quarkus/security/jpa/Username.java @@ -7,7 +7,7 @@ import java.lang.annotation.Target; /** - * Indicates that this field or property should be used as a source of user name for security. Only + * Indicates that this field or property should be used as a source of username for security. Only * supports the {@link String} type. */ @Target({ ElementType.FIELD, ElementType.METHOD }) diff --git a/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnAutomaticTest.java b/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnAutomaticTest.java index 86197119a31ce..e7e9c2930e2ae 100644 --- a/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnAutomaticTest.java +++ b/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnAutomaticTest.java @@ -55,7 +55,7 @@ public void test() throws Exception { // reset cookies for the login phase cookieFilter = new CookieFilter(); - // now try to login + // now try to log in challenge = WebAuthnEndpointHelper.invokeLogin("stef", cookieFilter); JsonObject login = hardwareKey.makeLoginJson(challenge); diff --git a/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnManualTest.java b/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnManualTest.java index c58fe69ccb044..fa261dcab99c8 100644 --- a/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnManualTest.java +++ b/extensions/security-webauthn/deployment/src/test/java/io/quarkus/security/webauthn/test/WebAuthnManualTest.java @@ -76,7 +76,7 @@ public void test() throws Exception { // reset cookies for the login phase cookieFilter = new CookieFilter(); - // now try to login + // now try to log in challenge = WebAuthnEndpointHelper.invokeLogin("stef", cookieFilter); JsonObject login = hardwareKey.makeLoginJson(challenge); diff --git a/extensions/security-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnRunTimeConfig.java b/extensions/security-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnRunTimeConfig.java index 4d82c3322b80d..240e66e48f4ae 100644 --- a/extensions/security-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnRunTimeConfig.java +++ b/extensions/security-webauthn/runtime/src/main/java/io/quarkus/security/webauthn/WebAuthnRunTimeConfig.java @@ -24,7 +24,7 @@ public class WebAuthnRunTimeConfig { /** * The origin of the application. The origin is basically protocol, host and port. * - * If your are calling WebAuthn API while your application is located at {@code https://example.com/login}, + * If you are calling WebAuthn API while your application is located at {@code https://example.com/login}, * then origin will be {@code https://example.com}. * * If you are calling from {@code http://localhost:2823/test}, then the origin will be @@ -42,7 +42,7 @@ public class WebAuthnRunTimeConfig { * hardening reasons. * * By default, all transports should be allowed. If your application is to be used by mobile phone users, - * you may want to restrict only the {@code INTERNAL} authenticatior to be allowed. + * you may want to restrict only the {@code INTERNAL} authenticator to be allowed. * * Permitted values are: * @@ -99,7 +99,7 @@ public class WebAuthnRunTimeConfig { public Optional userVerification; /** - * Non negative User Verification timeout. Authentication must occur within the timeout, this will prevent the user + * Non-negative User Verification timeout. Authentication must occur within the timeout, this will prevent the user * browser from being blocked with a pop-up required user verification, and the whole ceremony must be completed * within the timeout period. After the timeout, any previously issued challenge is automatically invalidated. */ @@ -117,7 +117,7 @@ public class WebAuthnRunTimeConfig { * *