[GitHub] [camel-k] jamesnetherton opened a new pull request #1225: fix: Command execution fails with result must be a pointer
jamesnetherton opened a new pull request #1225: fix: Command execution fails with result must be a pointer URL: https://github.com/apache/camel-k/pull/1225 fixes #1221 **Release Note** ```release-note NONE ``` This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] branch master updated: I love the regen's smell in the morning
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new 8cfe3f7 I love the regen's smell in the morning 8cfe3f7 is described below commit 8cfe3f74ff0898433f34d98624de1695db63dae7 Author: Andrea Cosentino AuthorDate: Thu Jan 23 09:11:13 2020 +0100 I love the regen's smell in the morning --- .../camel/builder/endpoint/dsl/InfinispanEndpointBuilderFactory.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/InfinispanEndpointBuilderFactory.java b/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/InfinispanEndpointBuilderFactory.java index 505ed0f..3a45588 100644 --- a/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/InfinispanEndpointBuilderFactory.java +++ b/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/InfinispanEndpointBuilderFactory.java @@ -1287,7 +1287,8 @@ public interface InfinispanEndpointBuilderFactory { COMMAND_RETRY, ROLLING_UPGRADE, REMOTE_ITERATION, -SKIP_SIZE_OPTIMIZATION; +SKIP_SIZE_OPTIMIZATION, +IGNORE_TRANSACTION; } public interface InfinispanBuilders {
[camel] branch master updated: Upgrade Box Java SDK to version 2.44.0
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new 2651db8 Upgrade Box Java SDK to version 2.44.0 2651db8 is described below commit 2651db85b40704b16b73fb8681bb0c038acd7e52 Author: Andrea Cosentino AuthorDate: Thu Jan 23 09:15:14 2020 +0100 Upgrade Box Java SDK to version 2.44.0 --- parent/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/parent/pom.xml b/parent/pom.xml index 51b3d10..4f21882 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -100,7 +100,7 @@ 1.4.6 2.4.0 1.64 -2.43.0 +2.44.0 2.106.0 5.9.1 1.12
[camel-spring-boot] 04/04: regen
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git commit efae5522ee27164e7561efd59a82b1249e094656 Author: Claus Ibsen AuthorDate: Thu Jan 23 09:28:35 2020 +0100 regen --- .../springboot/catalog/components/iec60870-client.json | 1 + .../springboot/catalog/components/iec60870-server.json | 1 + .../camel/springboot/catalog/components/salesforce.json | 1 + camel-spring-boot/src/main/docs/spring-boot.adoc| 5 +++-- components-starter/camel-corda-starter/pom.xml | 12 .../client/springboot/ClientComponentConfiguration.java | 13 + .../springboot/SalesforceComponentConfiguration.java| 12 7 files changed, 31 insertions(+), 14 deletions(-) diff --git a/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-client.json b/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-client.json index 644922a..042bbe1 100644 --- a/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-client.json +++ b/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-client.json @@ -43,6 +43,7 @@ "timeout2": { "kind": "parameter", "displayName": "Timeout2", "group": "connection", "label": "connection", "required": false, "type": "integer", "javaType": "int", "deprecated": false, "secret": false, "defaultValue": 1, "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "Timeout T2 in milliseconds." }, "timeout3": { "kind": "parameter", "displayName": "Timeout3", "group": "connection", "label": "connection", "required": false, "type": "integer", "javaType": "int", "deprecated": false, "secret": false, "defaultValue": 2, "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "Timeout T3 in milliseconds." }, "causeSourceAddress": { "kind": "parameter", "displayName": "Cause Source Address", "group": "data", "label": "data", "required": false, "type": "integer", "javaType": "byte", "deprecated": false, "secret": false, "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "Whether to include the source address" }, +"connectionTimeout": { "kind": "parameter", "displayName": "Connection Timeout", "group": "data", "label": "data", "required": false, "type": "integer", "javaType": "int", "deprecated": false, "secret": false, "defaultValue": 1, "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "Timeout in millis to wait for client to establish a connected connection." }, "ignoreBackgroundScan": { "kind": "parameter", "displayName": "Ignore Background Scan", "group": "data", "label": "data", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": true, "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "Whether background scan transmissions should be ignored." }, "ignoreDaylightSavingTime": { "kind": "parameter", "displayName": "Ignore Daylight Saving Time", "group": "data", "label": "data", "required": false, "type": "boolean", "javaType": "boolean", "deprecated": false, "secret": false, "defaultValue": false, "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "Whether to ignore or respect DST" }, "timeZone": { "kind": "parameter", "displayName": "Time Zone", "group": "data", "label": "data", "required": false, "type": "object", "javaType": "java.util.TimeZone", "deprecated": false, "secret": false, "defaultValue": "UTC", "configurationClass": "org.apache.camel.component.iec60870.client.ClientOptions", "configurationField": "connectionOptions", "description": "The timezone to use. May be any Java time zone string" }, diff --git a/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-server.json b/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-server.json index 2868306..6f2da11 100644 --- a/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec60870-server.json +++ b/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/iec
[camel-spring-boot] 03/04: CAMEL-14426: camel-core - Optimize inflight repository
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git commit c86698a0f0d9c89aa2beb039b4b5ae2085e0ebff Author: Claus Ibsen AuthorDate: Thu Jan 23 08:29:30 2020 +0100 CAMEL-14426: camel-core - Optimize inflight repository --- .../apache/camel/spring/boot/CamelConfigurationProperties.java | 9 + 1 file changed, 9 insertions(+) diff --git a/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java b/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java index 6041683..05f8181 100644 --- a/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java +++ b/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java @@ -109,6 +109,8 @@ public class CamelConfigurationProperties extends DefaultConfigurationProperties /** * Sets whether to log information about the inflight Exchanges which are still running * during a shutdown which didn't complete without the given timeout. + * + * This requires to enable the option inflightRepositoryExchangeEnabled. */ private boolean shutdownLogInflightExchangesOnTimeout = true; @@ -338,6 +340,13 @@ public class CamelConfigurationProperties extends DefaultConfigurationProperties private boolean allowUseOriginalMessage; /** + * Sets whether the inflight repository should track each inflight exchange. + * + * This is by default disabled as there is a very slight performance overhead when enabled. + */ +private boolean inflightRepositoryExchangeEnabled; + +/** * Sets whether endpoint runtime statistics is enabled (gathers runtime usage of each incoming and outgoing endpoints). * * The default value is false.
[camel-spring-boot] 02/04: Regen
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git commit 2fbeafd51845d8a0797cfbc355d5f59132c9c6df Author: Claus Ibsen AuthorDate: Thu Jan 23 08:26:06 2020 +0100 Regen --- tooling/camel-spring-boot-dependencies/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tooling/camel-spring-boot-dependencies/pom.xml b/tooling/camel-spring-boot-dependencies/pom.xml index 358714f..a8f7b26 100644 --- a/tooling/camel-spring-boot-dependencies/pom.xml +++ b/tooling/camel-spring-boot-dependencies/pom.xml @@ -169,7 +169,7 @@ io.methvin directory-watcher -0.9.6 +0.9.9 io.nessus
[camel-spring-boot] branch master updated (25fea2c -> efae552)
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git. from 25fea2c Upgrade Spring Boot to version 2.2.4.RELEASE new a012b6b CAMEL-14421: optimize core - ServiceSupport should not have instance logger. Use static logger instead. new 2fbeafd Regen new c86698a CAMEL-14426: camel-core - Optimize inflight repository new efae552 regen The 4 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: .../springboot/catalog/components/iec60870-client.json | 1 + .../springboot/catalog/components/iec60870-server.json | 1 + .../camel/springboot/catalog/components/salesforce.json | 1 + camel-spring-boot/src/main/docs/spring-boot.adoc| 5 +++-- .../camel/spring/boot/CamelConfigurationProperties.java | 9 + .../spring/boot/CamelSpringBootApplicationListener.java | 4 ++-- .../apache/camel/spring/boot/SpringBootCamelContext.java| 7 ++- components-starter/camel-corda-starter/pom.xml | 12 .../client/springboot/ClientComponentConfiguration.java | 13 + .../springboot/SalesforceComponentConfiguration.java| 12 tooling/camel-spring-boot-dependencies/pom.xml | 2 +- 11 files changed, 49 insertions(+), 18 deletions(-)
[camel-spring-boot] 01/04: CAMEL-14421: optimize core - ServiceSupport should not have instance logger. Use static logger instead.
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git commit a012b6b078422a04183f38909cd0fbad09ad04de Author: Claus Ibsen AuthorDate: Thu Jan 23 08:25:47 2020 +0100 CAMEL-14421: optimize core - ServiceSupport should not have instance logger. Use static logger instead. --- .../camel/spring/boot/CamelSpringBootApplicationListener.java | 4 ++-- .../java/org/apache/camel/spring/boot/SpringBootCamelContext.java | 7 ++- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelSpringBootApplicationListener.java b/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelSpringBootApplicationListener.java index 107d749..58ace8a 100644 --- a/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelSpringBootApplicationListener.java +++ b/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelSpringBootApplicationListener.java @@ -173,11 +173,11 @@ public class CamelSpringBootApplicationListener implements ApplicationListener
[camel-k] branch master updated (acab986 -> 09dc4b8)
This is an automated email from the ASF dual-hosted git repository. astefanutti pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel-k.git. from acab986 Merge pull request #1217 from nicolaferraro/save-config add 09dc4b8 chore(dev): Upgrade Operator SDK to version 0.15.0 No new revisions were added by this update. Summary of changes: go.mod | 2 +- go.sum | 8 +--- script/build_olm.sh | 2 +- 3 files changed, 7 insertions(+), 5 deletions(-)
[GitHub] [camel-k] astefanutti merged pull request #1224: chore(dev): Upgrade Operator SDK to version 0.15.0
astefanutti merged pull request #1224: chore(dev): Upgrade Operator SDK to version 0.15.0 URL: https://github.com/apache/camel-k/pull/1224 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT
lburgazzoli commented on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT URL: https://github.com/apache/camel-quarkus/issues/650#issuecomment-577580484 We can probably workaround that by moving the registration of the message history metrics at runtime through a `MainListener.configure` and check that the `CamelMicroProfileMetricsConfig.enableMessageHistory` and the message history are properly ar in a consistent state, otherwhise we should log a warning This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT
lburgazzoli edited a comment on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT URL: https://github.com/apache/camel-quarkus/issues/650#issuecomment-577580484 We can probably workaround that by moving the registration of the message history metrics at runtime through a `MainListener.configure` and check that the `CamelMicroProfileMetricsConfig.enableMessageHistory` and the message history are in a consistent state, otherwise we should log a warning This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT
lburgazzoli edited a comment on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT URL: https://github.com/apache/camel-quarkus/issues/650#issuecomment-577580484 We can probably fix that by moving the registration of the message history metrics at runtime through a `MainListener.configure` and check that the `CamelMicroProfileMetricsConfig.enableMessageHistory` and the message history are in a consistent state, otherwise we should log a warning. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel-spring-boot] branch master updated: CAMEL-14426: camel-core - Optimize inflight repository
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git The following commit(s) were added to refs/heads/master by this push: new 1c5fa90 CAMEL-14426: camel-core - Optimize inflight repository 1c5fa90 is described below commit 1c5fa9093fb21e1cad2cb8fa62169c9afffb357a Author: Claus Ibsen AuthorDate: Thu Jan 23 09:43:23 2020 +0100 CAMEL-14426: camel-core - Optimize inflight repository --- .../org/apache/camel/spring/boot/CamelConfigurationProperties.java| 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java b/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java index 05f8181..f5b0ce3 100644 --- a/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java +++ b/camel-spring-boot/src/main/java/org/apache/camel/spring/boot/CamelConfigurationProperties.java @@ -340,11 +340,11 @@ public class CamelConfigurationProperties extends DefaultConfigurationProperties private boolean allowUseOriginalMessage; /** - * Sets whether the inflight repository should track each inflight exchange. + * Sets whether the inflight repository should allow browsing each inflight exchange. * * This is by default disabled as there is a very slight performance overhead when enabled. */ -private boolean inflightRepositoryExchangeEnabled; +private boolean inflightRepositoryBrowseEnabled; /** * Sets whether endpoint runtime statistics is enabled (gathers runtime usage of each incoming and outgoing endpoints).
[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT
lburgazzoli edited a comment on issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT URL: https://github.com/apache/camel-quarkus/issues/650#issuecomment-577580484 We can probably fix that by moving the registration of the message history metrics at runtime through a `MainListener.configure` and check that the `CamelMicroProfileMetricsConfig.enableMessageHistory` and the message history are in a consistent state, otherwise we should log a warning. This is something to fix also on master as we should hit the same issue by explicitly turning off the message history This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] branch master updated (2651db8 -> 37fc64f)
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from 2651db8 Upgrade Box Java SDK to version 2.44.0 add 37fc64f CAMEL-14426: camel-core - Optimize inflight repository No new revisions were added by this update. Summary of changes: .../camel/blueprint/CamelContextFactoryBean.java | 8 .../camel/cdi/xml/CamelContextFactoryBean.java | 8 .../apache/camel/spring/CamelContextFactoryBean.java | 10 +- .../org/apache/camel/spi/InflightRepository.java | 10 +- .../camel/impl/engine/DefaultInflightRepository.java | 6 +++--- .../core/xml/AbstractCamelContextFactoryBean.java| 6 +++--- .../impl/InflightRepositoryBrowseFromRouteTest.java | 2 +- .../camel/impl/InflightRepositoryBrowseTest.java | 2 +- .../camel/main/DefaultConfigurationConfigurer.java | 2 +- .../camel/main/DefaultConfigurationProperties.java | 20 ++-- .../META-INF/camel-main-configuration-metadata.json | 6 +++--- .../mbean/ManagedInflightRepositoryMBean.java| 4 ++-- .../management/mbean/ManagedInflightRepository.java | 4 ++-- ...java => ManagedInflightRepositoryBrowseTest.java} | 5 ++--- .../management/ManagedInflightRepositoryTest.java| 7 --- .../management/ManagedInflightStatisticsTest.java| 2 +- .../modules/ROOT/pages/camel-3x-upgrade-guide.adoc | 9 - 17 files changed, 51 insertions(+), 60 deletions(-) copy core/camel-management/src/test/java/org/apache/camel/management/{ManagedInflightRepositoryTest.java => ManagedInflightRepositoryBrowseTest.java} (93%)
[camel-spring-boot] branch master updated (1c5fa90 -> eca2f68)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git. from 1c5fa90 CAMEL-14426: camel-core - Optimize inflight repository new 51c0e45 Camel-Infinispan test: Lets align with correct deps new eca2f68 Camel-Infinispan: Lets ignore itest for the moment The 2 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: .../org/apache/camel/itest/springboot/CamelInfinispanTest.java| 8 1 file changed, 8 insertions(+)
[camel-spring-boot] 01/02: Camel-Infinispan test: Lets align with correct deps
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git commit 51c0e455875531b864394c568eebe0b96e2939f7 Author: Andrea Cosentino AuthorDate: Thu Jan 23 09:51:12 2020 +0100 Camel-Infinispan test: Lets align with correct deps --- .../java/org/apache/camel/itest/springboot/CamelInfinispanTest.java | 6 ++ 1 file changed, 6 insertions(+) diff --git a/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java b/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java index 9b0f45b..6b3f1aa 100644 --- a/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java +++ b/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java @@ -35,6 +35,12 @@ public class CamelInfinispanTest extends AbstractSpringBootTestSupport { public static ITestConfig createTestConfig() { return new ITestConfigBuilder() .module(inferModuleName(CamelInfinispanTest.class)) + .dependency("org.infinispan:infinispan-client-hotrod:10.1.1.Final") +.dependency("org.infinispan:infinispan-commons:10.1.1.Final") + .dependency("org.infinispan:infinispan-component-annotations:10.1.1.Final") +.dependency("org.infinispan:infinispan-core:10.1.1.Final") +.dependency("org.infinispan:infinispan-query-dsl:10.1.1.Final") + .dependency("org.infinispan:infinispan-remote-query-client:10.1.1.Final") .disableJmx("org.infinispan:*") .build(); }
[camel-spring-boot] 02/02: Camel-Infinispan: Lets ignore itest for the moment
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git commit eca2f68868e679051c6773224e4aa288f234301e Author: Andrea Cosentino AuthorDate: Thu Jan 23 09:52:17 2020 +0100 Camel-Infinispan: Lets ignore itest for the moment --- .../java/org/apache/camel/itest/springboot/CamelInfinispanTest.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java b/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java index 6b3f1aa..fe4b62a 100644 --- a/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java +++ b/camel-itest-spring-boot/src/test/java/org/apache/camel/itest/springboot/CamelInfinispanTest.java @@ -21,10 +21,12 @@ import org.jboss.arquillian.container.test.api.Deployment; import org.jboss.arquillian.junit.Arquillian; import org.jboss.shrinkwrap.api.Archive; import org.junit.Test; +import org.junit.Ignore; import org.junit.runner.RunWith; @RunWith(Arquillian.class) +@Ignore("Still doesn't work with infinispan 10.x") public class CamelInfinispanTest extends AbstractSpringBootTestSupport { @Deployment
[camel] branch master updated: CAMEL-14414: for aggregation group completion setting and removal is handled through methods to enforce code cohesion and avoid repetitions. fixed issue for CURRENT_GROU
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new 5b84d35 CAMEL-14414: for aggregation group completion setting and removal is handled through methods to enforce code cohesion and avoid repetitions. fixed issue for CURRENT_GROUP and ALL_GROUPS flag removal. adapted tests and documentation (#3496) 5b84d35 is described below commit 5b84d3597dc58d83d470a86785e8b0bd23e97085 Author: grigoni AuthorDate: Thu Jan 23 09:53:25 2020 +0100 CAMEL-14414: for aggregation group completion setting and removal is handled through methods to enforce code cohesion and avoid repetitions. fixed issue for CURRENT_GROUP and ALL_GROUPS flag removal. adapted tests and documentation (#3496) --- .../LevelDBAggregateForceCompletionHeaderTest.java | 2 +- .../JdbcAggregateForceCompletionHeaderTest.java| 2 +- .../processor/aggregate/AggregateProcessor.java| 55 -- .../src/main/docs/eips/aggregate-eip.adoc | 2 +- ...eCompletionHeaderInAggregationStrategyTest.java | 2 + .../AggregateForceCompletionHeaderTest.java| 3 +- .../AggregationStrategyCompleteByPropertyTest.java | 8 7 files changed, 56 insertions(+), 18 deletions(-) diff --git a/components/camel-leveldb/src/test/java/org/apache/camel/component/leveldb/LevelDBAggregateForceCompletionHeaderTest.java b/components/camel-leveldb/src/test/java/org/apache/camel/component/leveldb/LevelDBAggregateForceCompletionHeaderTest.java index c033dcf..3c2f744 100644 --- a/components/camel-leveldb/src/test/java/org/apache/camel/component/leveldb/LevelDBAggregateForceCompletionHeaderTest.java +++ b/components/camel-leveldb/src/test/java/org/apache/camel/component/leveldb/LevelDBAggregateForceCompletionHeaderTest.java @@ -55,7 +55,7 @@ public class LevelDBAggregateForceCompletionHeaderTest extends CamelTestSupport getMockEndpoint("mock:aggregated").expectedPropertyReceived(Exchange.AGGREGATED_COMPLETED_BY, "force"); //now send the signal message to trigger completion of all groups, message should NOT be aggregated -template.sendBodyAndHeader("direct:start", "test5", Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, true); +template.sendBodyAndProperty("direct:start", "test5", Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, true); assertMockEndpointsSatisfied(); } diff --git a/components/camel-sql/src/test/java/org/apache/camel/processor/aggregate/jdbc/JdbcAggregateForceCompletionHeaderTest.java b/components/camel-sql/src/test/java/org/apache/camel/processor/aggregate/jdbc/JdbcAggregateForceCompletionHeaderTest.java index d26cb59..ed6b157 100644 --- a/components/camel-sql/src/test/java/org/apache/camel/processor/aggregate/jdbc/JdbcAggregateForceCompletionHeaderTest.java +++ b/components/camel-sql/src/test/java/org/apache/camel/processor/aggregate/jdbc/JdbcAggregateForceCompletionHeaderTest.java @@ -45,7 +45,7 @@ public class JdbcAggregateForceCompletionHeaderTest extends AbstractJdbcAggregat getMockEndpoint("mock:aggregated").expectedPropertyReceived(Exchange.AGGREGATED_COMPLETED_BY, "force"); //now send the signal message to trigger completion of all groups, message should NOT be aggregated -template.sendBodyAndHeader("direct:start", "test5", Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, true); +template.sendBodyAndProperty("direct:start", "test5", Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, true); assertMockEndpointsSatisfied(); } diff --git a/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java b/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java index e898e32..83578eb 100644 --- a/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java +++ b/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java @@ -328,7 +328,8 @@ public class AggregateProcessor extends AsyncProcessorSupport implements Navigat } //check for the special header to force completion of all groups (and ignore the exchange otherwise) -if (getAndRemoveBooleanHeader(exchange, Exchange.AGGREGATION_COMPLETE_ALL_GROUPS)) { +if (isCompleteAllGroups(exchange)) { +removeFlagCompleteAllGroups(exchange); forceCompletionOfAllGroups(); callback.done(false); return; @@ -391,10 +392,10 @@ public class AggregateProcessor extends AsyncProcessorSupport implements Navigat // the aggregated output runs in another unit of work Exchange copy = ExchangeHelper.createCorrelatedCopy(exchange, false); -// remove the complete all groups headers as it should not be on the copy -copy.getIn().removeHeader(Exchange
[camel] branch master updated: CAMEL-14414: Updated doc
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new a63be81 CAMEL-14414: Updated doc a63be81 is described below commit a63be81739ff29bcf9118da8f9970d498fbb Author: Claus Ibsen AuthorDate: Thu Jan 23 09:54:16 2020 +0100 CAMEL-14414: Updated doc --- core/camel-core-engine/src/main/docs/eips/aggregate-eip.adoc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/core/camel-core-engine/src/main/docs/eips/aggregate-eip.adoc b/core/camel-core-engine/src/main/docs/eips/aggregate-eip.adoc index 005d6b4..923517d 100644 --- a/core/camel-core-engine/src/main/docs/eips/aggregate-eip.adoc +++ b/core/camel-core-engine/src/main/docs/eips/aggregate-eip.adoc @@ -221,7 +221,7 @@ completion predicates / sizes / timeouts etc, and complete the group. For example the following logic (from an unit test) will complete the group if the message body size is larger than 5. This is done by setting -the property `Exchange.AGGREGATION_COMPLETE_CURRENT_GROUP` to `true`. +the exchange property `Exchange.AGGREGATION_COMPLETE_CURRENT_GROUP` to `true`. [source,java] @@ -278,12 +278,12 @@ setting the property `Exchange.AGGREGATION_COMPLETE_ALL_GROUPS` to `true`. == Manually Force the Completion of All Aggregated Exchanges Immediately You can manually trigger completion of all current aggregated exchanges -by sending an exchange containing the property +by sending an exchange containing the exchange property `Exchange.AGGREGATION_COMPLETE_ALL_GROUPS` set to `true`. The message is considered a signal message only, the message headers/contents will not be processed otherwise. -You can alternatively set the header +You can alternatively set the exchange property `Exchange.AGGREGATION_COMPLETE_ALL_GROUPS_INCLUSIVE` to `true` to trigger completion of all groups after processing the current message.
[camel] branch master updated: CAMEL-14414: Polished
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new c733bd7 CAMEL-14414: Polished c733bd7 is described below commit c733bd71124b6f0e30a1394c80745928dc3f4d11 Author: Claus Ibsen AuthorDate: Thu Jan 23 09:57:23 2020 +0100 CAMEL-14414: Polished --- .../org/apache/camel/processor/aggregate/AggregateProcessor.java| 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java b/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java index 83578eb..42cdae8 100644 --- a/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java +++ b/core/camel-base/src/main/java/org/apache/camel/processor/aggregate/AggregateProcessor.java @@ -434,14 +434,12 @@ public class AggregateProcessor extends AsyncProcessorSupport implements Navigat private Object removeFlagCompleteAllGroups(Exchange exchange) { Object removedHeader = exchange.getIn().removeHeader(Exchange.AGGREGATION_COMPLETE_ALL_GROUPS); Object removedProp = exchange.removeProperty(Exchange.AGGREGATION_COMPLETE_ALL_GROUPS); - return removedHeader == null ? removedProp: removedHeader; } private Boolean isCompleteAllGroups(Exchange exchange) { -Boolean retVal; -retVal = exchange.getIn().getHeader(Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, false, boolean.class); -if(!Boolean.TRUE.equals(retVal)) { +boolean retVal = exchange.getIn().getHeader(Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, false, boolean.class); +if (!retVal) { // according to doc it is a property but it is sometimes read as header // some test don't fail because they use the header expression which contains a fallback to properties retVal = exchange.getProperty(Exchange.AGGREGATION_COMPLETE_ALL_GROUPS, false, boolean.class);
[camel] branch master updated: CAMEL-14426: camel-core - Optimize inflight repository
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new de6ac91 CAMEL-14426: camel-core - Optimize inflight repository de6ac91 is described below commit de6ac9104a2911524b594da9613b16618794af74 Author: Claus Ibsen AuthorDate: Thu Jan 23 10:08:20 2020 +0100 CAMEL-14426: camel-core - Optimize inflight repository --- .../src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java index 334c4ef..e9195b5 100644 --- a/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java +++ b/components/camel-spring/src/main/java/org/apache/camel/spring/CamelContextFactoryBean.java @@ -922,7 +922,7 @@ public class CamelContextFactoryBean extends AbstractCamelContextFactoryBean
[GitHub] [camel-k] nicolaferraro closed issue #1221: kamel kit create returns error - result must be a pointer
nicolaferraro closed issue #1221: kamel kit create returns error - result must be a pointer URL: https://github.com/apache/camel-k/issues/1221 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel-k] branch master updated: fix: Command execution fails with result must be a pointer
This is an automated email from the ASF dual-hosted git repository. nferraro pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-k.git The following commit(s) were added to refs/heads/master by this push: new 8b76567 fix: Command execution fails with result must be a pointer 8b76567 is described below commit 8b7656789dfa38e7b086151bb549c8efc2a4d626 Author: James Netherton AuthorDate: Thu Jan 23 08:10:18 2020 + fix: Command execution fails with result must be a pointer fixes #1221 --- pkg/cmd/describe_kit.go | 2 +- pkg/cmd/describe_platform.go | 2 +- pkg/cmd/kit_create.go| 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/cmd/describe_kit.go b/pkg/cmd/describe_kit.go index 11e9b7b..d0e4c19 100644 --- a/pkg/cmd/describe_kit.go +++ b/pkg/cmd/describe_kit.go @@ -38,7 +38,7 @@ func newDescribeKitCmd(rootCmdOptions *RootCmdOptions) (*cobra.Command, *describ Use: "kit", Short: "Describe an Integration Kit", Long:`Describe an Integration Kit.`, - PreRunE: decode(options), + PreRunE: decode(&options), RunE: func(_ *cobra.Command, args []string) error { if err := options.validate(args); err != nil { return err diff --git a/pkg/cmd/describe_platform.go b/pkg/cmd/describe_platform.go index 0778ca4..5021422 100644 --- a/pkg/cmd/describe_platform.go +++ b/pkg/cmd/describe_platform.go @@ -38,7 +38,7 @@ func newDescribePlatformCmd(rootCmdOptions *RootCmdOptions) (*cobra.Command, *de Use: "platform", Short: "Describe an Integration Platform", Long:`Describe an Integration Platform.`, - PreRunE: decode(options), + PreRunE: decode(&options), RunE: func(_ *cobra.Command, args []string) error { if err := options.validate(args); err != nil { return err diff --git a/pkg/cmd/kit_create.go b/pkg/cmd/kit_create.go index cb35495..cfec037 100644 --- a/pkg/cmd/kit_create.go +++ b/pkg/cmd/kit_create.go @@ -43,7 +43,7 @@ func newKitCreateCmd(rootCmdOptions *RootCmdOptions) (*cobra.Command, *kitCreate Short: "Create an Integration Kit", Long:`Create an Integration Kit.`, Args:options.validateArgs, - PreRunE: decode(options), + PreRunE: decode(&options), RunE:options.run, }
[GitHub] [camel-k] nicolaferraro merged pull request #1225: fix: Command execution fails with result must be a pointer
nicolaferraro merged pull request #1225: fix: Command execution fails with result must be a pointer URL: https://github.com/apache/camel-k/pull/1225 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel-quarkus] branch master updated (7f9ec9c -> 5477f3b)
This is an automated email from the ASF dual-hosted git repository. lburgazzoli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git. from 7f9ec9c Fix #599 Document the need to set quarkus.native.add-all-charsets = true in HTTP extensions add d7ea504 fast-camel-context: don't lookup for custom Injector add 5477f3b factory-finder: include constant among the list of languages to bound to the rigistry No new revisions were added by this update. Summary of changes: .../apache/camel/quarkus/core/deployment/BuildProcessor.java | 5 +++-- .../java/org/apache/camel/quarkus/core/FastCamelContext.java | 2 +- .../src/test/java/org/apache/camel/quarkus/core/CamelTest.java | 10 ++ 3 files changed, 10 insertions(+), 7 deletions(-)
[GitHub] [camel-quarkus] lburgazzoli merged pull request #656: Context and FactoryFinder improvements
lburgazzoli merged pull request #656: Context and FactoryFinder improvements URL: https://github.com/apache/camel-quarkus/pull/656 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli merged pull request #657: factory-finder: move reactive-executor filter to core processor as the related service is always programmatically configured
lburgazzoli merged pull request #657: factory-finder: move reactive-executor filter to core processor as the related service is always programmatically configured URL: https://github.com/apache/camel-quarkus/pull/657 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel-quarkus] branch master updated: factory-finder: move reactive-executor filter to core processor as the related service is always programmatically configured
This is an automated email from the ASF dual-hosted git repository. lburgazzoli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git The following commit(s) were added to refs/heads/master by this push: new 9c4a4f1 factory-finder: move reactive-executor filter to core processor as the related service is always programmatically configured 9c4a4f1 is described below commit 9c4a4f1348dc4a72d9d46ba4f552a50255aed3b2 Author: lburgazzoli AuthorDate: Thu Jan 23 09:55:44 2020 +0100 factory-finder: move reactive-executor filter to core processor as the related service is always programmatically configured --- .../apache/camel/quarkus/core/deployment/BuildProcessor.java | 6 ++ .../quarkus/reactive/executor/deployment/BuildProcessor.java | 11 --- .../test/java/org/apache/camel/quarkus/core/CamelTest.java| 5 + 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/BuildProcessor.java b/extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/BuildProcessor.java index 0797adc..1f27ff7 100644 --- a/extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/BuildProcessor.java +++ b/extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/BuildProcessor.java @@ -125,6 +125,12 @@ class BuildProcessor { void coreServiceFilter(BuildProducer filterBuildItems) { filterBuildItems.produce( new CamelServiceFilterBuildItem(CamelServiceFilter.forService("properties-component-factory"))); + +// The reactive executor is programmatically configured by an extension or +// a default implementation is provided by this processor thus we can safely +// prevent loading of this service. +filterBuildItems.produce( +new CamelServiceFilterBuildItem(CamelServiceFilter.forService("reactive-executor"))); } @BuildStep diff --git a/extensions/reactive-executor/deployment/src/main/java/org/apache/camel/quarkus/reactive/executor/deployment/BuildProcessor.java b/extensions/reactive-executor/deployment/src/main/java/org/apache/camel/quarkus/reactive/executor/deployment/BuildProcessor.java index ebdaf57..77020e9 100644 --- a/extensions/reactive-executor/deployment/src/main/java/org/apache/camel/quarkus/reactive/executor/deployment/BuildProcessor.java +++ b/extensions/reactive-executor/deployment/src/main/java/org/apache/camel/quarkus/reactive/executor/deployment/BuildProcessor.java @@ -22,20 +22,9 @@ import io.quarkus.deployment.annotations.Record; import io.quarkus.vertx.deployment.VertxBuildItem; import org.apache.camel.quarkus.core.Flags; import org.apache.camel.quarkus.core.deployment.CamelReactiveExecutorBuildItem; -import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; -import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; import org.apache.camel.quarkus.reactive.executor.ReactiveExecutorRecorder; public class BuildProcessor { -/* - * The reactive executor is programmatically configured by the extension thus - * we can safely prevent camel-quarkus-core to instantiate a default instance. - */ -@BuildStep -CamelServiceFilterBuildItem serviceFilter() { -return new CamelServiceFilterBuildItem(CamelServiceFilter.forService("reactive-executor")); -} - @Record(value = ExecutionTime.RUNTIME_INIT, optional = true) @BuildStep(onlyIf = Flags.MainEnabled.class) CamelReactiveExecutorBuildItem reactiveExecutor(ReactiveExecutorRecorder recorder, VertxBuildItem vertx) { diff --git a/integration-tests/core-main/src/test/java/org/apache/camel/quarkus/core/CamelTest.java b/integration-tests/core-main/src/test/java/org/apache/camel/quarkus/core/CamelTest.java index 627bbf5..214c35f 100644 --- a/integration-tests/core-main/src/test/java/org/apache/camel/quarkus/core/CamelTest.java +++ b/integration-tests/core-main/src/test/java/org/apache/camel/quarkus/core/CamelTest.java @@ -126,6 +126,11 @@ public class CamelTest { // misc assertThat(factoryFinderMap) .hasKeySatisfying(startsWith("META-INF/services/org/apache/camel/configurer/")); + +// core +assertThat(factoryFinderMap) + .hasKeySatisfying(doesNotStartWith("META-INF/services/org/apache/camel/properties-component-factory")) + .hasKeySatisfying(doesNotStartWith("META-INF/services/org/apache/camel/reactive-executor")); } @Test
[GitHub] [camel-k] jamesnetherton commented on issue #1220: feat: Offer easier access to trait list
jamesnetherton commented on issue #1220: feat: Offer easier access to trait list URL: https://github.com/apache/camel-k/pull/1220#issuecomment-577614733 Pushed some changes to support `kamel help trait` along with the additional args we discussed. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] gsmet commented on issue #658: Update Quarkus to v1.2.0.Final
gsmet commented on issue #658: Update Quarkus to v1.2.0.Final URL: https://github.com/apache/camel-quarkus/pull/658#issuecomment-577615477 Ah ah, I just pushed a branch to create the PR :). This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on issue #647: Fix #518 Rely on configurers for Configuration classes instead of using
ppalaga commented on issue #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#issuecomment-577621881 36102f2 rebased and tested properly This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli opened a new pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli opened a new pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659 Fixes #304 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
svn commit: r37732 - /release/camel/apache-camel/2.25.0/
Author: gzurowski Date: Thu Jan 23 11:15:10 2020 New Revision: 37732 Log: Apache Camel 2.25.0 release distro placeholder. Added: release/camel/apache-camel/2.25.0/
[camel-quarkus] branch master updated: Update Quarkus to v1.2.0.Final
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git The following commit(s) were added to refs/heads/master by this push: new c2ea610 Update Quarkus to v1.2.0.Final new c6b1b27 Merge pull request #658 from lburgazzoli/quarkus-1.2.0.Final c2ea610 is described below commit c2ea610a7dc1fdd3157a4f768303860336dd1b60 Author: lburgazzoli AuthorDate: Thu Jan 23 11:05:39 2020 +0100 Update Quarkus to v1.2.0.Final --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 7935963..4a0bc20 100644 --- a/pom.xml +++ b/pom.xml @@ -43,7 +43,7 @@ 2.10.4 3.0.1 4.1.0 -1.2.0.CR1 +1.2.0.Final 4.1.3 9.4.18.v20190429 1.4.11
[GitHub] [camel-quarkus] oscerd merged pull request #658: Update Quarkus to v1.2.0.Final
oscerd merged pull request #658: Update Quarkus to v1.2.0.Final URL: https://github.com/apache/camel-quarkus/pull/658 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] branch master updated (de6ac91 -> 9eecb92)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from de6ac91 CAMEL-14426: camel-core - Optimize inflight repository add 9eecb92 Upgrade Karaf to version 4.2.8 No new revisions were added by this update. Summary of changes: parent/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[camel] branch master updated (9eecb92 -> f984a85)
This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from 9eecb92 Upgrade Karaf to version 4.2.8 add 35fe272 CAMEL-14427: camel-core - Optimize UnitOfWork route context push/pop add 740fd2c CAMEL-14427: camel-core - Optimize UnitOfWork route context push/pop add f984a85 CAMEL-14426: camel-core - Optimize inflight repository No new revisions were added by this update. Summary of changes: .../camel/cdi/xml/CamelContextFactoryBean.java | 2 +- .../main/java/org/apache/camel/CamelContext.java | 5 ++ .../main/java/org/apache/camel/spi/UnitOfWork.java | 4 +- .../camel/impl/engine/AbstractCamelContext.java| 5 ++ .../camel/impl/engine/DefaultUnitOfWork.java | 25 ++--- .../apache/camel/impl/engine/MDCUnitOfWork.java| 6 +- .../org/apache/camel/processor/SimpleMockTest.java | 4 ++ ...olderTest.java => SimpleMockTwoRoutesTest.java} | 52 -- .../management/mbean/ManagedCamelContext.java | 2 +- .../org/apache/camel/support/UnitOfWorkHelper.java | 64 ++ 10 files changed, 102 insertions(+), 67 deletions(-) copy core/camel-core/src/test/java/org/apache/camel/processor/{SimpleMockPlaceholderTest.java => SimpleMockTwoRoutesTest.java} (53%)
[GitHub] [camel-quarkus] lburgazzoli opened a new pull request #660: The MicroProfile test fails if message history is turned off
lburgazzoli opened a new pull request #660: The MicroProfile test fails if message history is turned off URL: https://github.com/apache/camel-quarkus/pull/660 Fixes #650 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on issue #660: The MicroProfile test fails if message history is turned off
lburgazzoli commented on issue #660: The MicroProfile test fails if message history is turned off URL: https://github.com/apache/camel-quarkus/pull/660#issuecomment-577646452 @ppalaga @jamesnetherton do you mind having a look ? need to be better tested but haven't yet found a easy way to check agains different set of properties without creating ad additional test module This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #660: The MicroProfile test fails if message history is turned off
lburgazzoli edited a comment on issue #660: The MicroProfile test fails if message history is turned off URL: https://github.com/apache/camel-quarkus/pull/660#issuecomment-577646452 @ppalaga @jamesnetherton do you mind having a look ? need to be better tested but haven't yet found a easy way to check agains different set of properties without creating ad additional test module (we can create a new one that tests only in jvm mode) This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370073728 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: would' be better to have a build item generated by the respective extensions ? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370073728 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: would' be better to have a build item generated by the respective extensions ? developer writing custom camel component extension outside this repo could also leverage that build item if needed. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] jamesnetherton commented on a change in pull request #660: The MicroProfile test fails if message history is turned off
jamesnetherton commented on a change in pull request #660: The MicroProfile test fails if message history is turned off URL: https://github.com/apache/camel-quarkus/pull/660#discussion_r370078501 ## File path: extensions/microprofile-metrics/runtime/src/main/java/org/apache/camel/quarkus/component/microprofile/metrics/runtime/CamelMicroProfileMetricsRecorder.java ## @@ -61,4 +63,24 @@ public void configureCamelContext(CamelMicroProfileMetricsConfig config, managementStrategy.addEventNotifier(new MicroProfileMetricsCamelContextEventNotifier()); } } + +private static class MicroProfileMetricsContextConfigurerListener extends MainListenerSupport { +private static final Logger LOGGER = Logger.getLogger(MicroProfileMetricsContextConfigurerListener.class); +private final CamelMicroProfileMetricsConfig config; + +public MicroProfileMetricsContextConfigurerListener(CamelMicroProfileMetricsConfig config) { +this.config = config; +} + +@Override +public void configure(CamelContext camelContext) { +if (camelContext.isMessageHistory() && config.enableMessageHistory) { Review comment: I wonder if we can simplify this to just: ``` camelContext.setMessageHistory(true); camelContext.setMessageHistoryFactory(new MicroProfileMetricsMessageHistoryFactory()); ``` And not bother with the other checks? In 3.1.0, when you add the `MessageHistoryFactory`, message history is [auto enabled](https://github.com/apache/camel/blob/f984a85ced885e0f8da58d3477b703d9cc2ff88c/core/camel-base/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java#L3887) on the `CamelContext` anyway. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
ppalaga commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370086006 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: No, I'd prefer a central location to see better what is whitelisted. I hope the list will gradually disappear. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370086411 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: But external extension won't have a way to make it working This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #660: The MicroProfile test fails if message history is turned off
lburgazzoli commented on a change in pull request #660: The MicroProfile test fails if message history is turned off URL: https://github.com/apache/camel-quarkus/pull/660#discussion_r370087866 ## File path: extensions/microprofile-metrics/runtime/src/main/java/org/apache/camel/quarkus/component/microprofile/metrics/runtime/CamelMicroProfileMetricsRecorder.java ## @@ -61,4 +63,24 @@ public void configureCamelContext(CamelMicroProfileMetricsConfig config, managementStrategy.addEventNotifier(new MicroProfileMetricsCamelContextEventNotifier()); } } + +private static class MicroProfileMetricsContextConfigurerListener extends MainListenerSupport { +private static final Logger LOGGER = Logger.getLogger(MicroProfileMetricsContextConfigurerListener.class); +private final CamelMicroProfileMetricsConfig config; + +public MicroProfileMetricsContextConfigurerListener(CamelMicroProfileMetricsConfig config) { +this.config = config; +} + +@Override +public void configure(CamelContext camelContext) { +if (camelContext.isMessageHistory() && config.enableMessageHistory) { Review comment: It can certainly be done, my only reasoning was: what if the user has explicitly disabled message history ? by adding the extension we would then revert its change without notice. I guess I can add a log that say that the message history has been turned on in case it was not, maybe the same should happen on 3.1.x. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370090912 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", + +/* org.apache.camel.component.consul.* can be removed after the upgrade to Camel 3.1 */ +"org.apache.camel.component.consul.ConsulConfiguration", +"org.apache.camel.component.consul.ConsulClientConfiguration", + "org.apache.camel.component.consul.health.ConsulHealthCheckRepositoryConfiguration", + "org.apache.camel.component.consul.cloud.ConsulServiceRegistryConfiguration")); + +@BuildStep +void bannedReflectiveClasses( +CombinedIndexBuildItem combinedIndex, +List reflectiveClass, +BuildProducer dummy // to force the execution of this method +) { +final DotName uriParamsDotName = DotName.createSimple("org.apache.camel.spi.UriParams"); + +final Set bannedClassNames = combinedIndex.getIndex() +.getAnnotations(uriParamsDotName) +.stream() +.filter(ai -> ai.target().kind() == Kind.CLASS) +.map(ai -> ai.target().asClass().name().toString()) +.collect(Collectors.toSet()); + +Set violations = reflectiveClass.stream() +.map(ReflectiveClassBuildItem::getClassNames) +.flatMap(Collection::stream) +.filter(cl -> !URI_PARAMS_WHITELIST.contains(cl)) +.filter(bannedClassNames::contains) +.collect(Collectors.toSet()); + +if (!violations.isEmpty()) { +throw new IllegalStateException( Review comment: all the camel components have a property that can control if the generated configurer have to be used or not so I feel we should log a warning but I'd rather avoid to fail the build. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370090912 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", + +/* org.apache.camel.component.consul.* can be removed after the upgrade to Camel 3.1 */ +"org.apache.camel.component.consul.ConsulConfiguration", +"org.apache.camel.component.consul.ConsulClientConfiguration", + "org.apache.camel.component.consul.health.ConsulHealthCheckRepositoryConfiguration", + "org.apache.camel.component.consul.cloud.ConsulServiceRegistryConfiguration")); + +@BuildStep +void bannedReflectiveClasses( +CombinedIndexBuildItem combinedIndex, +List reflectiveClass, +BuildProducer dummy // to force the execution of this method +) { +final DotName uriParamsDotName = DotName.createSimple("org.apache.camel.spi.UriParams"); + +final Set bannedClassNames = combinedIndex.getIndex() +.getAnnotations(uriParamsDotName) +.stream() +.filter(ai -> ai.target().kind() == Kind.CLASS) +.map(ai -> ai.target().asClass().name().toString()) +.collect(Collectors.toSet()); + +Set violations = reflectiveClass.stream() +.map(ReflectiveClassBuildItem::getClassNames) +.flatMap(Collection::stream) +.filter(cl -> !URI_PARAMS_WHITELIST.contains(cl)) +.filter(bannedClassNames::contains) +.collect(Collectors.toSet()); + +if (!violations.isEmpty()) { +throw new IllegalStateException( Review comment: all the camel components have a property that can control if the generated configurer have to be used or not so as lonng as the property is there, I feel we should log a warning but I'd rather avoid to fail the build. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370090912 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", + +/* org.apache.camel.component.consul.* can be removed after the upgrade to Camel 3.1 */ +"org.apache.camel.component.consul.ConsulConfiguration", +"org.apache.camel.component.consul.ConsulClientConfiguration", + "org.apache.camel.component.consul.health.ConsulHealthCheckRepositoryConfiguration", + "org.apache.camel.component.consul.cloud.ConsulServiceRegistryConfiguration")); + +@BuildStep +void bannedReflectiveClasses( +CombinedIndexBuildItem combinedIndex, +List reflectiveClass, +BuildProducer dummy // to force the execution of this method +) { +final DotName uriParamsDotName = DotName.createSimple("org.apache.camel.spi.UriParams"); + +final Set bannedClassNames = combinedIndex.getIndex() +.getAnnotations(uriParamsDotName) +.stream() +.filter(ai -> ai.target().kind() == Kind.CLASS) +.map(ai -> ai.target().asClass().name().toString()) +.collect(Collectors.toSet()); + +Set violations = reflectiveClass.stream() +.map(ReflectiveClassBuildItem::getClassNames) +.flatMap(Collection::stream) +.filter(cl -> !URI_PARAMS_WHITELIST.contains(cl)) +.filter(bannedClassNames::contains) +.collect(Collectors.toSet()); + +if (!violations.isEmpty()) { +throw new IllegalStateException( Review comment: all the camel components have a property that can control if the generated configurer have to be used or not so as long as the property is there, I feel we should log a warning but I'd rather avoid to fail the build. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] 01/02: Remove unneeded variables
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git commit 5d45536fb8b6fdfd3417296d1e75f2d32e5d0014 Author: Gregor Zurowski AuthorDate: Thu Jan 23 13:21:11 2020 +0100 Remove unneeded variables --- etc/scripts/release-distro.sh | 8 1 file changed, 8 deletions(-) diff --git a/etc/scripts/release-distro.sh b/etc/scripts/release-distro.sh index 4ca67d7..43d9fa3 100755 --- a/etc/scripts/release-distro.sh +++ b/etc/scripts/release-distro.sh @@ -22,15 +22,7 @@ mkdir -p ${DOWNLOAD} 2>/dev/null # The following component contain schema definitions that must be published RUNDIR=$(cd ${0%/*} && echo $PWD) -COMPLIST=( "camel-spring:spring" - "camel-cxf:cxf" - "camel-osgi:osgi" - "camel-spring-integration:spring/integration" - "camel-spring-security:spring-security" - "camel-blueprint:blueprint" ) DIST_REPO="https://dist.apache.org/repos/dist/release/camel/apache-camel/"; -SITE_DIR="/www/camel.apache.org" - if [ -z "${VERSION}" -o ! -d "${DOWNLOAD}" ] then
[camel] 02/02: Replace md5 and sha1 with sha512 files
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git commit 44453bdd6374ac7f36ba2c71123e95e2aac4dceb Author: Gregor Zurowski AuthorDate: Thu Jan 23 13:37:55 2020 +0100 Replace md5 and sha1 with sha512 files --- etc/scripts/release-distro.sh | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/etc/scripts/release-distro.sh b/etc/scripts/release-distro.sh index 43d9fa3..070c894 100755 --- a/etc/scripts/release-distro.sh +++ b/etc/scripts/release-distro.sh @@ -39,11 +39,21 @@ wget -e robots=off --wait 3 --no-check-certificate \ -r -np "--reject=html,txt" "--follow-tags=" \ -P "${DOWNLOAD}/${VERSION}" -nH "--cut-dirs=3" "--level=1" "--ignore-length" \ "https://repository.apache.org/content/repositories/releases/org/apache/camel/apache-camel/${VERSION}/"; + +DOWNLOAD_LOCATION="${DOWNLOAD}/${VERSION}/org/apache/camel/apache-camel/${VERSION}" + # Remove duplicate signature files -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.asc -# Remove the signature check sum files -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.md5 -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.sha1 +rm "${DOWNLOAD_LOCATION}/"*.asc.asc + +# Remove the md5 and sha1 check sum files +rm "${DOWNLOAD_LOCATION}/"*.md5 +rm "${DOWNLOAD_LOCATION}/"*.sha1 + +# Create sha512 check sum files +cd "${DOWNLOAD_LOCATION}" +for file in *.pom *.tar.gz *.zip; do + sha512sum "${file}" > "${file}.sha512" +done echo "" echo " RESET GROUP PERMISSIONS "
[camel] branch master updated (f984a85 -> 44453bd)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from f984a85 CAMEL-14426: camel-core - Optimize inflight repository new 5d45536 Remove unneeded variables new 44453bd Replace md5 and sha1 with sha512 files The 2 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: etc/scripts/release-distro.sh | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-)
svn commit: r37733 [3/3] - /release/camel/apache-camel/2.25.0/
Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.asc == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.asc (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.asc Thu Jan 23 12:40:53 2020 @@ -0,0 +1,17 @@ +-BEGIN PGP SIGNATURE- +Version: GnuPG v1 + +iQIcBAABCgAGBQJeJJ9lAAoJEJ1b7/pZQsBJ84AQAJDGsz6Z+AlDJ3R6F25cDR6w +PYQjduGfbK7sTOgsvQe6rr0wedLt+JJN/Hzcd9f4X0YZ+/7i/daOiYL0e9WWnLjG +hQCFJ2hDgdfhB//A76I168XHbMU4T+EpTqXskvC5Qdx7WYSWytbeFB9vm7GeSY/R +mV1HlWf5mmtnvcrXxmv1LGET/JrUpJH2rpwPE0AuSB+7xDdETcAR9yBIhpWua4FU +WWQ108p7U/0Qhewmr6aJ/p4PfF1W0MLO5bH1bbrz6DhZl3T/5c5cp74MhXTi66r4 +OI3XQ2AE80wrQZj11g6mdUQ209aMhnwHilotRxk3+uBN7N2KYexVZFoR7ig5VVgO +eIpDMhKA80wbr1/VYKXajIZvvWG6Lt0rsS0R+yIQ1F7yMmDD4YVIypxQvPJMUso8 +cFZetETVxN+2VeKfs4ZqLwvzgDE/rBqNUXQ4xrX4qq6l4mAy1kH9fUEj1Zs6wG/W ++d68Hcg/JPwpuZB6/DLk49r4wQvhOOpvUP3k0m/SADdqocGAvLcAhH9MzxneL/fj +eDHTQCiYj5Y0PIUaZWrb7RoPDRv1Tn3ejjM/zufLQ00tYkp9nC79MJWafsMhXDdd +QUAH8p6Gl/csYZehLxu8t6Wt8vQIZHzPft/fu+96Msk34AFjiT+Q4eq1zwX+DS6v +IZmhgmQ3fXT7V2C8YZ9c +=qGAE +-END PGP SIGNATURE- Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.sha512 == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.sha512 (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.sha512 Thu Jan 23 12:40:53 2020 @@ -0,0 +1 @@ +a5ec31515a8445a66cdde7d91721d5d1b6c2c7561e2af7dd6855f556522e5db500634e747890d0ab0d757ae0515af6dff8f24a69ad2740ddcacb3a011c5821f3 apache-camel-2.25.0.pom Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz == Binary file - no diff available. Propchange: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz -- svn:mime-type = application/octet-stream Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.asc == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.asc (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.asc Thu Jan 23 12:40:53 2020 @@ -0,0 +1,17 @@ +-BEGIN PGP SIGNATURE- +Version: GnuPG v1 + +iQIcBAABCgAGBQJeJJ9lAAoJEJ1b7/pZQsBJlB0P/iDU6ImrfU+guMRi8LL45Dwc +s3zUl3FV2tL3HUOKDczOXj9oiyfBcu4od170kjrKT2N718DRwhidFXDV5rHdOJqI +s+KTMqCikiZVcrI5soASXj5yFYMKep63Xejg8O/N3nAh5LWodMDC4YiVQheNjQGD +/+J6hzakFnbDyZy9iucx6yNx3RZRQEiFyrnpNOXS3iYstfuQvItJCC46zxVnfGm1 +ga8Bw1z0GM0IaKdhOHxXyntPv+CMyHJ8xnoXGyYozyCkcwOTkeazC/R40bi4QDYM +sKbCf7JP0efhwvotmY5Pm08ro8tIYvjaVJUghjGm46x+qt/D+6vxUbuLQx9PzsME +zvw5afE41AaKIsVRtZOf6jgcPmdICpli6DBj3S+TA3JVMTi1TLi2ciSgz2VFSqS1 +efKkU2LeCX9aMIROvVs40n65717FoXLmy0KzV0v0uTnA/E8es8iaaS3P6pEKZ/ek +0fG61LvvjwaDXhhc/QTUKTCpHY20O89yPvfgYtD+3DRLcMW7eK1pPH3tJoATRJVe +so19AqhLC6JRtF6e4FAHF9wXcfxH4m3R5BP4P+T+g1oWoeARAM/v/UhE3cWCJOjH +3MC4+0Vd5OflC4lrNyE+q/jgYhp3Vjg/dTE7xRfN69x5eQPBL0dfhwj4QvZDH7zO +N7vb4KPuQehXQbw0+ld5 +=TS5M +-END PGP SIGNATURE- Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.sha512 == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.sha512 (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.sha512 Thu Jan 23 12:40:53 2020 @@ -0,0 +1 @@ +0928ba08fd8b21ea55ab8dbcca2eed85330d9756bc5b9d604ba5f6f70c6b41b8a2eec2a823c2745f573114ddbd47539ce8ece9587ce908eac7a4e4a53c938073 apache-camel-2.25.0.tar.gz Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip == Binary file - no diff available. Propchange: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip -- svn:mime-type = application/octet-stream Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip.asc == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip.asc (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip.asc Thu Jan 23 12:40:53 2020 @@ -0,0 +1,17 @@ +-BEGIN PGP SIGNATURE- +Version: GnuPG v1 + +iQIcBAABCgAGBQJeJJ9lAAoJEJ1b7/pZQsBJ1gcP/R2qcrL825Oh7eipcPR58q07 +bggA611v7j34LKAik2xFS6HBw4V+3lIRwjJzW5Q+3rtYY66EnEQbnhalqzDLgg8t +lzmoMXx2sub3hsVfW7qSuJ9tKdN2/jgZJnHvXl1NsL4V/mFfcZbOUMrEmmX9qABl +nhCpKn5lDmWZ65LDM8t1qXX9jZKkWBND7kVvE16sfuF5TDuceMRLo+dzTFKjzO5J +QMQyMqvZCsN2w1+bI5GgACQ40b69qmiSSolyL1EZ0bKbv8mHR4liPKexjJ+ufmHk +X6ZtiycpOthdDqk4WSR4mT5SbHtyOgEwaaRYol6d+SEJ9e5mxSXKaBtF7583XZ9G +5oJ3iKbblzDCIgMXmrWt1uENnw8nBlHruVL+YHA1OBBNBC5XB9A/AQ9YYtgHzGF/ +3rPiqoBXWBt6DYXzR41WAn7Mbhc7i1F1+SHeRseGH+Zpfjr0+C/1kVSM
svn commit: r37733 [2/3] - /release/camel/apache-camel/2.25.0/
Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom Thu Jan 23 12:40:53 2020 @@ -0,0 +1,2744 @@ + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + + 4.0.0 + + +org.apache.camel +camel-parent +2.25.0 +../parent + + + apache-camel + Camel :: Assembly + Apache Camel Assembly + pom + + +true +false + +true + + + + + + org.apache.camel + camel-core + + + org.apache.camel + camel-core-osgi + + + org.apache.camel + camel-core-xml + + + org.apache.camel + camel-ahc + + + org.apache.camel + camel-ahc-ws + + + org.apache.camel + camel-amqp + + + org.apache.camel + camel-apns + + + org.apache.camel + camel-asn1 + + + org.apache.camel + camel-asterisk + + + org.apache.camel + camel-atmos + + + org.apache.camel + camel-atmosphere-websocket + + + org.apache.camel + camel-atom + + + org.apache.camel + camel-atomix + + + org.apache.camel + camel-aws + + + org.apache.camel + camel-aws-xray + + + org.apache.camel + camel-azure + + + org.apache.camel + camel-avro + + + org.apache.camel + camel-bam + + + org.apache.camel + camel-base64 + + + org.apache.camel + camel-beanio + + + org.apache.camel + camel-beanstalk + + + org.apache.camel + camel-bean-validator + + + org.apache.camel + camel-barcode + + + org.apache.camel + camel-bindy + + + org.apache.camel + camel-bonita + + + org.apache.camel + camel-box + + + org.apache.camel + camel-box-api + + + org.apache.camel + camel-braintree + + + org.apache.camel + camel-blueprint + + + org.apache.camel + camel-boon + + + org.apache.camel + camel-cache + + + org.apache.camel + camel-caffeine + + + org.apache.camel + camel-cassandraql + + + org.apache.camel + camel-castor + + + org.apache.camel + camel-cdi + + + org.apache.camel + camel-chronicle + + + org.apache.camel + camel-chunk + + + org.apache.camel + camel-cm-sms + + + org.apache.camel + camel-cmis + + + org.apache.camel + camel-coap + + + org.apache.camel + camel-cometd + + + org.apache.camel + camel-connector + + + org.apache.camel + camel-consul + + + org.apache.camel + camel-context + + + org.apache.camel + camel-corda + + + org.apache.camel + camel-couchbase + + + org.apache.camel + camel-couchdb + + + org.apache.camel + camel-crypto + + + org.apache.camel + camel-crypto-cms + + + org.apache.camel + camel-csv + + + org.apache.camel + camel-cxf + + + org.apache.camel + camel-cxf-transport + + + org.apache.camel + camel-digitalocean + + + org.apache.camel + camel-disruptor + + + org.apache.camel + camel-dns + + + org.apache.camel + camel-docker + + + org.apache.camel + camel-dozer + + + org.apache.camel + camel-drill + + + org.apache.camel + camel-dropbox + + + org.apache.camel + camel-eclipse + + + org.apache.camel + camel-ehcache + + + org.apache.camel + camel-ejb + + + org.apache.camel + camel-elasticsearch + + + org.apache.camel + camel-elasticsearch-rest + + + org.apache.camel + camel-elasticsearch5 + + + org.apache.camel + camel-elsql + + + org.apache.camel + camel-etcd + + + org.apache.camel + camel-eventadmin + + + org.apache.camel + camel-exec + + + org.apache.camel + camel-facebook + + + org.apache.camel + camel-fastjson + + + org.apache.camel + camel-fhir + + + org.apache.camel + camel-flatpack + + + org.apache.camel + camel-flink + + + org.apache.camel + camel-fop +
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #660: The MicroProfile test fails if message history is turned off
lburgazzoli commented on a change in pull request #660: The MicroProfile test fails if message history is turned off URL: https://github.com/apache/camel-quarkus/pull/660#discussion_r370094527 ## File path: extensions/microprofile-metrics/runtime/src/main/java/org/apache/camel/quarkus/component/microprofile/metrics/runtime/CamelMicroProfileMetricsRecorder.java ## @@ -61,4 +63,24 @@ public void configureCamelContext(CamelMicroProfileMetricsConfig config, managementStrategy.addEventNotifier(new MicroProfileMetricsCamelContextEventNotifier()); } } + +private static class MicroProfileMetricsContextConfigurerListener extends MainListenerSupport { +private static final Logger LOGGER = Logger.getLogger(MicroProfileMetricsContextConfigurerListener.class); +private final CamelMicroProfileMetricsConfig config; + +public MicroProfileMetricsContextConfigurerListener(CamelMicroProfileMetricsConfig config) { +this.config = config; +} + +@Override +public void configure(CamelContext camelContext) { +if (camelContext.isMessageHistory() && config.enableMessageHistory) { Review comment: @jamesnetherton I've simplified it a bit, does it look better now ? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
svn commit: r37733 [1/3] - /release/camel/apache-camel/2.25.0/
Author: gzurowski Date: Thu Jan 23 12:40:53 2020 New Revision: 37733 Log: Apache Camel 2.25.0 released artifacts. Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip (with props) release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.asc release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.sha512 release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.asc release/camel/apache-camel/2.25.0/apache-camel-2.25.0.pom.sha512 release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz (with props) release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.asc release/camel/apache-camel/2.25.0/apache-camel-2.25.0.tar.gz.sha512 release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip (with props) release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip.asc release/camel/apache-camel/2.25.0/apache-camel-2.25.0.zip.sha512 Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip == Binary file - no diff available. Propchange: release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip -- svn:mime-type = application/octet-stream Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.asc == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.asc (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.asc Thu Jan 23 12:40:53 2020 @@ -0,0 +1,17 @@ +-BEGIN PGP SIGNATURE- +Version: GnuPG v1 + +iQIcBAABCgAGBQJeJJ9lAAoJEJ1b7/pZQsBJJW0QAJqhPoBgdp3DLBhmfqhCCMxs +1dxZMRaY/wZXO15PEIOlGPwv6+nhL0DvyDpOOck6ZVxgPrjqOGm6ffrJ7Vucd0qn +ds3NI2bTyFCh1O2si5gNEKjOGGKJSmmZDZ4BzZHWrj4yQUaNHmhvlnVIP+UNoRm6 +dn7zN9UpgJTRkAQViSVOyTY/qnTiLpoxwEfHFwdL7QP34/EAQkIy/oFPmHZHApHJ +3gml5TRTGWhfGnTao+KA4wuz92HiyxYBF3eS7z+UQvlN3R81KjYVDqT1iXJKib0/ +BCeun9rORjFngC31BoVqtii5/CM5FhAlmnkG4VJOswLXeUAnAZ0VdjGtxUjvooeL +mQs6WgKXNLAnZ7rKRob46ZL/kPJzrmamwtLd4RVch+kYFN04ti8Nha5JeL0jSlJk +HkljX7Gp4owhz2wRBDb7d4zwJMwwmxYGGlm+P2p9/3/SGdg7nmqZiPeHhsxP/DlK +GYWPZb60AWJXJGdXOQeUWS/apfPtGnJYQCxVpvOV8v1Zn6NTnkZIvVI6UHychcyY +PCWhEMU40OHbduD9QGZURzg7h1I/FBhYc+64KH0c+gMRi3g34ddVfNL+EM9OyjTY +roh3F0GW2HaROHbSxfr8VwjY4fEUlF5hS66jpMsBQ/n9w45rp1JtdmSsuSVk6G+D +SfVivWbSven7k8RzFxCl +=UIFK +-END PGP SIGNATURE- Added: release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.sha512 == --- release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.sha512 (added) +++ release/camel/apache-camel/2.25.0/apache-camel-2.25.0-src.zip.sha512 Thu Jan 23 12:40:53 2020 @@ -0,0 +1 @@ +5ace2c403005480ae169e8ffd0fd457ab63548e265630fd65880049c0894cf1904d1f64c1aee20af966dfbc5c8ea9f2e477567cd56be64e7a253039a68d4fd1a apache-camel-2.25.0-src.zip
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370095777 ## File path: extensions/reactive-streams/runtime/pom.xml ## @@ -0,0 +1,82 @@ + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> +4.0.0 + +org.apache.camel.quarkus +camel-quarkus-reactive-streams-parent +1.1.0-SNAPSHOT +../pom.xml + + +camel-quarkus-reactive-streams +Camel Quarkus :: Reactive Streams :: Runtime + + +1.2.0 Review comment: I think 1.0.0 is going to be the next? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370091382 ## File path: extensions/reactive-streams/deployment/src/main/java/org/apache/camel/quarkus/component/reactive/streams/deployment/ReactiveStreamsProcessor.java ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams.deployment; + +import io.quarkus.arc.deployment.AdditionalBeanBuildItem; +import io.quarkus.arc.deployment.BeanContainerBuildItem; +import io.quarkus.deployment.annotations.BuildProducer; +import io.quarkus.deployment.annotations.BuildStep; +import io.quarkus.deployment.annotations.ExecutionTime; +import io.quarkus.deployment.annotations.Overridable; +import io.quarkus.deployment.annotations.Record; +import io.quarkus.deployment.builditem.FeatureBuildItem; +import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsProducers; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsRecorder; +import org.apache.camel.quarkus.core.Flags; +import org.apache.camel.quarkus.core.deployment.CamelBeanBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelContextBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; + +class ReactiveStreamsProcessor { +private static final String SCHEME = "reactive-streams"; +private static final String FEATURE = "camel-reactive-streams"; + +@BuildStep +FeatureBuildItem feature() { +return new FeatureBuildItem(FEATURE); +} + +@BuildStep +CamelServiceFilterBuildItem serviceFilter() { +return new CamelServiceFilterBuildItem(CamelServiceFilter.forComponent(SCHEME)); +} + +@BuildStep +void reflectiveCLasses(BuildProducer reflectiveClasses) { +reflectiveClasses.produce(new ReflectiveClassBuildItem( +true, +false, + "org.apache.camel.component.reactive.streams.engine.ReactiveStreamsEngineConfiguration")); +} + +@BuildStep(onlyIf = Flags.MainEnabled.class) +void beans(BuildProducer beanProducer) { + beanProducer.produce(AdditionalBeanBuildItem.unremovableOf(ReactiveStreamsProducers.class)); Review comment: Injectable beans should be documented. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370095294 ## File path: extensions/reactive-streams/runtime/src/main/java/org/apache/camel/quarkus/component/reactive/streams/ReactiveStreamsRecorder.java ## @@ -0,0 +1,87 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams; + +import io.quarkus.arc.runtime.BeanContainer; +import io.quarkus.runtime.RuntimeValue; +import io.quarkus.runtime.annotations.Recorder; +import org.apache.camel.CamelContext; +import org.apache.camel.RuntimeCamelException; +import org.apache.camel.component.reactive.streams.ReactiveStreamsComponent; +import org.apache.camel.component.reactive.streams.api.CamelReactiveStreamsService; +import org.apache.camel.component.reactive.streams.api.CamelReactiveStreamsServiceFactory; +import org.apache.camel.component.reactive.streams.engine.DefaultCamelReactiveStreamsServiceFactory; +import org.apache.camel.support.service.ServiceHelper; + +@Recorder +public class ReactiveStreamsRecorder { +public RuntimeValue createDefaultReactiveStreamsServiceFactory() { +return new RuntimeValue<>(new DefaultCamelReactiveStreamsServiceFactory()); +} + +public RuntimeValue createReactiveStreamsComponent( +RuntimeValue serviceFactory) { +return new RuntimeValue<>(new QuarkusReactiveStreamsComponent(serviceFactory.getValue())); +} + +@SuppressWarnings("unchecked") +public void publishCamelReactiveStreamsService( +BeanContainer beanContainer, +RuntimeValue camelContext, +RuntimeValue serviceFactory) { + +// register to the container +beanContainer.instance(ReactiveStreamsProducers.class).init( +camelContext.getValue(), +serviceFactory.getValue()); +} + +private class QuarkusReactiveStreamsComponent extends ReactiveStreamsComponent { +private final CamelReactiveStreamsServiceFactory reactiveStreamServiceFactory; +private CamelReactiveStreamsService reactiveStreamService; + +public QuarkusReactiveStreamsComponent(CamelReactiveStreamsServiceFactory reactiveStreamServiceFactory) { +this.reactiveStreamServiceFactory = reactiveStreamServiceFactory; +} + +@Override +public synchronized CamelReactiveStreamsService getReactiveStreamsService() { Review comment: It could perhaps be made less deadlock-prone using synchronize(this.lock) (the lock being from ServiceSupport)? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370086982 ## File path: extensions/reactive-streams/deployment/src/main/java/org/apache/camel/quarkus/component/reactive/streams/deployment/ReactiveStreamsProcessor.java ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams.deployment; + +import io.quarkus.arc.deployment.AdditionalBeanBuildItem; +import io.quarkus.arc.deployment.BeanContainerBuildItem; +import io.quarkus.deployment.annotations.BuildProducer; +import io.quarkus.deployment.annotations.BuildStep; +import io.quarkus.deployment.annotations.ExecutionTime; +import io.quarkus.deployment.annotations.Overridable; +import io.quarkus.deployment.annotations.Record; +import io.quarkus.deployment.builditem.FeatureBuildItem; +import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsProducers; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsRecorder; +import org.apache.camel.quarkus.core.Flags; +import org.apache.camel.quarkus.core.deployment.CamelBeanBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelContextBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; + +class ReactiveStreamsProcessor { +private static final String SCHEME = "reactive-streams"; +private static final String FEATURE = "camel-reactive-streams"; + +@BuildStep +FeatureBuildItem feature() { +return new FeatureBuildItem(FEATURE); +} + +@BuildStep +CamelServiceFilterBuildItem serviceFilter() { +return new CamelServiceFilterBuildItem(CamelServiceFilter.forComponent(SCHEME)); +} + +@BuildStep +void reflectiveCLasses(BuildProducer reflectiveClasses) { +reflectiveClasses.produce(new ReflectiveClassBuildItem( +true, +false, + "org.apache.camel.component.reactive.streams.engine.ReactiveStreamsEngineConfiguration")); Review comment: Is this really necessary? If so we'll need to whitelist in https://github.com/apache/camel-quarkus/pull/647/files/36102f2f069a3e3ad9aa27255b2c6b6f80d78b70#diff-87b8f9583824da15e8833362367682ecR79 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370090865 ## File path: extensions/reactive-streams/deployment/src/main/java/org/apache/camel/quarkus/component/reactive/streams/deployment/ReactiveStreamsProcessor.java ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams.deployment; + +import io.quarkus.arc.deployment.AdditionalBeanBuildItem; +import io.quarkus.arc.deployment.BeanContainerBuildItem; +import io.quarkus.deployment.annotations.BuildProducer; +import io.quarkus.deployment.annotations.BuildStep; +import io.quarkus.deployment.annotations.ExecutionTime; +import io.quarkus.deployment.annotations.Overridable; +import io.quarkus.deployment.annotations.Record; +import io.quarkus.deployment.builditem.FeatureBuildItem; +import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsProducers; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsRecorder; +import org.apache.camel.quarkus.core.Flags; +import org.apache.camel.quarkus.core.deployment.CamelBeanBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelContextBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; + +class ReactiveStreamsProcessor { +private static final String SCHEME = "reactive-streams"; +private static final String FEATURE = "camel-reactive-streams"; + +@BuildStep +FeatureBuildItem feature() { +return new FeatureBuildItem(FEATURE); +} + +@BuildStep +CamelServiceFilterBuildItem serviceFilter() { +return new CamelServiceFilterBuildItem(CamelServiceFilter.forComponent(SCHEME)); +} + +@BuildStep +void reflectiveCLasses(BuildProducer reflectiveClasses) { +reflectiveClasses.produce(new ReflectiveClassBuildItem( +true, +false, + "org.apache.camel.component.reactive.streams.engine.ReactiveStreamsEngineConfiguration")); +} + +@BuildStep(onlyIf = Flags.MainEnabled.class) +void beans(BuildProducer beanProducer) { + beanProducer.produce(AdditionalBeanBuildItem.unremovableOf(ReactiveStreamsProducers.class)); +} + +@Overridable +@BuildStep +@Record(value = ExecutionTime.STATIC_INIT, optional = true) +public ReactiveStreamsServiceFactoryBuildItem defaultReactiveStreamsServiceFactory( Review comment: Why optional? Are there alternative implementations already now (or planned in near future)? Are we not unnecessarily overcomplicating it? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370092310 ## File path: extensions/reactive-streams/runtime/src/main/java/org/apache/camel/quarkus/component/reactive/streams/ReactiveStreamsRecorder.java ## @@ -0,0 +1,87 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams; + +import io.quarkus.arc.runtime.BeanContainer; +import io.quarkus.runtime.RuntimeValue; +import io.quarkus.runtime.annotations.Recorder; +import org.apache.camel.CamelContext; +import org.apache.camel.RuntimeCamelException; +import org.apache.camel.component.reactive.streams.ReactiveStreamsComponent; +import org.apache.camel.component.reactive.streams.api.CamelReactiveStreamsService; +import org.apache.camel.component.reactive.streams.api.CamelReactiveStreamsServiceFactory; +import org.apache.camel.component.reactive.streams.engine.DefaultCamelReactiveStreamsServiceFactory; +import org.apache.camel.support.service.ServiceHelper; + +@Recorder +public class ReactiveStreamsRecorder { +public RuntimeValue createDefaultReactiveStreamsServiceFactory() { +return new RuntimeValue<>(new DefaultCamelReactiveStreamsServiceFactory()); +} + +public RuntimeValue createReactiveStreamsComponent( +RuntimeValue serviceFactory) { +return new RuntimeValue<>(new QuarkusReactiveStreamsComponent(serviceFactory.getValue())); +} + +@SuppressWarnings("unchecked") +public void publishCamelReactiveStreamsService( +BeanContainer beanContainer, +RuntimeValue camelContext, +RuntimeValue serviceFactory) { + +// register to the container +beanContainer.instance(ReactiveStreamsProducers.class).init( +camelContext.getValue(), +serviceFactory.getValue()); +} + +private class QuarkusReactiveStreamsComponent extends ReactiveStreamsComponent { Review comment: Could the class be static maybe? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370099111 ## File path: extensions/reactive-streams/runtime/pom.xml ## @@ -0,0 +1,82 @@ + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> +4.0.0 + +org.apache.camel.quarkus +camel-quarkus-reactive-streams-parent +1.1.0-SNAPSHOT +../pom.xml + + +camel-quarkus-reactive-streams +Camel Quarkus :: Reactive Streams :: Runtime + + +1.2.0 Review comment: btw, this is what the extension generator does This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370099592 ## File path: extensions/reactive-streams/deployment/src/main/java/org/apache/camel/quarkus/component/reactive/streams/deployment/ReactiveStreamsProcessor.java ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams.deployment; + +import io.quarkus.arc.deployment.AdditionalBeanBuildItem; +import io.quarkus.arc.deployment.BeanContainerBuildItem; +import io.quarkus.deployment.annotations.BuildProducer; +import io.quarkus.deployment.annotations.BuildStep; +import io.quarkus.deployment.annotations.ExecutionTime; +import io.quarkus.deployment.annotations.Overridable; +import io.quarkus.deployment.annotations.Record; +import io.quarkus.deployment.builditem.FeatureBuildItem; +import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsProducers; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsRecorder; +import org.apache.camel.quarkus.core.Flags; +import org.apache.camel.quarkus.core.deployment.CamelBeanBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelContextBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; + +class ReactiveStreamsProcessor { +private static final String SCHEME = "reactive-streams"; +private static final String FEATURE = "camel-reactive-streams"; + +@BuildStep +FeatureBuildItem feature() { +return new FeatureBuildItem(FEATURE); +} + +@BuildStep +CamelServiceFilterBuildItem serviceFilter() { +return new CamelServiceFilterBuildItem(CamelServiceFilter.forComponent(SCHEME)); +} + +@BuildStep +void reflectiveCLasses(BuildProducer reflectiveClasses) { +reflectiveClasses.produce(new ReflectiveClassBuildItem( +true, +false, + "org.apache.camel.component.reactive.streams.engine.ReactiveStreamsEngineConfiguration")); +} + +@BuildStep(onlyIf = Flags.MainEnabled.class) +void beans(BuildProducer beanProducer) { + beanProducer.produce(AdditionalBeanBuildItem.unremovableOf(ReactiveStreamsProducers.class)); +} + +@Overridable +@BuildStep +@Record(value = ExecutionTime.STATIC_INIT, optional = true) +public ReactiveStreamsServiceFactoryBuildItem defaultReactiveStreamsServiceFactory( Review comment: yes there are some additional implementations I''d like to make later on This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370101108 ## File path: extensions/reactive-streams/deployment/src/main/java/org/apache/camel/quarkus/component/reactive/streams/deployment/ReactiveStreamsProcessor.java ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams.deployment; + +import io.quarkus.arc.deployment.AdditionalBeanBuildItem; +import io.quarkus.arc.deployment.BeanContainerBuildItem; +import io.quarkus.deployment.annotations.BuildProducer; +import io.quarkus.deployment.annotations.BuildStep; +import io.quarkus.deployment.annotations.ExecutionTime; +import io.quarkus.deployment.annotations.Overridable; +import io.quarkus.deployment.annotations.Record; +import io.quarkus.deployment.builditem.FeatureBuildItem; +import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsProducers; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsRecorder; +import org.apache.camel.quarkus.core.Flags; +import org.apache.camel.quarkus.core.deployment.CamelBeanBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelContextBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; + +class ReactiveStreamsProcessor { +private static final String SCHEME = "reactive-streams"; +private static final String FEATURE = "camel-reactive-streams"; + +@BuildStep +FeatureBuildItem feature() { +return new FeatureBuildItem(FEATURE); +} + +@BuildStep +CamelServiceFilterBuildItem serviceFilter() { +return new CamelServiceFilterBuildItem(CamelServiceFilter.forComponent(SCHEME)); +} + +@BuildStep +void reflectiveCLasses(BuildProducer reflectiveClasses) { +reflectiveClasses.produce(new ReflectiveClassBuildItem( +true, +false, + "org.apache.camel.component.reactive.streams.engine.ReactiveStreamsEngineConfiguration")); Review comment: It probably not, I'll check but I really dislike the idea to have to make changes to `core` in order to register something an extension eventually need This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
ppalaga commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370101239 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: True. I was thinking of creating a dedicated extension for this (let's call it "policy extension") that would only be pulled by our tests. This would allow us to enforce the policy for our extensions and let 3rd party extensions do what they want. WDYT? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370099592 ## File path: extensions/reactive-streams/deployment/src/main/java/org/apache/camel/quarkus/component/reactive/streams/deployment/ReactiveStreamsProcessor.java ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.camel.quarkus.component.reactive.streams.deployment; + +import io.quarkus.arc.deployment.AdditionalBeanBuildItem; +import io.quarkus.arc.deployment.BeanContainerBuildItem; +import io.quarkus.deployment.annotations.BuildProducer; +import io.quarkus.deployment.annotations.BuildStep; +import io.quarkus.deployment.annotations.ExecutionTime; +import io.quarkus.deployment.annotations.Overridable; +import io.quarkus.deployment.annotations.Record; +import io.quarkus.deployment.builditem.FeatureBuildItem; +import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsProducers; +import org.apache.camel.quarkus.component.reactive.streams.ReactiveStreamsRecorder; +import org.apache.camel.quarkus.core.Flags; +import org.apache.camel.quarkus.core.deployment.CamelBeanBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelContextBuildItem; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilter; +import org.apache.camel.quarkus.core.deployment.CamelServiceFilterBuildItem; + +class ReactiveStreamsProcessor { +private static final String SCHEME = "reactive-streams"; +private static final String FEATURE = "camel-reactive-streams"; + +@BuildStep +FeatureBuildItem feature() { +return new FeatureBuildItem(FEATURE); +} + +@BuildStep +CamelServiceFilterBuildItem serviceFilter() { +return new CamelServiceFilterBuildItem(CamelServiceFilter.forComponent(SCHEME)); +} + +@BuildStep +void reflectiveCLasses(BuildProducer reflectiveClasses) { +reflectiveClasses.produce(new ReflectiveClassBuildItem( +true, +false, + "org.apache.camel.component.reactive.streams.engine.ReactiveStreamsEngineConfiguration")); +} + +@BuildStep(onlyIf = Flags.MainEnabled.class) +void beans(BuildProducer beanProducer) { + beanProducer.produce(AdditionalBeanBuildItem.unremovableOf(ReactiveStreamsProducers.class)); +} + +@Overridable +@BuildStep +@Record(value = ExecutionTime.STATIC_INIT, optional = true) +public ReactiveStreamsServiceFactoryBuildItem defaultReactiveStreamsServiceFactory( Review comment: yes there are some additional implementations I''d like to make later on, please have a look about how camel-reactive-extension works. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370104731 ## File path: extensions/reactive-streams/runtime/pom.xml ## @@ -0,0 +1,82 @@ + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> +4.0.0 + +org.apache.camel.quarkus +camel-quarkus-reactive-streams-parent +1.1.0-SNAPSHOT +../pom.xml + + +camel-quarkus-reactive-streams +Camel Quarkus :: Reactive Streams :: Runtime + + +1.2.0 Review comment: Yes, the snapshot version in our poms is not what the generator assumes. It is hard to fix unless we stick to a stable versioning policy. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] 01/02: Publish XML schemas on new Camel website (Git repository)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git commit f5b27c4b9b9bdce3b0f2269b81cee476467dcc9c Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:02:34 2020 +0100 Publish XML schemas on new Camel website (Git repository) --- etc/scripts/release-website.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/etc/scripts/release-website.sh b/etc/scripts/release-website.sh index 1d8eb1c..22f6486 100755 --- a/etc/scripts/release-website.sh +++ b/etc/scripts/release-website.sh @@ -29,7 +29,7 @@ COMPLIST=( "camel-spring:spring" "camel-blueprint:blueprint" ) SITE_DIR="${DOWNLOAD}/websites/production/camel" WEBSITE_URL="https://svn.apache.org/repos/infra/websites/production/camel/content"; - +GIT_WEBSITE_URL="https://gitbox.apache.org/repos/asf/camel-website.git"; if [ -z "${VERSION}" -o ! -d "${DOWNLOAD}" ] then @@ -60,9 +60,9 @@ done echo echo "" -echo " CHECKOUT SCHEMAS WEBSITE " +echo " CHECKOUT CAMEL WEBSITE " echo "" -cd "${SITE_DIR}/${VERSION}" && svn co --non-interactive "${WEBSITE_URL}/schema/" +cd "${SITE_DIR}/${VERSION}" && git clone "${GIT_WEBSITE_URL}" echo "" echo " PUBLISH CAMEL SCHEMAS " @@ -70,7 +70,7 @@ echo "## for comp in ${COMPLIST[*]}; do src=${comp%:*} dest=${comp#*:} - cp ${DOWNLOAD}/${VERSION}/org/apache/camel/${src}/${VERSION}/*.xsd ${SITE_DIR}/${VERSION}/schema/${dest}/ + cp ${DOWNLOAD}/${VERSION}/org/apache/camel/${src}/${VERSION}/*.xsd ${SITE_DIR}/${VERSION}/camel-website/static/schema/${dest}/ # update_latest_released_schema("${SITE_DIR}/content/schema/${dest}/") done echo @@ -98,10 +98,10 @@ echo "NOTE: Manual steps required! Check the schemas and manual files for new ar echo " add them to the repository as required and commit your changes. This step" echo " is intentionally not automated at this point to avoid errors." echo -echo "cd ${SITE_DIR}/${VERSION}/schema/" -echo "svn status" -echo "svn add .xsd" -echo "svn ci -m \"Uploading released schemas for camel-${VERSION}\"" +echo "cd ${SITE_DIR}/${VERSION}/camel-website/" +echo "git status" +echo "git add .xsd" +echo "git commit -m \"Add XML schemas for camel-${VERSION}\"" echo echo "cd ${SITE_DIR}/${VERSION}/manual/" echo "svn status"
[camel] branch master updated (44453bd -> 814181e)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from 44453bd Replace md5 and sha1 with sha512 files new f5b27c4 Publish XML schemas on new Camel website (Git repository) new 814181e [Minor] Update commit message The 2 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: etc/scripts/release-website.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-)
[camel] 02/02: [Minor] Update commit message
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git commit 814181ede4bdef37871adc64fcc94f23f6173343 Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:05:29 2020 +0100 [Minor] Update commit message --- etc/scripts/release-website.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/etc/scripts/release-website.sh b/etc/scripts/release-website.sh index 22f6486..d72ec57 100755 --- a/etc/scripts/release-website.sh +++ b/etc/scripts/release-website.sh @@ -101,7 +101,7 @@ echo echo "cd ${SITE_DIR}/${VERSION}/camel-website/" echo "git status" echo "git add .xsd" -echo "git commit -m \"Add XML schemas for camel-${VERSION}\"" +echo "git commit -m \"Add XML schemas for Camel ${VERSION}\"" echo echo "cd ${SITE_DIR}/${VERSION}/manual/" echo "svn status"
[camel] 01/01: Merge branch 'release/2.25.0' into camel-2.25.x
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git commit db3b3e1c90e977a4d9778dd7a4ccfcef2ceab8d0 Merge: b82cb10 f15d54d Author: Gregor Zurowski AuthorDate: Thu Jan 23 05:08:28 2020 -0800 Merge branch 'release/2.25.0' into camel-2.25.x apache-camel/pom.xml| 2 +- archetypes/camel-archetype-activemq/pom.xml | 2 +- archetypes/camel-archetype-api-component/pom.xml| 2 +- archetypes/camel-archetype-blueprint/pom.xml| 2 +- archetypes/camel-archetype-cdi/pom.xml | 2 +- archetypes/camel-archetype-component/pom.xml| 2 +- archetypes/camel-archetype-connector/pom.xml| 2 +- archetypes/camel-archetype-dataformat/pom.xml | 2 +- archetypes/camel-archetype-groovy/pom.xml | 2 +- archetypes/camel-archetype-java/pom.xml | 2 +- archetypes/camel-archetype-java8/pom.xml| 2 +- archetypes/camel-archetype-scala/pom.xml| 2 +- archetypes/camel-archetype-scr/pom.xml | 2 +- archetypes/camel-archetype-spring-boot/pom.xml | 2 +- archetypes/camel-archetype-spring/pom.xml | 2 +- archetypes/camel-archetype-web/pom.xml | 2 +- archetypes/pom.xml | 2 +- bom/camel-bom/pom.xml | 2 +- bom/pom.xml | 2 +- buildingtools/pom.xml | 2 +- camel-core/pom.xml | 2 +- components/camel-ahc-ws/pom.xml | 2 +- components/camel-ahc/pom.xml| 2 +- components/camel-amqp/pom.xml | 2 +- components/camel-apns/pom.xml | 2 +- components/camel-as2/camel-as2-api/pom.xml | 2 +- components/camel-as2/camel-as2-component/pom.xml| 2 +- components/camel-as2/pom.xml| 2 +- components/camel-asn1/pom.xml | 2 +- components/camel-asterisk/pom.xml | 2 +- components/camel-atmos/pom.xml | 2 +- components/camel-atmosphere-websocket/pom.xml | 2 +- components/camel-atom/pom.xml | 2 +- components/camel-atomix/pom.xml | 2 +- components/camel-avro/pom.xml | 2 +- components/camel-aws-xray/pom.xml | 2 +- components/camel-aws/pom.xml| 2 +- components/camel-azure/pom.xml | 2 +- components/camel-bam/pom.xml| 2 +- components/camel-barcode/pom.xml| 2 +- components/camel-base64/pom.xml | 2 +- components/camel-bean-validator/pom.xml | 2 +- components/camel-beanio/pom.xml | 2 +- components/camel-beanstalk/pom.xml | 2 +- components/camel-bindy/pom.xml | 2 +- components/camel-blueprint/pom.xml | 2 +- components/camel-bonita/pom.xml | 2 +- components/camel-boon/pom.xml | 2 +- components/camel-box/camel-box-api/pom.xml | 2 +- components/camel-box/camel-box-component/pom.xml| 2 +- components/camel-box/pom.xml| 2 +- components/camel-braintree/pom.xml | 2 +- components/camel-cache/pom.xml | 2 +- components/camel-caffeine/pom.xml | 2 +- components/camel-cassandraql/pom.xml| 2 +- components/camel-castor/pom.xml | 2 +- components/camel-cdi/pom.xml| 2 +- components/camel-chronicle/pom.xml | 2 +- components/camel-chunk/pom.xml | 2 +- components/camel-cm-sms/pom.xml | 2 +- components/camel-cmis/pom.xml
[camel] branch camel-2.25.x updated (b82cb10 -> db3b3e1)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a change to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git. from b82cb10 Add license header to sites.yml add fa2a3c8 [maven-release-plugin] prepare release camel-2.25.0 add f15d54d [maven-release-plugin] prepare for next development iteration new db3b3e1 Merge branch 'release/2.25.0' into camel-2.25.x The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: apache-camel/pom.xml| 2 +- archetypes/camel-archetype-activemq/pom.xml | 2 +- archetypes/camel-archetype-api-component/pom.xml| 2 +- archetypes/camel-archetype-blueprint/pom.xml| 2 +- archetypes/camel-archetype-cdi/pom.xml | 2 +- archetypes/camel-archetype-component/pom.xml| 2 +- archetypes/camel-archetype-connector/pom.xml| 2 +- archetypes/camel-archetype-dataformat/pom.xml | 2 +- archetypes/camel-archetype-groovy/pom.xml | 2 +- archetypes/camel-archetype-java/pom.xml | 2 +- archetypes/camel-archetype-java8/pom.xml| 2 +- archetypes/camel-archetype-scala/pom.xml| 2 +- archetypes/camel-archetype-scr/pom.xml | 2 +- archetypes/camel-archetype-spring-boot/pom.xml | 2 +- archetypes/camel-archetype-spring/pom.xml | 2 +- archetypes/camel-archetype-web/pom.xml | 2 +- archetypes/pom.xml | 2 +- bom/camel-bom/pom.xml | 2 +- bom/pom.xml | 2 +- buildingtools/pom.xml | 2 +- camel-core/pom.xml | 2 +- components/camel-ahc-ws/pom.xml | 2 +- components/camel-ahc/pom.xml| 2 +- components/camel-amqp/pom.xml | 2 +- components/camel-apns/pom.xml | 2 +- components/camel-as2/camel-as2-api/pom.xml | 2 +- components/camel-as2/camel-as2-component/pom.xml| 2 +- components/camel-as2/pom.xml| 2 +- components/camel-asn1/pom.xml | 2 +- components/camel-asterisk/pom.xml | 2 +- components/camel-atmos/pom.xml | 2 +- components/camel-atmosphere-websocket/pom.xml | 2 +- components/camel-atom/pom.xml | 2 +- components/camel-atomix/pom.xml | 2 +- components/camel-avro/pom.xml | 2 +- components/camel-aws-xray/pom.xml | 2 +- components/camel-aws/pom.xml| 2 +- components/camel-azure/pom.xml | 2 +- components/camel-bam/pom.xml| 2 +- components/camel-barcode/pom.xml| 2 +- components/camel-base64/pom.xml | 2 +- components/camel-bean-validator/pom.xml | 2 +- components/camel-beanio/pom.xml | 2 +- components/camel-beanstalk/pom.xml | 2 +- components/camel-bindy/pom.xml | 2 +- components/camel-blueprint/pom.xml | 2 +- components/camel-bonita/pom.xml | 2 +- components/camel-boon/pom.xml | 2 +- components/camel-box/camel-box-api/pom.xml | 2 +- components/camel-box/camel-box-component/pom.xml| 2 +- components/camel-box/pom.xml| 2 +- components/camel-braintree/pom.xml | 2 +- components/camel-cache/pom.xml | 2 +- components/camel-caffeine/pom.xml | 2 +- components/camel-cassandraql/pom.xml| 2 +- components/camel-castor/pom.xml | 2 +- components/camel-cdi/pom.xml
[camel] 01/04: Remove unneeded variables
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git commit 2783572a06954c06b2a2437b2504f7e7ea8b7400 Author: Gregor Zurowski AuthorDate: Thu Jan 23 13:21:11 2020 +0100 Remove unneeded variables --- etc/scripts/release-distro.sh | 8 1 file changed, 8 deletions(-) diff --git a/etc/scripts/release-distro.sh b/etc/scripts/release-distro.sh index 6b46fc9..8a585ce 100755 --- a/etc/scripts/release-distro.sh +++ b/etc/scripts/release-distro.sh @@ -22,15 +22,7 @@ mkdir -p ${DOWNLOAD} 2>/dev/null # The following component contain schema definitions that must be published RUNDIR=$(cd ${0%/*} && echo $PWD) -COMPLIST=( "camel-spring:spring" - "camel-cxf:cxf" - "camel-osgi:osgi" - "camel-spring-integration:spring/integration" - "camel-spring-security:spring-security" - "camel-blueprint:blueprint" ) DIST_REPO="https://dist.apache.org/repos/dist/release/camel/apache-camel/"; -SITE_DIR="/www/camel.apache.org" - if [ -z "${VERSION}" -o ! -d "${DOWNLOAD}" ] then
[camel] 03/04: Publish XML schemas on new Camel website (Git repository)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git commit 32ca541b92870578a29bcc1b7cadca09ad6f8bb0 Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:02:34 2020 +0100 Publish XML schemas on new Camel website (Git repository) --- etc/scripts/release-website.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/etc/scripts/release-website.sh b/etc/scripts/release-website.sh index 481eb37..b0d7029 100755 --- a/etc/scripts/release-website.sh +++ b/etc/scripts/release-website.sh @@ -29,7 +29,7 @@ COMPLIST=( "camel-spring:spring" "camel-blueprint:blueprint" ) SITE_DIR="${DOWNLOAD}/websites/production/camel" WEBSITE_URL="https://svn.apache.org/repos/infra/websites/production/camel/content"; - +GIT_WEBSITE_URL="https://gitbox.apache.org/repos/asf/camel-website.git"; if [ -z "${VERSION}" -o ! -d "${DOWNLOAD}" ] then @@ -60,9 +60,9 @@ done echo echo "" -echo " CHECKOUT SCHEMAS WEBSITE " +echo " CHECKOUT CAMEL WEBSITE " echo "" -cd "${SITE_DIR}/${VERSION}" && svn co --non-interactive "${WEBSITE_URL}/schema/" +cd "${SITE_DIR}/${VERSION}" && git clone "${GIT_WEBSITE_URL}" echo "" echo " PUBLISH CAMEL SCHEMAS " @@ -70,7 +70,7 @@ echo "## for comp in ${COMPLIST[*]}; do src=${comp%:*} dest=${comp#*:} - cp ${DOWNLOAD}/${VERSION}/org/apache/camel/${src}/${VERSION}/*.xsd ${SITE_DIR}/${VERSION}/schema/${dest}/ + cp ${DOWNLOAD}/${VERSION}/org/apache/camel/${src}/${VERSION}/*.xsd ${SITE_DIR}/${VERSION}/camel-website/static/schema/${dest}/ # update_latest_released_schema("${SITE_DIR}/content/schema/${dest}/") done echo @@ -98,10 +98,10 @@ echo "NOTE: Manual steps required! Check the schemas and manual files for new ar echo " add them to the repository as required and commit your changes. This step" echo " is intentionally not automated at this point to avoid errors." echo -echo "cd ${SITE_DIR}/${VERSION}/schema/" -echo "svn status" -echo "svn add .xsd" -echo "svn ci -m \"Uploading released schemas for camel-${VERSION}\"" +echo "cd ${SITE_DIR}/${VERSION}/camel-website/" +echo "git status" +echo "git add .xsd" +echo "git commit -m \"Add XML schemas for camel-${VERSION}\"" echo echo "cd ${SITE_DIR}/${VERSION}/manual/" echo "svn status"
[camel] 02/04: Replace md5 and sha1 with sha512 files
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git commit 1a169eed9b543d79cf9b5711c10cdfab6066 Author: Gregor Zurowski AuthorDate: Thu Jan 23 13:37:55 2020 +0100 Replace md5 and sha1 with sha512 files --- etc/scripts/release-distro.sh | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/etc/scripts/release-distro.sh b/etc/scripts/release-distro.sh index 8a585ce..b2f6329 100755 --- a/etc/scripts/release-distro.sh +++ b/etc/scripts/release-distro.sh @@ -39,11 +39,21 @@ wget -e robots=off --wait 3 --no-check-certificate \ -r -np "--reject=html,txt" "--follow-tags=" \ -P "${DOWNLOAD}/${VERSION}" -nH "--cut-dirs=3" "--level=1" "--ignore-length" \ "https://repository.apache.org/content/repositories/releases/org/apache/camel/apache-camel/${VERSION}/"; + +DOWNLOAD_LOCATION="${DOWNLOAD}/${VERSION}/org/apache/camel/apache-camel/${VERSION}" + # Remove duplicate signature files -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.asc -# Remove the signature check sum files -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.md5 -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.sha1 +rm "${DOWNLOAD_LOCATION}/"*.asc.asc + +# Remove the md5 and sha1 check sum files +rm "${DOWNLOAD_LOCATION}/"*.md5 +rm "${DOWNLOAD_LOCATION}/"*.sha1 + +# Create sha512 check sum files +cd "${DOWNLOAD_LOCATION}" +for file in *.pom *.tar.gz *.zip; do + sha512sum "${file}" > "${file}.sha512" +done echo "" echo " RESET GROUP PERMISSIONS "
[camel] branch camel-2.25.x updated (db3b3e1 -> 5fe8d48)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a change to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git. from db3b3e1 Merge branch 'release/2.25.0' into camel-2.25.x new 2783572 Remove unneeded variables new 1a169ee Replace md5 and sha1 with sha512 files new 32ca541 Publish XML schemas on new Camel website (Git repository) new 5fe8d48 [Minor] Update commit message The 4 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: etc/scripts/release-distro.sh | 26 ++ etc/scripts/release-website.sh | 16 2 files changed, 22 insertions(+), 20 deletions(-)
[camel] 04/04: [Minor] Update commit message
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-2.25.x in repository https://gitbox.apache.org/repos/asf/camel.git commit 5fe8d48f7ef1b7e5dd7da1781e79276ef4d59108 Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:05:29 2020 +0100 [Minor] Update commit message --- etc/scripts/release-website.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/etc/scripts/release-website.sh b/etc/scripts/release-website.sh index b0d7029..ee90759 100755 --- a/etc/scripts/release-website.sh +++ b/etc/scripts/release-website.sh @@ -101,7 +101,7 @@ echo echo "cd ${SITE_DIR}/${VERSION}/camel-website/" echo "git status" echo "git add .xsd" -echo "git commit -m \"Add XML schemas for camel-${VERSION}\"" +echo "git commit -m \"Add XML schemas for Camel ${VERSION}\"" echo echo "cd ${SITE_DIR}/${VERSION}/manual/" echo "svn status"
[camel] branch release/2.25.0 deleted (was f15d54d)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a change to branch release/2.25.0 in repository https://gitbox.apache.org/repos/asf/camel.git. was f15d54d [maven-release-plugin] prepare for next development iteration The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
[camel] 04/04: [Minor] Update commit message
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-3.0.x in repository https://gitbox.apache.org/repos/asf/camel.git commit 65d23246470a98d7ac7561ef122134955a0ab58c Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:05:29 2020 +0100 [Minor] Update commit message --- etc/scripts/release-website.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/etc/scripts/release-website.sh b/etc/scripts/release-website.sh index 22f6486..d72ec57 100755 --- a/etc/scripts/release-website.sh +++ b/etc/scripts/release-website.sh @@ -101,7 +101,7 @@ echo echo "cd ${SITE_DIR}/${VERSION}/camel-website/" echo "git status" echo "git add .xsd" -echo "git commit -m \"Add XML schemas for camel-${VERSION}\"" +echo "git commit -m \"Add XML schemas for Camel ${VERSION}\"" echo echo "cd ${SITE_DIR}/${VERSION}/manual/" echo "svn status"
[camel] 02/04: Replace md5 and sha1 with sha512 files
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-3.0.x in repository https://gitbox.apache.org/repos/asf/camel.git commit f2bdb6474cdafb69a36f92ed3acff4e716fa4248 Author: Gregor Zurowski AuthorDate: Thu Jan 23 13:37:55 2020 +0100 Replace md5 and sha1 with sha512 files --- etc/scripts/release-distro.sh | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/etc/scripts/release-distro.sh b/etc/scripts/release-distro.sh index 43d9fa3..070c894 100755 --- a/etc/scripts/release-distro.sh +++ b/etc/scripts/release-distro.sh @@ -39,11 +39,21 @@ wget -e robots=off --wait 3 --no-check-certificate \ -r -np "--reject=html,txt" "--follow-tags=" \ -P "${DOWNLOAD}/${VERSION}" -nH "--cut-dirs=3" "--level=1" "--ignore-length" \ "https://repository.apache.org/content/repositories/releases/org/apache/camel/apache-camel/${VERSION}/"; + +DOWNLOAD_LOCATION="${DOWNLOAD}/${VERSION}/org/apache/camel/apache-camel/${VERSION}" + # Remove duplicate signature files -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.asc -# Remove the signature check sum files -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.md5 -rm ${DOWNLOAD}/dist/${VERSION}/*.asc.sha1 +rm "${DOWNLOAD_LOCATION}/"*.asc.asc + +# Remove the md5 and sha1 check sum files +rm "${DOWNLOAD_LOCATION}/"*.md5 +rm "${DOWNLOAD_LOCATION}/"*.sha1 + +# Create sha512 check sum files +cd "${DOWNLOAD_LOCATION}" +for file in *.pom *.tar.gz *.zip; do + sha512sum "${file}" > "${file}.sha512" +done echo "" echo " RESET GROUP PERMISSIONS "
[camel] branch camel-3.0.x updated (edcb1cb -> 65d2324)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a change to branch camel-3.0.x in repository https://gitbox.apache.org/repos/asf/camel.git. from edcb1cb CAMEL-14424: HTTP responses over 4 MB fail (#3505) new e35ba39 Remove unneeded variables new f2bdb64 Replace md5 and sha1 with sha512 files new 4eebbbe Publish XML schemas on new Camel website (Git repository) new 65d2324 [Minor] Update commit message The 4 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: etc/scripts/release-distro.sh | 26 ++ etc/scripts/release-website.sh | 16 2 files changed, 22 insertions(+), 20 deletions(-)
[camel] 01/04: Remove unneeded variables
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-3.0.x in repository https://gitbox.apache.org/repos/asf/camel.git commit e35ba39415a53c46f6c73ff9913725b9f9ba9e8a Author: Gregor Zurowski AuthorDate: Thu Jan 23 13:21:11 2020 +0100 Remove unneeded variables --- etc/scripts/release-distro.sh | 8 1 file changed, 8 deletions(-) diff --git a/etc/scripts/release-distro.sh b/etc/scripts/release-distro.sh index 4ca67d7..43d9fa3 100755 --- a/etc/scripts/release-distro.sh +++ b/etc/scripts/release-distro.sh @@ -22,15 +22,7 @@ mkdir -p ${DOWNLOAD} 2>/dev/null # The following component contain schema definitions that must be published RUNDIR=$(cd ${0%/*} && echo $PWD) -COMPLIST=( "camel-spring:spring" - "camel-cxf:cxf" - "camel-osgi:osgi" - "camel-spring-integration:spring/integration" - "camel-spring-security:spring-security" - "camel-blueprint:blueprint" ) DIST_REPO="https://dist.apache.org/repos/dist/release/camel/apache-camel/"; -SITE_DIR="/www/camel.apache.org" - if [ -z "${VERSION}" -o ! -d "${DOWNLOAD}" ] then
[camel] 03/04: Publish XML schemas on new Camel website (Git repository)
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch camel-3.0.x in repository https://gitbox.apache.org/repos/asf/camel.git commit 4eebbbed5d82fbaf871e9ccb39c2e54e550d2c8a Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:02:34 2020 +0100 Publish XML schemas on new Camel website (Git repository) --- etc/scripts/release-website.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/etc/scripts/release-website.sh b/etc/scripts/release-website.sh index 1d8eb1c..22f6486 100755 --- a/etc/scripts/release-website.sh +++ b/etc/scripts/release-website.sh @@ -29,7 +29,7 @@ COMPLIST=( "camel-spring:spring" "camel-blueprint:blueprint" ) SITE_DIR="${DOWNLOAD}/websites/production/camel" WEBSITE_URL="https://svn.apache.org/repos/infra/websites/production/camel/content"; - +GIT_WEBSITE_URL="https://gitbox.apache.org/repos/asf/camel-website.git"; if [ -z "${VERSION}" -o ! -d "${DOWNLOAD}" ] then @@ -60,9 +60,9 @@ done echo echo "" -echo " CHECKOUT SCHEMAS WEBSITE " +echo " CHECKOUT CAMEL WEBSITE " echo "" -cd "${SITE_DIR}/${VERSION}" && svn co --non-interactive "${WEBSITE_URL}/schema/" +cd "${SITE_DIR}/${VERSION}" && git clone "${GIT_WEBSITE_URL}" echo "" echo " PUBLISH CAMEL SCHEMAS " @@ -70,7 +70,7 @@ echo "## for comp in ${COMPLIST[*]}; do src=${comp%:*} dest=${comp#*:} - cp ${DOWNLOAD}/${VERSION}/org/apache/camel/${src}/${VERSION}/*.xsd ${SITE_DIR}/${VERSION}/schema/${dest}/ + cp ${DOWNLOAD}/${VERSION}/org/apache/camel/${src}/${VERSION}/*.xsd ${SITE_DIR}/${VERSION}/camel-website/static/schema/${dest}/ # update_latest_released_schema("${SITE_DIR}/content/schema/${dest}/") done echo @@ -98,10 +98,10 @@ echo "NOTE: Manual steps required! Check the schemas and manual files for new ar echo " add them to the repository as required and commit your changes. This step" echo " is intentionally not automated at this point to avoid errors." echo -echo "cd ${SITE_DIR}/${VERSION}/schema/" -echo "svn status" -echo "svn add .xsd" -echo "svn ci -m \"Uploading released schemas for camel-${VERSION}\"" +echo "cd ${SITE_DIR}/${VERSION}/camel-website/" +echo "git status" +echo "git add .xsd" +echo "git commit -m \"Add XML schemas for camel-${VERSION}\"" echo echo "cd ${SITE_DIR}/${VERSION}/manual/" echo "svn status"
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
lburgazzoli commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370110506 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: I really do not like to have to touch core to make an extension working so if it were me to decide, I'd go for a simple build item also, I do not know it it really make sense to put such enforcement This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel-website] branch master updated: Add release note and blog entry for release 2.25.0
This is an automated email from the ASF dual-hosted git repository. gzurowski pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel-website.git The following commit(s) were added to refs/heads/master by this push: new 09997f8 Add release note and blog entry for release 2.25.0 09997f8 is described below commit 09997f8f9ea5c0752d2a8cebf4f0943354d3837b Author: Gregor Zurowski AuthorDate: Thu Jan 23 14:20:17 2020 +0100 Add release note and blog entry for release 2.25.0 --- content/blog/RELEASE-2.25.0/camel-featured.jpg | Bin 0 -> 1541883 bytes content/blog/RELEASE-2.25.0/index.md | 21 + content/releases/release-2.25.0.md | 13 + 3 files changed, 34 insertions(+) diff --git a/content/blog/RELEASE-2.25.0/camel-featured.jpg b/content/blog/RELEASE-2.25.0/camel-featured.jpg new file mode 100644 index 000..4843585 Binary files /dev/null and b/content/blog/RELEASE-2.25.0/camel-featured.jpg differ diff --git a/content/blog/RELEASE-2.25.0/index.md b/content/blog/RELEASE-2.25.0/index.md new file mode 100644 index 000..9a4cd7d --- /dev/null +++ b/content/blog/RELEASE-2.25.0/index.md @@ -0,0 +1,21 @@ +--- +title: "RELEASE 2.25.0" +url: /blog/release-2-25-0.html +date: 2020-01-23 +draft: false +author: Gregor Zurowski +categories: ["Releases"] +preview: "The Camel community announces the immediate availability of Camel 2.25.0" +--- + + +The Camel community announces the immediate availability of Camel 2.25.0, a new minor release with 112 improvements and fixes. + +Please note that Camel 2.25.x is the last minor release for Camel 2. + +The artifacts are published and ready for you to download from the Central Maven repository. For more details please take a look at the [release notes](https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12345517&projectId=12311211). + +Many thanks to all who made this release possible. + +On behalf of the Camel PMC, +Gregor Zurowski diff --git a/content/releases/release-2.25.0.md b/content/releases/release-2.25.0.md new file mode 100644 index 000..6854e7d --- /dev/null +++ b/content/releases/release-2.25.0.md @@ -0,0 +1,13 @@ +--- +date: 2020-01-23 +draft: false +type: release-note +version: 2.25.0 +title: "Release 2.25.0" +preview: "New minor release 2.25.0" +apiBreaking: "" +knownIssues: "" +jiraVersionId: 12345517 +--- + +This release is the new Camel 2.25.0 minor release.
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using
ppalaga commented on a change in pull request #647: Fix #518 Rely on configurers for Configuration classes instead of using URL: https://github.com/apache/camel-quarkus/pull/647#discussion_r370116321 ## File path: extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/NativeImageProcessor.java ## @@ -68,6 +71,66 @@ PropertiesComponent.class, DataFormat.class); +/** + * A list of classes annotated with @UriParams which we accept to be registered for reflection + * mostly because there are errors when they are removed. TODO: solve the underlying problems and remove as + * many entries as possible from the list. + */ +private static final Set URI_PARAMS_WHITELIST = new HashSet<>(Arrays.asList( +"org.apache.camel.support.processor.DefaultExchangeFormatter", +"org.apache.camel.component.pdf.PdfConfiguration", +"org.apache.camel.component.netty.NettyConfiguration", + "org.apache.camel.component.netty.NettyServerBootstrapConfiguration", + "org.apache.camel.component.fhir.FhirUpdateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirOperationEndpointConfiguration", +"org.apache.camel.component.fhir.FhirConfiguration", + "org.apache.camel.component.fhir.FhirLoadPageEndpointConfiguration", + "org.apache.camel.component.fhir.FhirSearchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirTransactionEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCreateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirValidateEndpointConfiguration", + "org.apache.camel.component.fhir.FhirReadEndpointConfiguration", + "org.apache.camel.component.fhir.FhirCapabilitiesEndpointConfiguration", + "org.apache.camel.component.fhir.FhirHistoryEndpointConfiguration", + "org.apache.camel.component.fhir.FhirMetaEndpointConfiguration", + "org.apache.camel.component.fhir.FhirPatchEndpointConfiguration", + "org.apache.camel.component.fhir.FhirDeleteEndpointConfiguration", Review comment: > I really do not like to have to touch core to make an extension working so if it were me to decide, I'd go for a simple build item. I still think policy definitions are better to keep centrally, but OK, let me try to implement this with a build item. > also, I do not know it it really make sense to put such enforcement Well, I think this is the most effective, scalable (and sustainable) way to reduce the use of reflection. I do not think it is possible to be able to keep control once we will have 100+ extensions. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] branch master updated (814181e -> e14bc04)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from 814181e [Minor] Update commit message add e14bc04 Camel-CXF: Fixed CS No new revisions were added by this update. Summary of changes: .../src/main/java/org/apache/camel/component/cxf/CxfConsumer.java | 4 ++-- .../src/main/java/org/apache/camel/component/cxf/CxfEndpoint.java | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-)
[GitHub] [camel-quarkus] lburgazzoli commented on issue #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on issue #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#issuecomment-577687537 @ppalaga should be better now This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
ppalaga commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370124845 ## File path: docs/modules/ROOT/pages/list-of-camel-quarkus-extensions.adoc ## @@ -141,6 +141,9 @@ Number of Camel components: 60 in 51 JAR artifacts (0 deprecated) | xref:extensions/platform-http.adoc[Platform HTTP] (camel-quarkus-platform-http) + `platform-http:path` | 0.3.0 | HTTP service leveraging existing runtime platform HTTP server +| link:https://camel.apache.org/components/latest/reactive-streams-component.html[Reactive Streams] (camel-quarkus-reactive-streams) + +`reactive-streams:stream` | 1.2.0 | Reactive Camel using reactive streams Review comment: Still 1.2.0. Could you please regen? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370125054 ## File path: docs/modules/ROOT/pages/list-of-camel-quarkus-extensions.adoc ## @@ -141,6 +141,9 @@ Number of Camel components: 60 in 51 JAR artifacts (0 deprecated) | xref:extensions/platform-http.adoc[Platform HTTP] (camel-quarkus-platform-http) + `platform-http:path` | 0.3.0 | HTTP service leveraging existing runtime platform HTTP server +| link:https://camel.apache.org/components/latest/reactive-streams-component.html[Reactive Streams] (camel-quarkus-reactive-streams) + +`reactive-streams:stream` | 1.2.0 | Reactive Camel using reactive streams Review comment: damn regen This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] branch master updated (e14bc04 -> 3be2f95)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel.git. from e14bc04 Camel-CXF: Fixed CS add 3be2f95 Camel-File: Fixed CS No new revisions were added by this update. Summary of changes: .../java/org/apache/camel/component/file/GenericFileEndpoint.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation)
lburgazzoli commented on a change in pull request #659: Create a Camel ReactiveStreams extension (initial implementation) URL: https://github.com/apache/camel-quarkus/pull/659#discussion_r370126804 ## File path: docs/modules/ROOT/pages/list-of-camel-quarkus-extensions.adoc ## @@ -141,6 +141,9 @@ Number of Camel components: 60 in 51 JAR artifacts (0 deprecated) | xref:extensions/platform-http.adoc[Platform HTTP] (camel-quarkus-platform-http) + `platform-http:path` | 0.3.0 | HTTP service leveraging existing runtime platform HTTP server +| link:https://camel.apache.org/components/latest/reactive-streams-component.html[Reactive Streams] (camel-quarkus-reactive-streams) + +`reactive-streams:stream` | 1.2.0 | Reactive Camel using reactive streams Review comment: fixed This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[camel] branch master updated: Camel-Hystrix: Fixed CS
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new 2a73877 Camel-Hystrix: Fixed CS 2a73877 is described below commit 2a73877a681d7e2c12e58735e4f1eac3c9953d88 Author: Andrea Cosentino AuthorDate: Thu Jan 23 14:51:36 2020 +0100 Camel-Hystrix: Fixed CS --- .../camel/component/hystrix/metrics/HystrixEventStreamService.java| 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/camel-hystrix/src/main/java/org/apache/camel/component/hystrix/metrics/HystrixEventStreamService.java b/components/camel-hystrix/src/main/java/org/apache/camel/component/hystrix/metrics/HystrixEventStreamService.java index 9956e20..136244f 100644 --- a/components/camel-hystrix/src/main/java/org/apache/camel/component/hystrix/metrics/HystrixEventStreamService.java +++ b/components/camel-hystrix/src/main/java/org/apache/camel/component/hystrix/metrics/HystrixEventStreamService.java @@ -39,16 +39,16 @@ import org.slf4j.LoggerFactory; @ManagedResource(description = "Managed Hystrix EventStreamService") public class HystrixEventStreamService extends ServiceSupport implements StaticService, HystrixMetricsPoller.MetricsAsJsonPollerListener { -private static final Logger LOG = LoggerFactory.getLogger(HystrixEventStreamService.class); - public static final int METRICS_QUEUE_SIZE = 1000; +private static final Logger LOG = LoggerFactory.getLogger(HystrixEventStreamService.class); private int delay = 500; private int queueSize = METRICS_QUEUE_SIZE; private HystrixMetricsPoller poller; // use a queue with a upper limit to avoid storing too many metrics private Queue queue; + public int getDelay() { return delay; }
[camel] branch master updated: Camel-Jgroups: Fixed CS
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new d570edf Camel-Jgroups: Fixed CS d570edf is described below commit d570edfa30653e824f842d60300ef54ea5c707c8 Author: Andrea Cosentino AuthorDate: Thu Jan 23 14:52:44 2020 +0100 Camel-Jgroups: Fixed CS --- .../main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java index bf63a62..a1409ed 100644 --- a/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java +++ b/components/camel-jgroups/src/main/java/org/apache/camel/component/jgroups/JGroupsEndpoint.java @@ -40,13 +40,12 @@ import org.slf4j.LoggerFactory; @UriEndpoint(firstVersion = "2.13.0", scheme = "jgroups", title = "JGroups", syntax = "jgroups:clusterName", label = "clustering,messaging") public class JGroupsEndpoint extends DefaultEndpoint { -private static final Logger LOG = LoggerFactory.getLogger(JGroupsEndpoint.class); - public static final String HEADER_JGROUPS_ORIGINAL_MESSAGE = "JGROUPS_ORIGINAL_MESSAGE"; public static final String HEADER_JGROUPS_SRC = "JGROUPS_SRC"; public static final String HEADER_JGROUPS_DEST = "JGROUPS_DEST"; public static final String HEADER_JGROUPS_CHANNEL_ADDRESS = "JGROUPS_CHANNEL_ADDRESS"; +private static final Logger LOG = LoggerFactory.getLogger(JGroupsEndpoint.class); private AtomicInteger connectCount = new AtomicInteger(0); private JChannel channel;
[camel] branch master updated: Camel-REST: Fixed CS
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/master by this push: new 0978b7e Camel-REST: Fixed CS 0978b7e is described below commit 0978b7e33e39708ead8693530a04f61fbfe00209 Author: Andrea Cosentino AuthorDate: Thu Jan 23 14:54:45 2020 +0100 Camel-REST: Fixed CS --- .../src/main/java/org/apache/camel/component/rest/RestEndpoint.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestEndpoint.java b/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestEndpoint.java index a425ace..cfab1b7 100644 --- a/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestEndpoint.java +++ b/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestEndpoint.java @@ -50,13 +50,13 @@ import static org.apache.camel.support.RestProducerFactoryHelper.setupComponent; @UriEndpoint(firstVersion = "2.14.0", scheme = "rest", title = "REST", syntax = "rest:method:path:uriTemplate", label = "core,rest", lenientProperties = true) public class RestEndpoint extends DefaultEndpoint { -private static final Logger LOG = LoggerFactory.getLogger(RestEndpoint.class); - public static final String[] DEFAULT_REST_CONSUMER_COMPONENTS = new String[]{"coap", "netty-http", "jetty", "servlet", "spark-java", "undertow"}; public static final String[] DEFAULT_REST_PRODUCER_COMPONENTS = new String[]{"http", "netty-http", "undertow"}; public static final String DEFAULT_API_COMPONENT_NAME = "openapi"; public static final String RESOURCE_PATH = "META-INF/services/org/apache/camel/rest/"; +private static final Logger LOG = LoggerFactory.getLogger(RestEndpoint.class); + @UriPath(label = "common", enums = "get,post,put,delete,patch,head,trace,connect,options") @Metadata(required = true) private String method; @UriPath(label = "common") @Metadata(required = true)
[camel-quarkus] branch master updated (c6b1b27 -> 011b79f)
This is an automated email from the ASF dual-hosted git repository. lburgazzoli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git. from c6b1b27 Merge pull request #658 from lburgazzoli/quarkus-1.2.0.Final add 011b79f The MicroProfile test fails if message history is turned off #650 No new revisions were added by this update. Summary of changes: .../deployment/MicroProfileMetricsProcessor.java | 10 ++ .../runtime/CamelMicroProfileMetricsRecorder.java | 37 +++--- .../src/main/resources/application.properties | 7 +++- 3 files changed, 48 insertions(+), 6 deletions(-)
[GitHub] [camel-quarkus] lburgazzoli closed issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT
lburgazzoli closed issue #650: The MicroProfile test fails against Camel 3.1.0-SNAPSHOT URL: https://github.com/apache/camel-quarkus/issues/650 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services