[camel-website] branch master updated: fixed broken link
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-website.git
The following commit(s) were added to refs/heads/master by this push:
new 6c60320 fixed broken link
new 2da01bf Merge pull request #98 from dezibyte/master
6c60320 is described below
commit 6c60320912b0718df818385472b262027627242b
Author: dezibyte
AuthorDate: Thu Oct 31 22:18:08 2019 +0100
fixed broken link
---
content/_index.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/content/_index.md b/content/_index.md
index b3bafad..9b14660 100644
--- a/content/_index.md
+++ b/content/_index.md
@@ -80,7 +80,7 @@ Camel is an [Apache Software
Foundation](https://www.apache.org) project, availa
[Sources](./community/sources/), [mailing lists](./community/mailing-list/),
[issue tracker](./community/support/): it's fully open, you can access directly.
-We also love contributions: don't hesitate to
[contribute](./manual/latest/contributing.html). You can contribute by https://github.com/apache/camel-website/edit/master/content{{ .File.Path
}}">editing this page!
+We also love contributions: don't hesitate to
[contribute](./manual/latest/contributing.html). You can contribute by https://github.com/apache/camel-website/edit/master/content;>editing this
page!
[Be Involved In The Community](./manual/latest/contributing.html) | [How To
Contribute](./manual/latest/contributing.html)
[GitHub] [camel-quarkus] asf-ci commented on issue #362: chore: fix cs
asf-ci commented on issue #362: chore: fix cs URL: https://github.com/apache/camel-quarkus/pull/362#issuecomment-548625836 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/378/ 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 (c81a15c -> 6186777)
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 c81a15c Update pr-validate.yml
add 6186777 Initial camel-quarkus-netty support
No new revisions were added by this update.
Summary of changes:
.gitignore | 1 +
docs/modules/ROOT/pages/extensions/netty.adoc | 47 ++
.../pages/list-of-camel-quarkus-extensions.adoc| 5 +-
.../{platform-http => netty}/deployment/pom.xml| 21
.../netty/deployment/NettyProcessor.java} | 38 ++-
.../parent-pom.xml => extensions/netty/pom.xml | 6 +--
extensions/{opentracing => netty}/runtime/pom.xml | 25 +-
...ubstituteNettyServerBootstrapConfiguration.java | 2 +-
.../SubstituteNettyServerBossPoolBuilder.java | 2 +-
.../runtime/SubstituteNettyWorkerPoolBuilder.java | 2 +-
.../main/resources/META-INF/quarkus-extension.yaml | 3 +-
extensions/pom.xml | 1 +
extensions/readme.adoc | 5 +-
.../netty/pom.xml | 18 ---
.../camel/quarkus/component/netty}/CamelRoute.java | 39 +++
.../camel/quarkus/component/netty/Poetry.java | 56 ++
.../src/main/resources/application.properties | 4 ++
.../camel/quarkus/component/netty/NettyIT.java}| 4 +-
.../camel/quarkus/component/netty/NettyTest.java | 50 +++
integration-tests/pom.xml | 1 +
poms/bom-deployment/pom.xml| 5 ++
poms/bom/pom.xml | 10
22 files changed, 267 insertions(+), 78 deletions(-)
create mode 100644 docs/modules/ROOT/pages/extensions/netty.adoc
copy extensions/{platform-http => netty}/deployment/pom.xml (81%)
copy
extensions/{salesforce/deployment/src/main/java/org/apache/camel/quarkus/component/salesforce/deployment/SalesforceProcessor.java
=>
netty/deployment/src/main/java/org/apache/camel/quarkus/component/netty/deployment/NettyProcessor.java}
(60%)
copy build/create-extension-templates/parent-pom.xml =>
extensions/netty/pom.xml (91%)
copy extensions/{opentracing => netty}/runtime/pom.xml (80%)
copy
extensions/{netty-http/runtime/src/main/java/org/apache/camel/quarkus/component/netty/http
=>
netty/runtime/src/main/java/org/apache/camel/quarkus/component/netty}/runtime/SubstituteNettyServerBootstrapConfiguration.java
(94%)
copy
extensions/{netty-http/runtime/src/main/java/org/apache/camel/quarkus/component/netty/http
=>
netty/runtime/src/main/java/org/apache/camel/quarkus/component/netty}/runtime/SubstituteNettyServerBossPoolBuilder.java
(96%)
copy
extensions/{netty-http/runtime/src/main/java/org/apache/camel/quarkus/component/netty/http
=>
netty/runtime/src/main/java/org/apache/camel/quarkus/component/netty}/runtime/SubstituteNettyWorkerPoolBuilder.java
(96%)
copy extensions/{core =>
netty}/runtime/src/main/resources/META-INF/quarkus-extension.yaml (73%)
copy build/create-extension-templates/integration-test-pom.xml =>
integration-tests/netty/pom.xml (89%)
copy
integration-tests/{servlet/src/main/java/org/apache/camel/quarkus/component/servlet
=>
netty/src/main/java/org/apache/camel/quarkus/component/netty}/CamelRoute.java
(54%)
create mode 100644
integration-tests/netty/src/main/java/org/apache/camel/quarkus/component/netty/Poetry.java
copy
tooling/maven/package-maven-plugin/src/main/resources/camel-quarkus-package-maven-plugin.properties
=> integration-tests/netty/src/main/resources/application.properties (95%)
copy
integration-tests/{paho/src/test/java/org/apache/camel/quarkus/component/paho/it/PahoIT.java
=> netty/src/test/java/org/apache/camel/quarkus/component/netty/NettyIT.java}
(91%)
create mode 100644
integration-tests/netty/src/test/java/org/apache/camel/quarkus/component/netty/NettyTest.java
[GitHub] [camel-quarkus] lburgazzoli merged pull request #353: Camel quarkus netty
lburgazzoli merged pull request #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353 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] asf-ci commented on issue #353: Camel quarkus netty
asf-ci commented on issue #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#issuecomment-548596671 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/377/ 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 #362: chore: fix cs
lburgazzoli commented on issue #362: chore: fix cs URL: https://github.com/apache/camel-quarkus/pull/362#issuecomment-548586690 ok to test 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 closed pull request #361: DO NOT MERGEwq
lburgazzoli closed pull request #361: DO NOT MERGEwq URL: https://github.com/apache/camel-quarkus/pull/361 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 (d8fc9b1 -> c81a15c)
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 d8fc9b1 Update pr-validate.yml add c81a15c Update pr-validate.yml No new revisions were added by this update. Summary of changes: .github/workflows/pr-validate.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[camel-quarkus] branch master updated (80ef8f7 -> 5c2c1c2)
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 80ef8f7 Fix #354 Update contributor guide: s/json/yaml/, extension adoc page add 5c2c1c2 Add PR validation workflow No new revisions were added by this update. Summary of changes: .github/workflows/pr-validate.yml | 17 + 1 file changed, 17 insertions(+) create mode 100644 .github/workflows/pr-validate.yml
[camel-quarkus] branch master updated (8871ed9 -> 80ef8f7)
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 8871ed9 Use uppercase FHIR where appropriate (#359) add 80ef8f7 Fix #354 Update contributor guide: s/json/yaml/, extension adoc page No new revisions were added by this update. Summary of changes: build/create-extension-templates/runtime-pom.xml | 4 docs/modules/ROOT/pages/contributor-guide.adoc | 12 +++- 2 files changed, 15 insertions(+), 1 deletion(-)
[GitHub] [camel-quarkus] lburgazzoli closed issue #354: Update contributor guide: s/json/yaml/, extension adoc page
lburgazzoli closed issue #354: Update contributor guide: s/json/yaml/, extension adoc page URL: https://github.com/apache/camel-quarkus/issues/354 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 #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page
lburgazzoli merged pull request #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page URL: https://github.com/apache/camel-quarkus/pull/355 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] asf-ci commented on issue #353: Camel quarkus netty
asf-ci commented on issue #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#issuecomment-548561707 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/376/ 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] asf-ci commented on issue #353: Camel quarkus netty
asf-ci commented on issue #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#issuecomment-548559528 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/375/ 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] oscerd commented on issue #353: Camel quarkus netty
oscerd commented on issue #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#issuecomment-548555804 Sorry just realised it's a gitignore already there. Sorry. Looks good 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] oscerd commented on a change in pull request #353: Camel quarkus netty
oscerd commented on a change in pull request #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341349939 ## File path: .gitignore ## @@ -33,3 +33,6 @@ nb-configuration.xml Review comment: I still see the gitignore file. 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] davsclaus merged pull request #359: Use uppercase FHIR where appropriate
davsclaus merged pull request #359: Use uppercase FHIR where appropriate URL: https://github.com/apache/camel-quarkus/pull/359 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: Use uppercase FHIR where appropriate (#359)
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-quarkus.git The following commit(s) were added to refs/heads/master by this push: new 8871ed9 Use uppercase FHIR where appropriate (#359) 8871ed9 is described below commit 8871ed9289cd16c09327aed2b4b19730013b361f Author: Peter Palaga AuthorDate: Thu Oct 31 20:11:26 2019 +0100 Use uppercase FHIR where appropriate (#359) --- integration-tests/fhir/pom.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/integration-tests/fhir/pom.xml b/integration-tests/fhir/pom.xml index c3a1d2c..3cc32e4 100644 --- a/integration-tests/fhir/pom.xml +++ b/integration-tests/fhir/pom.xml @@ -8,8 +8,8 @@ camel-quarkus-integration-test-fhir -Camel Quarkus :: Integration Tests :: fhir -Integration tests for Camel Quarkus fhir extension +Camel Quarkus :: Integration Tests :: FHIR +Integration tests for Camel Quarkus FHIR extension
[GitHub] [camel-quarkus] asf-ci commented on issue #359: Use uppercase FHIR where appropriate
asf-ci commented on issue #359: Use uppercase FHIR where appropriate URL: https://github.com/apache/camel-quarkus/pull/359#issuecomment-548520298 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/374/ 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-14092: Add camel-debezium-mongodb (#3305)
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 2808dbf CAMEL-14092: Add camel-debezium-mongodb (#3305)
2808dbf is described below
commit 2808dbf6c9e78953024ca66846b194582e5362e3
Author: Omar Al-Safi
AuthorDate: Thu Oct 31 19:15:35 2019 +0100
CAMEL-14092: Add camel-debezium-mongodb (#3305)
Signed-off-by: Omar Al-Safi
---
apache-camel/pom.xml | 10 +
apache-camel/src/main/descriptors/common-bin.xml | 2 +
bom/camel-bom/pom.xml | 10 +
components/camel-debezium-mongodb/pom.xml | 149 ++
.../src/main/docs/debezium-mongodb-component.adoc | 227 +
.../debezium/DebeziumMongodbComponent.java | 57 +++
.../debezium/DebeziumMongodbEndpoint.java | 49 ++
.../debezium/DebeziumMongodbComponentTest.java | 119 +
...ConnectorEmbeddedDebeziumConfigurationTest.java | 89
.../src/test/resources/log4j2.properties | 30 ++
components/pom.xml | 1 +
parent/pom.xml | 10 +
.../karaf/features/src/main/resources/features.xml | 12 +-
.../camel-debezium-mongodb-starter/pom.xml | 53 ++
.../DebeziumMongodbComponentAutoConfiguration.java | 129 +
.../DebeziumMongodbComponentConfiguration.java | 566 +
.../src/main/resources/META-INF/LICENSE.txt| 203
.../src/main/resources/META-INF/NOTICE.txt | 11 +
.../src/main/resources/META-INF/spring.factories | 19 +
.../src/main/resources/META-INF/spring.provides| 17 +
platforms/spring-boot/components-starter/pom.xml | 1 +
.../camel-spring-boot-dependencies/pom.xml | 10 +
.../itest/karaf/CamelDebeziumMongodbTest.java | 35 ++
.../itest/springboot/CamelDebeziumMongodbTest.java | 46 ++
24 files changed, 1854 insertions(+), 1 deletion(-)
diff --git a/apache-camel/pom.xml b/apache-camel/pom.xml
index aab587b..fd753ab 100644
--- a/apache-camel/pom.xml
+++ b/apache-camel/pom.xml
@@ -473,6 +473,11 @@
org.apache.camel
+ camel-debezium-mongodb
+ ${project.version}
+
+
+ org.apache.camel
camel-debezium-mysql
${project.version}
@@ -2047,6 +2052,11 @@
org.apache.camel
+ camel-debezium-mongodb-starter
+ ${project.version}
+
+
+ org.apache.camel
camel-debezium-mysql-starter
${project.version}
diff --git a/apache-camel/src/main/descriptors/common-bin.xml
b/apache-camel/src/main/descriptors/common-bin.xml
index 74b2a9c..aa37152 100644
--- a/apache-camel/src/main/descriptors/common-bin.xml
+++ b/apache-camel/src/main/descriptors/common-bin.xml
@@ -115,6 +115,7 @@
org.apache.camel:camel-dataformat
org.apache.camel:camel-dataset
org.apache.camel:camel-debezium-common
+org.apache.camel:camel-debezium-mongodb
org.apache.camel:camel-debezium-mysql
org.apache.camel:camel-debezium-postgres
org.apache.camel:camel-digitalocean
@@ -469,6 +470,7 @@
org.apache.camel:camel-dataformat-starter
org.apache.camel:camel-dataset-starter
org.apache.camel:camel-debezium-common-starter
+org.apache.camel:camel-debezium-mongodb-starter
org.apache.camel:camel-debezium-mysql-starter
org.apache.camel:camel-debezium-postgres-starter
org.apache.camel:camel-digitalocean-starter
diff --git a/bom/camel-bom/pom.xml b/bom/camel-bom/pom.xml
index a2dc471..6bbf0e5 100644
--- a/bom/camel-bom/pom.xml
+++ b/bom/camel-bom/pom.xml
@@ -829,6 +829,16 @@
org.apache.camel
+camel-debezium-mongodb
+${project.version}
+
+
+org.apache.camel
+camel-debezium-mongodb-starter
+${project.version}
+
+
+org.apache.camel
camel-debezium-mysql
${project.version}
diff --git a/components/camel-debezium-mongodb/pom.xml
b/components/camel-debezium-mongodb/pom.xml
new file mode 100644
index 000..3786f43
--- /dev/null
+++ b/components/camel-debezium-mongodb/pom.xml
@@ -0,0 +1,149 @@
+
+
+http://www.w3.org/2001/XMLSchema-instance;
xmlns="http://maven.apache.org/POM/4.0.0;
+ 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
+components
+3.0.0-SNAPSHOT
+
+
+camel-debezium-mongodb
+jar
+Camel :: Debezium :: MongoDB
+Camel Debezium MongoDB support
+
+
+
+
+org.apache.camel
+camel-debezium-common
+
+
+
+
+io.debezium
+debezium-connector-mongodb
+
[camel] branch camel-2.x updated: Updating CXF
This is an automated email from the ASF dual-hosted git repository. coheigea pushed a commit to branch camel-2.x in repository https://gitbox.apache.org/repos/asf/camel.git The following commit(s) were added to refs/heads/camel-2.x by this push: new faaa578 Updating CXF faaa578 is described below commit faaa5784665c388023e3ea1cd6e7a4536e920244 Author: Colm O hEigeartaigh AuthorDate: Thu Oct 31 16:31:22 2019 + Updating CXF --- parent/pom.xml | 2 +- .../camel-spring-boot-dependencies/pom.xml | 56 +++--- 2 files changed, 29 insertions(+), 29 deletions(-) diff --git a/parent/pom.xml b/parent/pom.xml index a187221..621503c 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -169,7 +169,7 @@ 1.4.13 1.4.13_1 2.12.0 -3.3.3 +3.3.4 [3.1,4.0) 3.2.0 3.2.0 diff --git a/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml b/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml index d143613..a8a4325 100644 --- a/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml +++ b/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml @@ -3226,142 +3226,142 @@ org.apache.cxf.services.sts cxf-services-sts-core -3.3.3 +3.3.4 org.apache.cxf.services.ws-discovery cxf-services-ws-discovery-api -3.3.3 +3.3.4 org.apache.cxf cxf-core -3.3.3 +3.3.4 org.apache.cxf cxf-rt-bindings-soap -3.3.3 +3.3.4 org.apache.cxf cxf-rt-bindings-xml -3.3.3 +3.3.4 org.apache.cxf cxf-rt-databinding-jaxb -3.3.3 +3.3.4 org.apache.cxf cxf-rt-features-clustering -3.3.3 +3.3.4 org.apache.cxf cxf-rt-features-logging -3.3.3 +3.3.4 org.apache.cxf cxf-rt-frontend-jaxrs -3.3.3 +3.3.4 org.apache.cxf cxf-rt-frontend-jaxws -3.3.3 +3.3.4 org.apache.cxf cxf-rt-frontend-simple -3.3.3 +3.3.4 org.apache.cxf cxf-rt-management -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-client -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-extension-providers -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-security-oauth2 -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-service-description-swagger -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-http -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-http-jetty -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-jms -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-local -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-addr -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-mex -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-policy -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-rm -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-security -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-security-oauth2 -3.3.3 +3.3.4 org.apache.cxf cxf-rt-wsdl -3.3.3 +3.3.4 org.apache.cxf cxf-testutils -3.3.3 +3.3.4 org.apache.ftpserver
[GitHub] [camel-quarkus] lburgazzoli opened a new issue #360: Add description to extension matedata
lburgazzoli opened a new issue #360: Add description to extension matedata URL: https://github.com/apache/camel-quarkus/issues/360 The extension metadata yaml file lacks description of the extension. If not provided the quarkus tooling take it from the description of the runtime pom 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-k] nicolaferraro opened a new issue #1038: Changing trait config should lead to redeploy
nicolaferraro opened a new issue #1038: Changing trait config should lead to redeploy URL: https://github.com/apache/camel-k/issues/1038 Integration keeps running instead. Context: I've an integration running, I edit the trait config (without `kamel`) but the change is not noticed. 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] asf-ci commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
asf-ci commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548467609 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/373/Failed Tests: 94camel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-examples-observability: 4org.acme.observability.ObservabilityTest.healthorg.acme.observability.ObservabilityTest.metricsorg.acme.observability.ObservabilityIT.healthorg.acme.observability.ObservabilityIT.metricscamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-examples-rest-json: 4org.acme.rest.json.RestJsonTest.fruitsorg.acme.rest.json.RestJsonTest.legumesorg.acme.rest.json.RestJsonIT.fruitsorg.acme.rest.json.RestJsonIT.legumescamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-csv: 4org.apache.camel.quarkus.component.csv.it.CsvTest.json2csvorg.apache.camel.quarkus.component.csv.it.CsvTest.csv2jsonorg.apache.camel.quarkus.component.csv.it.CsvIT.json2csvorg.apache.camel.quarkus.component.csv.it.CsvIT.csv2jsoncamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-infinispan: 1org.apache.camel.quarkus.component.infinispan.CamelTest.testInfinispancamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-jackson: 2org.apache.camel.quarkus.component.jackson.JacksonTest.testRoutesorg.apache.camel.quarkus.component.jackson.JacksonITCase.testRoutescamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-jdbc: 4org.apache.camel.quarkus.component.jdbc.CamelJdbcTest.testExecuteStatementorg.apache.camel.quarkus.component.jdbc.CamelJdbcTest.testGetSpeciesByIdorg.apache.camel.quarkus.component.jdbc.CamelJdbcIT.testExecuteStatementorg.apache.camel.quarkus.component.jdbc.CamelJdbcIT.testGetSpeciesByIdcamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-mail: 2org.apache.camel.quarkus.component.mail.MailTest.testSendAsMailorg.apache.camel.quarkus.component.mail.MailITCase.testSendAsMailcamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-microprofile-metrics: 22org.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsCamelContextEventNotifierorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsRouteEventNotifierorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsConcurrentGaugeorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsHistogramorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsMessageHistoryFactoryorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsExchangeEventNotifierorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsRoutePolicyFactoryorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsCounterorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsGaugeorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsMeterorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroProfileMetricsTest.testMicroProfileMetricsTimerorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsCamelContextEventNotifierorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsRouteEventNotifierorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsConcurrentGaugeorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsHistogramorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsMessageHistoryFactoryorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsExchangeEventNotifierorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsRoutePolicyFactoryorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsCounterorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsGaugeorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsMeterorg.apache.camel.quarkus.component.microprofile.metrics.it.MicroprofileMetricsIT.testMicroProfileMetricsTimercamel-quarkus-pr/org.apache.camel.quarkus:camel-quarkus-integration-test-netty-http:
[camel] branch master updated: Updating CXF
This is an automated email from the ASF dual-hosted git repository. coheigea 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 03720ad Updating CXF 03720ad is described below commit 03720ad62eba7c52272167184d501fe64e637c1b Author: Colm O hEigeartaigh AuthorDate: Thu Oct 31 16:31:22 2019 + Updating CXF --- parent/pom.xml | 2 +- .../camel-spring-boot-dependencies/pom.xml | 56 +++--- 2 files changed, 29 insertions(+), 29 deletions(-) diff --git a/parent/pom.xml b/parent/pom.xml index 60b7908..db822d4 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -152,7 +152,7 @@ 1.4.13 1.4.13_1 2.12.0 -3.3.3 +3.3.4 [3.3,4.0) 3.3.2 3.2.0 diff --git a/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml b/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml index 82a9162..441296c 100644 --- a/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml +++ b/platforms/spring-boot/spring-boot-dm/camel-spring-boot-dependencies/pom.xml @@ -3541,142 +3541,142 @@ org.apache.cxf.services.sts cxf-services-sts-core -3.3.3 +3.3.4 org.apache.cxf.services.ws-discovery cxf-services-ws-discovery-api -3.3.3 +3.3.4 org.apache.cxf cxf-core -3.3.3 +3.3.4 org.apache.cxf cxf-rt-bindings-soap -3.3.3 +3.3.4 org.apache.cxf cxf-rt-bindings-xml -3.3.3 +3.3.4 org.apache.cxf cxf-rt-databinding-jaxb -3.3.3 +3.3.4 org.apache.cxf cxf-rt-features-clustering -3.3.3 +3.3.4 org.apache.cxf cxf-rt-features-logging -3.3.3 +3.3.4 org.apache.cxf cxf-rt-frontend-jaxrs -3.3.3 +3.3.4 org.apache.cxf cxf-rt-frontend-jaxws -3.3.3 +3.3.4 org.apache.cxf cxf-rt-frontend-simple -3.3.3 +3.3.4 org.apache.cxf cxf-rt-management -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-client -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-extension-providers -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-security-oauth2 -3.3.3 +3.3.4 org.apache.cxf cxf-rt-rs-service-description-swagger -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-http -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-http-jetty -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-jms -3.3.3 +3.3.4 org.apache.cxf cxf-rt-transports-local -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-addr -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-mex -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-policy -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-rm -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-security -3.3.3 +3.3.4 org.apache.cxf cxf-rt-ws-security-oauth2 -3.3.3 +3.3.4 org.apache.cxf cxf-rt-wsdl -3.3.3 +3.3.4 org.apache.cxf cxf-testutils -3.3.3 +3.3.4 org.apache.ftpserver
[camel-k-runtime] branch master updated: camel-knative: use EnumSource as annotation for parametrized tests
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-k-runtime.git
The following commit(s) were added to refs/heads/master by this push:
new 0aae73b camel-knative: use EnumSource as annotation for parametrized
tests
0aae73b is described below
commit 0aae73bf5b40a3c0a2f1c207145a52815fa05bf0
Author: lburgazzoli
AuthorDate: Thu Oct 31 17:18:44 2019 +0100
camel-knative: use EnumSource as annotation for parametrized tests
---
.../component/knative/http/KnativeHttpTest.java| 62 +-
1 file changed, 26 insertions(+), 36 deletions(-)
diff --git
a/camel-knative/camel-knative-http/src/test/java/org/apache/camel/component/knative/http/KnativeHttpTest.java
b/camel-knative/camel-knative-http/src/test/java/org/apache/camel/component/knative/http/KnativeHttpTest.java
index 390efd4..0ff0014 100644
---
a/camel-knative/camel-knative-http/src/test/java/org/apache/camel/component/knative/http/KnativeHttpTest.java
+++
b/camel-knative/camel-knative-http/src/test/java/org/apache/camel/component/knative/http/KnativeHttpTest.java
@@ -26,7 +26,6 @@ import java.util.UUID;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.atomic.AtomicReference;
import java.util.stream.Collectors;
-import java.util.stream.Stream;
import com.fasterxml.jackson.databind.ObjectMapper;
import io.undertow.Undertow;
@@ -51,8 +50,7 @@ import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.params.ParameterizedTest;
-import org.junit.jupiter.params.provider.Arguments;
-import org.junit.jupiter.params.provider.MethodSource;
+import org.junit.jupiter.params.provider.EnumSource;
import static
org.apache.camel.component.knative.http.KnativeHttpTestSupport.configureKnativeComponent;
import static
org.apache.camel.component.knative.spi.KnativeEnvironment.channel;
@@ -102,16 +100,8 @@ public class KnativeHttpTest {
}
-private static Stream provideCloudEventsImplementations() {
-return Stream.of(
-Arguments.of(CloudEvents.V01),
-Arguments.of(CloudEvents.V02),
-Arguments.of(CloudEvents.V03)
-);
-}
-
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testInvokeEndpoint(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -156,7 +146,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testConsumeStructuredContent(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -240,7 +230,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testConsumeContent(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -301,7 +291,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testConsumeContentWithFilter(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -395,7 +385,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testConsumeContentWithRegExFilter(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -489,7 +479,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testConsumeEventContent(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -568,7 +558,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testReply(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -618,7 +608,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testInvokeServiceWithoutHost(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@ -650,7 +640,7 @@ public class KnativeHttpTest {
}
@ParameterizedTest
-@MethodSource("provideCloudEventsImplementations")
+@EnumSource(CloudEvents.class)
void testInvokeNotExistingEndpoint(CloudEvent ce) throws Exception {
configureKnativeComponent(
context,
@@
[GitHub] [camel-quarkus] dhartford commented on a change in pull request #353: Camel quarkus netty
dhartford commented on a change in pull request #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341221311 ## File path: integration-tests/netty/pom.xml ## @@ -0,0 +1,132 @@ + + +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-integration-tests +0.3.2-SNAPSHOT + + +camel-quarkus-integration-test-netty +Camel Quarkus :: Integration Tests :: Netty +Integration tests for Camel Quarkus Netty extension + + + +org.apache.camel.quarkus +camel-quarkus-netty + + +org.apache.camel.quarkus +camel-quarkus-log + + +io.quarkus +quarkus-resteasy + Review comment: Definitely an important item - the images were hanging and locking the port number without undertow as part of the build (i.e. not test scope, has to be included as normal dependency). 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] dhartford commented on a change in pull request #353: Camel quarkus netty
dhartford commented on a change in pull request #353: Camel quarkus netty
URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341220333
##
File path: extensions/netty/runtime/pom.xml
##
@@ -0,0 +1,104 @@
+
+
+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-netty-parent
+0.3.2-SNAPSHOT
+../pom.xml
+
+
+camel-quarkus-netty
+Camel Quarkus :: Netty :: Runtime
+
+
+0.4.0
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-bom
+${project.version}
+pom
+import
+
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-core
+
+
+io.quarkus
+quarkus-netty
+
+
+io.quarkus
+quarkus-caffeine
+
+
+org.apache.camel
+camel-endpointdsl
+
Review comment:
Good catch, I was flying blind on why the native tests were failing, and it
was fixed after adding this but appeared to have been a coincidence, testing
fine without 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] lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548433118 > > if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope > > Maybe I am missing something trivial, but adding `@ApplicationScoped` to a class extending `RouteBuilder` does not make `@Inject` in that same class work. The itest in this PR implements exactly that scenario and I was not able to make it work without the proposed fix. Or you mean something else by "annotate the route"? > I think it does not work because camel discovers it on the classpath and load it on ts own so it bypass ArC. > > still need to fix the problem of having the same route discovered at runtime and at build time > > Good point, let me add a test for that. 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 #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page
ppalaga commented on issue #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page URL: https://github.com/apache/camel-quarkus/pull/355#issuecomment-548432859 > Cross-check: For all the references for `-Pnative`, do you mean `-Dnative`? I just tested and there aren't any native profiles yet, just the native flag for activation. Good catch, thanks, the profile is called `native-image` let me fix 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 issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
ppalaga commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548430017 > if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope Maybe I am missing something trivial, but adding `@ApplicationScoped` to a class extending `RouteBuilder` does not make `@Inject` in that same class work. The itest in this PR implements exactly that scenario and I was not able to make it work without the proposed fix. Or you mean something else by "annotate the route"? > still need to fix the problem of having the same route discovered at runtime and at build time Good point, let me add a test for that. 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] dhartford commented on issue #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page
dhartford commented on issue #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page URL: https://github.com/apache/camel-quarkus/pull/355#issuecomment-548430215 Cross-check: For all the references for `-Pnative`, do you mean `-Dnative`? I just tested and there aren't any native profiles yet, just the native flag for activation. 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 #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
lburgazzoli edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548416614 I'm wonder if it worth doing that as there's basic dependency injection in camel that works out of the box [(example](https://github.com/apache/camel/blob/master/core/camel-main/src/test/java/org/apache/camel/main/MainIoCTest.java#L102-L156)) and if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope or any similar annotation (we still need to fix the problem of having the same route discovered at runtime and at build time, but that's something we need to do in any case). I would really like to keep instantiating routes in static init as much as possible. 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] dhartford commented on issue #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page
dhartford commented on issue #355: Fix #354 Update contributor guide: s/json/yaml/, extension adoc page URL: https://github.com/apache/camel-quarkus/pull/355#issuecomment-548427942 @ppalaga The only two areas I can think of would be: 1. Reference how to safely git rebase/squash commits before a PR. There are a lot of references on the internet, however not all are made equal as I found out. 2. A starter-guide for how to identify classes that need special reflection handling for GraalVM/native builds. I know that isn't necessarily camel-specific, but likely to have a lot of experienced java developers that want to contribute that will be in the dark with that item. I appreciate the addition to the contributor guide for` mvn validate -Plicense,sourcecheck -Dcheckstyle.failOnViolation=true` 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 #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
lburgazzoli edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548416614 I'm wonder if it worth doing that as there's basic dependency injection in camel that works out of the box [(example](https://github.com/apache/camel/blob/master/core/camel-main/src/test/java/org/apache/camel/main/MainIoCTest.java#L102-L156)) if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope or any similar annotation (we still need to fix the problem of having the same route discovered at runtime and at build time, but that's something we need to do in any case). I would really like to keep instantiating routes in static init as much as possible. 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 opened a new pull request #359: Use uppercase FHIR where appropriate
ppalaga opened a new pull request #359: Use uppercase FHIR where appropriate URL: https://github.com/apache/camel-quarkus/pull/359 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-k-runtime] lburgazzoli opened a new issue #182: Ad a specific section in the camel-k-runtime catalog to list loaders and related dependencies
lburgazzoli opened a new issue #182: Ad a specific section in the camel-k-runtime catalog to list loaders and related dependencies URL: https://github.com/apache/camel-k-runtime/issues/182 As today we do hard-code loaders dependencies in camel-k but as that's really tight to camel-k-runtime, we should add a section in the camel-k-runtime generated catalog to list the dependency for each loader/language we support 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 #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
lburgazzoli edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548416614 I'm wonder if it worth doing that as there's basic dependency injection in camel that works out of the box and if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope or any similar annotation (we still need to fix the problem of having the same route discovered at runtime and at build time, but that's something we need to do in any case). If possible I would really like to keep instantiating routes in static init as much as possible. 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 #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
lburgazzoli edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548416614 I'm wonder if it worth doing that as there's basic dependency injection in camel that works out of the box and if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope or any similar annotation (we still need to fix the problem of having the same route discovered at runtime and at build time, but that's something we need to do in any case). I would really like to keep instantiating routes in static init as much as possible. 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 #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548416614 I'm wonder if it worth doing that as there's basic dependency injection in camel that works out of the box and if people need to use MicroProfile / CDI injection then they could simply annotate the route with ApplicationScope or any similar annotation (we still need to fix the problem of having the same route discovered at runtime and at build time). 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: Add readme in examples subfolder
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-k.git The following commit(s) were added to refs/heads/master by this push: new 764681f Add readme in examples subfolder new b12edfc Merge pull request #1034 from apupier/AddReadmeInExampleSubFolder 764681f is described below commit 764681f6cf53ebc1f44095bad737a8fa59b9728f Author: Aurélien Pupier AuthorDate: Wed Oct 30 17:18:18 2019 +0100 Add readme in examples subfolder Signed-off-by: Aurélien Pupier --- examples/Readme.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/examples/Readme.md b/examples/Readme.md new file mode 100644 index 000..89fdbe8 --- /dev/null +++ b/examples/Readme.md @@ -0,0 +1,3 @@ +# Camel K Examples + +This folder contains examples of Camel K integrations defined in various languages. \ No newline at end of file
[GitHub] [camel-k] davsclaus merged pull request #1034: Add readme in examples subfolder
davsclaus merged pull request #1034: Add readme in examples subfolder URL: https://github.com/apache/camel-k/pull/1034 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 #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
ppalaga commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-548406473 This should have been a draft PR. Sorry. 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-kafka] remove invalid property for SEND_BUFFER_CONFIG (#3304)
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 6cb2a76 [camel-kafka] remove invalid property for SEND_BUFFER_CONFIG (#3304) 6cb2a76 is described below commit 6cb2a7617e5eab309057e584ec63f844610ed7c2 Author: Nastya Smirnova AuthorDate: Thu Oct 31 16:37:39 2019 +0200 [camel-kafka] remove invalid property for SEND_BUFFER_CONFIG (#3304) --- .../main/java/org/apache/camel/component/kafka/KafkaConfiguration.java | 1 - 1 file changed, 1 deletion(-) diff --git a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java index 02db043..c561de9 100644 --- a/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java +++ b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaConfiguration.java @@ -337,7 +337,6 @@ public class KafkaConfiguration implements Cloneable, HeaderFilterStrategyAware addPropertyIfNotNull(props, ProducerConfig.COMPRESSION_TYPE_CONFIG, getCompressionCodec()); addPropertyIfNotNull(props, ProducerConfig.RETRIES_CONFIG, getRetries()); addPropertyIfNotNull(props, ProducerConfig.INTERCEPTOR_CLASSES_CONFIG, getInterceptorClasses()); -addPropertyIfNotNull(props, ProducerConfig.SEND_BUFFER_CONFIG, getRetries()); addPropertyIfNotNull(props, ProducerConfig.BATCH_SIZE_CONFIG, getProducerBatchSize()); addPropertyIfNotNull(props, ProducerConfig.CLIENT_ID_CONFIG, getClientId()); addPropertyIfNotNull(props, ProducerConfig.CONNECTIONS_MAX_IDLE_MS_CONFIG, getConnectionMaxIdleMs());
[GitHub] [camel-quarkus] ppalaga opened a new pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
ppalaga opened a new pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358 The main idea here is to provide a custom `QuarkusInjector` that first looks up the given type in the Arc container before falling back to direct instantiation. At the same time RoutesBuilders are not instantiated by us, we rather supply the RoutesBuilder class names to Camel main to instantiate them via our `QuarkusInjector`. Let's discuss whether the above is a good idea. I see the following possible risks: 1. Slower instantiation and thus slower boot because all types are first looked up in the Arc container. I should perhaps try to measure the difference. 2. The instantiation of the RoutesBuilders moves from STATIC_INIT to RUNTIME_INIT. Native image boot time may get worse due to this. Both 1. and 2. could get mitigated for RoutesBuilders that do not contain any injection points by filtering out those and instantiating them as we do now, whereas only RoutesBuilders with injection points would be instantiated as sketched in this proposal. The question is whether the more complex implementation is worth the effort. 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] 02/03: CAMEL-14119: Endpoint DSL generator did not include all options.
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
commit 423daf2dd08879fb74fd9d71e9e44409bf085416
Author: Claus Ibsen
AuthorDate: Thu Oct 31 15:12:52 2019 +0100
CAMEL-14119: Endpoint DSL generator did not include all options.
---
.../java/org/apache/camel/maven/packaging/EndpointDslMojo.java| 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
index 4fce633..9448ceb 100644
---
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
+++
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
@@ -317,15 +317,15 @@ public class EndpointDslMojo extends AbstractMojo {
if (option.getLabel() != null) {
if (option.getLabel().contains("producer")) {
if (option.getLabel().contains("advanced")) {
-targets.add(advancedProducerClass);
+targets.add(advancedProducerClass != null ?
advancedProducerClass : advancedBuilderClass);
} else {
-targets.add(producerClass);
+targets.add(producerClass != null ? producerClass :
builderClass);
}
} else if (option.getLabel().contains("consumer")) {
if (option.getLabel().contains("advanced")) {
-targets.add(advancedConsumerClass);
+targets.add(advancedConsumerClass != null ?
advancedConsumerClass : advancedBuilderClass);
} else {
-targets.add(consumerClass);
+targets.add(consumerClass != null ? consumerClass :
builderClass);
}
} else {
if (option.getLabel().contains("advanced")) {
[camel] 01/03: 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
commit 5e60a587c0566bbbfe7e44783e810e53915b0caf
Author: Claus Ibsen
AuthorDate: Thu Oct 31 14:48:49 2019 +0100
Polished
---
.../camel/maven/packaging/EndpointDslMojo.java | 27 ++
1 file changed, 7 insertions(+), 20 deletions(-)
diff --git
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
index bc1542b..4fce633 100644
---
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
+++
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
@@ -208,10 +208,11 @@ public class EndpointDslMojo extends AbstractMojo {
Map enumClasses = new HashMap<>();
-boolean advanced = false;
+boolean hasAdvanced = false;
for (EndpointOptionModel option : model.getEndpointOptions()) {
if (option.getLabel().contains("advanced")) {
-advanced = true;
+hasAdvanced = true;
+break;
}
}
@@ -228,7 +229,7 @@ public class EndpointDslMojo extends AbstractMojo {
generateDummyClass(consumerClass.getCanonicalName());
consumerClass.getJavaDoc().setText("Builder for endpoint consumers
for the " + model.getTitle() + " component.");
-if (advanced) {
+if (hasAdvanced) {
advancedConsumerClass =
javaClass.addNestedType().setPublic().setClass(false);
advancedConsumerClass.setName("Advanced" + consumerName);
advancedConsumerClass.implementInterface("EndpointConsumerBuilder");
@@ -248,7 +249,7 @@ public class EndpointDslMojo extends AbstractMojo {
generateDummyClass(producerClass.getCanonicalName());
producerClass.getJavaDoc().setText("Builder for endpoint producers
for the " + model.getTitle() + " component.");
-if (advanced) {
+if (hasAdvanced) {
advancedProducerClass =
javaClass.addNestedType().setPublic().setClass(false);
advancedProducerClass.setName("Advanced" + producerName);
advancedProducerClass.implementInterface("EndpointProducerBuilder");
@@ -276,7 +277,7 @@ public class EndpointDslMojo extends AbstractMojo {
}
generateDummyClass(builderClass.getCanonicalName());
builderClass.getJavaDoc().setText("Builder for endpoint for the " +
model.getTitle() + " component.");
-if (advanced) {
+if (hasAdvanced) {
advancedBuilderClass =
javaClass.addNestedType().setPublic().setClass(false);
advancedBuilderClass.setName("Advanced" + builderName);
if
(realEndpointClass.getAnnotation(UriEndpoint.class).producerOnly()) {
@@ -522,20 +523,6 @@ public class EndpointDslMojo extends AbstractMojo {
return wrapped;
}
-private String getMethodName(String type) {
-String builderName = getEndpointName(type);
-String methodName = builderName.replace("EndpointBuilder", "");
-methodName = methodName.substring(0, 1).toLowerCase() +
methodName.substring(1);
-switch (type) {
-case "org.apache.camel.component.rest.RestComponent":
-return "restEndpoint";
-case "org.apache.camel.component.beanclass.ClassComponent":
-return "classEndpoint";
-default:
-return methodName;
-}
-}
-
private String getEndpointName(String type) {
int pos = type.lastIndexOf(".");
String name = type.substring(pos + 1).replace("Component",
"EndpointBuilder");
@@ -608,7 +595,7 @@ public class EndpointDslMojo extends AbstractMojo {
return field;
}
-static boolean isPrimitive(String type) {
+private static boolean isPrimitive(String type) {
return PRIMITIVEMAP.containsKey(type);
}
[camel] branch master updated (1a32473 -> 137888f)
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 1a32473 Polished new 5e60a58 Polished new 423daf2 CAMEL-14119: Endpoint DSL generator did not include all options. new 137888f CAMEL-14119: Regen to include all the missing options in the Endpoint DSL (there were a lot) The 3 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: .../endpoint/dsl/AhcEndpointBuilderFactory.java| 86 ++ .../endpoint/dsl/AtomEndpointBuilderFactory.java | 507 +++ .../dsl/AtomixMultiMapEndpointBuilderFactory.java | 95 ++ .../endpoint/dsl/BeanEndpointBuilderFactory.java | 38 + .../dsl/BeanValidatorEndpointBuilderFactory.java | 40 + .../endpoint/dsl/BonitaEndpointBuilderFactory.java | 95 ++ .../endpoint/dsl/CMEndpointBuilderFactory.java | 38 + .../dsl/ChatScriptEndpointBuilderFactory.java | 40 + .../endpoint/dsl/ChunkEndpointBuilderFactory.java | 38 + .../endpoint/dsl/CinderEndpointBuilderFactory.java | 39 + .../endpoint/dsl/ClassEndpointBuilderFactory.java | 38 + .../dsl/ControlBusEndpointBuilderFactory.java | 40 + .../dsl/CryptoCmsEndpointBuilderFactory.java | 40 + .../endpoint/dsl/CwEndpointBuilderFactory.java | 38 + .../dsl/DataFormatEndpointBuilderFactory.java | 40 + .../dsl/DataSetTestEndpointBuilderFactory.java | 372 + .../endpoint/dsl/DdbEndpointBuilderFactory.java| 38 + .../dsl/DdbStreamEndpointBuilderFactory.java | 624 + .../dsl/DebeziumMySqlEndpointBuilderFactory.java | 1439 .../DebeziumPostgresEndpointBuilderFactory.java| +++ .../dsl/DigitalOceanEndpointBuilderFactory.java| 40 + .../DigitalSignatureEndpointBuilderFactory.java| 40 + .../endpoint/dsl/DnsEndpointBuilderFactory.java| 38 + .../endpoint/dsl/DozerEndpointBuilderFactory.java | 38 + .../endpoint/dsl/DrillEndpointBuilderFactory.java | 38 + .../endpoint/dsl/EC2EndpointBuilderFactory.java| 150 ++ .../endpoint/dsl/ECSEndpointBuilderFactory.java| 144 ++ .../endpoint/dsl/EKSEndpointBuilderFactory.java| 144 ++ .../dsl/ElasticsearchEndpointBuilderFactory.java | 40 + .../endpoint/dsl/ExecEndpointBuilderFactory.java | 38 + .../dsl/FileWatchEndpointBuilderFactory.java | 313 + .../endpoint/dsl/FlinkEndpointBuilderFactory.java | 38 + .../endpoint/dsl/FopEndpointBuilderFactory.java| 38 + .../dsl/FreemarkerEndpointBuilderFactory.java | 40 + .../dsl/GangliaEndpointBuilderFactory.java | 40 + .../dsl/GeoCoderEndpointBuilderFactory.java| 40 + .../endpoint/dsl/GlanceEndpointBuilderFactory.java | 39 + .../dsl/GoogleBigQueryEndpointBuilderFactory.java | 40 + .../GoogleBigQuerySQLEndpointBuilderFactory.java | 40 + ...GoogleCalendarStreamEndpointBuilderFactory.java | 520 +++ .../GoogleMailStreamEndpointBuilderFactory.java| 517 +++ .../GoogleSheetsStreamEndpointBuilderFactory.java | 518 +++ .../endpoint/dsl/GrapeEndpointBuilderFactory.java | 38 + .../dsl/GraphqlEndpointBuilderFactory.java | 40 + ...azelcastAtomicnumberEndpointBuilderFactory.java | 40 + .../HazelcastInstanceEndpointBuilderFactory.java | 95 ++ .../HazelcastRingbufferEndpointBuilderFactory.java | 40 + .../endpoint/dsl/HttpEndpointBuilderFactory.java | 601 .../endpoint/dsl/IAMEndpointBuilderFactory.java| 144 ++ .../endpoint/dsl/IPFSEndpointBuilderFactory.java | 38 + .../dsl/IgniteComputeEndpointBuilderFactory.java | 149 ++ .../dsl/IgniteEventsEndpointBuilderFactory.java| 136 ++ .../dsl/IgniteIdGenEndpointBuilderFactory.java | 116 ++ .../dsl/IgniteQueueEndpointBuilderFactory.java | 145 ++ .../dsl/IgniteSetEndpointBuilderFactory.java | 102 ++ .../dsl/InfluxDbEndpointBuilderFactory.java| 40 + .../endpoint/dsl/JMXEndpointBuilderFactory.java| 162 +++ .../endpoint/dsl/JdbcEndpointBuilderFactory.java | 38 + .../dsl/JettyHttpEndpointBuilder9Factory.java | 593 .../endpoint/dsl/JingEndpointBuilderFactory.java | 38 + .../endpoint/dsl/JoltEndpointBuilderFactory.java | 38 + .../dsl/JsonValidatorEndpointBuilderFactory.java | 40 + .../endpoint/dsl/KMSEndpointBuilderFactory.java| 143 ++ .../dsl/KeystoneEndpointBuilderFactory.java| 40 + .../dsl/KinesisFirehoseEndpointBuilderFactory.java | 40 + ...KubernetesConfigMapsEndpointBuilderFactory.java | 51 + ...sistentVolumesClaimsEndpointBuilderFactory.java | 52 + ...tesPersistentVolumesEndpointBuilderFactory.java | 52 + ...rnetesResourcesQuotaEndpointBuilderFactory.java | 52 +
[GitHub] [camel-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341163207 ## File path: deploy/camel-catalog-quarkus-0.3.0-1.0.7-SNAPSHOT.yaml ## @@ -0,0 +1,337 @@ +# --- +# 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. +# --- + +apiVersion: camel.apache.org/v1alpha1 +kind: CamelCatalog +metadata: + name: camel-catalog-quarkus-0.3.0-1.0.7-snapshot + labels: +app: camel-k +camel.apache.org/catalog.version: 3.0.0-RC3 +camel.apache.org/catalog.loader.version: 3.0.0-RC3 +camel.apache.org/runtime.version: 1.0.7-SNAPSHOT +camel.apache.org/runtime.provider: quarkus +spec: + version: 3.0.0-RC3 + runtimeVersion: 1.0.7-SNAPSHOT + runtimeProvider: +quarkus: + camelQuarkusVersion: 0.3.0 + quarkusVersion: 0.26.1 + artifacts: +camel-k-loader-groovy: + groupId: org.apache.camel.k + artifactId: camel-k-loader-groovy Review comment: My understanding is that it's covered in the `addRuntimeDependencies` of the quarkus trait. 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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341145249
##
File path: pkg/builder/runtime/quarkus.go
##
@@ -0,0 +1,148 @@
+package runtime
+
+import (
+ "bufio"
+ "bytes"
+ "fmt"
+ "os"
+ "path"
+
+ "github.com/pkg/errors"
+
+ "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+ "github.com/apache/camel-k/pkg/builder"
+ "github.com/apache/camel-k/pkg/util/camel"
+ "github.com/apache/camel-k/pkg/util/defaults"
+ "github.com/apache/camel-k/pkg/util/maven"
+)
+
+// QuarkusSteps --
+var QuarkusSteps = []builder.Step{
+ Steps.LoadCamelQuarkusCatalog,
+ Steps.GenerateQuarkusProject,
+ Steps.ComputeQuarkusDependencies,
+}
+
+func loadCamelQuarkusCatalog(ctx *builder.Context) error {
+ catalog, err := camel.LoadCatalog(ctx.C, ctx.Client,
ctx.Build.Meta.Namespace, ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus)
+ if err != nil {
+ return err
+ }
+
+ if catalog == nil {
+ return fmt.Errorf("unable to find catalog matching version
requirement: camel=%s, runtime=%s, camel-quarkus=%s, quarkus=%s",
+ ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion)
+ }
+
+ ctx.Catalog = catalog
+
+ return nil
+}
+
+func generateQuarkusProject(ctx *builder.Context) error {
+ p := maven.NewProjectWithGAV("org.apache.camel.k.integration",
"camel-k-integration", defaults.Version)
+ p.Properties = ctx.Build.Platform.Build.Properties
+ p.DependencyManagement = {Dependencies:
make([]maven.Dependency, 0)}
+ p.Dependencies = make([]maven.Dependency, 0)
+ p.Build = {Plugins: make([]maven.Plugin, 0)}
+
+ // DependencyManagement
+ p.DependencyManagement.Dependencies =
append(p.DependencyManagement.Dependencies,
+ maven.Dependency{
+ GroupID:"org.apache.camel.quarkus",
+ ArtifactID: "camel-quarkus-bom",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ maven.Dependency{
+ GroupID:"org.apache.camel.k",
+ ArtifactID: "camel-k-runtime-bom",
+ Version:ctx.Build.RuntimeVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ )
+
+ // Plugins
+ p.Build.Plugins = append(p.Build.Plugins,
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-bootstrap-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ },
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ Executions: []maven.Execution{
+ {
+ Goals: []string{
+ "build",
+ },
+ },
+ },
+ },
+ )
+
+ ctx.Maven.Project = p
+
+ return nil
+}
+
+func computeQuarkusDependencies(ctx *builder.Context) error {
+ mc := maven.NewContext(path.Join(ctx.Path, "maven"), ctx.Maven.Project)
+ mc.SettingsContent = ctx.Maven.SettingsData
+ mc.LocalRepository = ctx.Build.Platform.Build.Maven.LocalRepository
+ mc.Timeout = ctx.Build.Platform.Build.Maven.Timeout.Duration
+
+ // Build the project, as the quarkus-bootstrap plugin build-tree goal
+ // requires the artifact to be installed
+ mc.AddArgument("install")
+ if err := maven.Run(mc); err != nil {
+ return errors.Wrap(err, "failure while building project")
+ }
+
+ // Call the Quarkus dependencies plugin
+ mc.AdditionalArguments = nil
+ mc.AddArguments("quarkus-bootstrap:build-tree")
Review comment:
I've just done a quick test and that seems to work. So the theory seems
valid at least on simple examples :)
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
[camel] 01/02: 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 commit 0ecbe53674b6321cc3c8169b524737bb2589f4a6 Author: Claus Ibsen AuthorDate: Thu Oct 31 14:36:40 2019 +0100 Polished --- components/camel-http/src/main/docs/http-component.adoc | 8 .../main/java/org/apache/camel/component/http/HttpEndpoint.java | 8 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/components/camel-http/src/main/docs/http-component.adoc b/components/camel-http/src/main/docs/http-component.adoc index 8b1dcf0..c8c1bd5 100644 --- a/components/camel-http/src/main/docs/http-component.adoc +++ b/components/camel-http/src/main/docs/http-component.adoc @@ -110,15 +110,11 @@ with the following path and query parameters: | *disableStreamCache* (common) | Determines whether or not the raw input stream from Servlet is cached or not (Camel will read the stream into a in memory/overflow to file, Stream caching) cache. By default Camel will cache the Servlet input stream to support reading it multiple times to ensure it Camel can retrieve all data from the stream. However you can set this option to true when you for example need to access the raw stream, such as streaming it directly to a file or other persis [...] | *headerFilterStrategy* (common) | To use a custom HeaderFilterStrategy to filter header to and from Camel message. | | HeaderFilterStrategy | *httpBinding* (common) | To use a custom HttpBinding to control the mapping between Camel message and HttpClient. | | HttpBinding -| *authenticationPreemptive* (producer) | If this option is true, camel-http sends preemptive basic authentication to the server. | false | boolean | *bridgeEndpoint* (producer) | If the option is true, HttpProducer will ignore the Exchange.HTTP_URI header, and use the endpoint's URI for request. You may also set the option throwExceptionOnFailure to be false to let the HttpProducer send all the fault response back. | false | boolean | *chunked* (producer) | If this option is false the Servlet will disable the HTTP streaming and set the content-length header on the response | true | boolean | *clearExpiredCookies* (producer) | Whether to clear expired cookies before sending the HTTP request. This ensures the cookies store does not keep growing by adding new cookies which is newer removed when they are expired. | true | boolean | *connectionClose* (producer) | Specifies whether a Connection Close header must be added to HTTP Request. By default connectionClose is false. | false | boolean -| *cookieStore* (producer) | To use a custom CookieStore. By default the BasicCookieStore is used which is an in-memory only cookie store. Notice if bridgeEndpoint=true then the cookie store is forced to be a noop cookie store as cookie shouldn't be stored as we are just bridging (eg acting as a proxy). If a cookieHandler is set then the cookie store is also forced to be a noop cookie store as cookie handling is then performed by the cookieHandler. | | CookieStore | *copyHeaders* (producer) | If this option is true then IN exchange headers will be copied to OUT exchange headers according to copy strategy. Setting this to false, allows to only include the headers from the HTTP response (not propagating IN headers). | true | boolean -| *deleteWithBody* (producer) | Whether the HTTP DELETE should include the message body or not. By default HTTP DELETE do not include any HTTP body. However in some rare cases users may need to be able to include the message body. | false | boolean -| *getWithBody* (producer) | Whether the HTTP GET should include the message body or not. By default HTTP GET do not include any HTTP body. However in some rare cases users may need to be able to include the message body. | false | boolean | *httpMethod* (producer) | Configure the HTTP method to use. The HttpMethod header cannot override this option if set. | | HttpMethods | *ignoreResponseBody* (producer) | If this option is true, The http producer won't read response body and cache the input stream | false | boolean | *lazyStartProducer* (producer) | Whether the producer should be started lazy (on the first message). By starting lazy you can use this to allow CamelContext and routes to startup in situations where a producer may otherwise fail during starting and cause the route to fail being started. By deferring this startup to be lazy then the startup failure can be handled during routing messages via Camel's routing error handlers. Beware that when the first message is processed then creating and [...] @@ -126,6 +122,9 @@ with the following path and query parameters: | *throwExceptionOnFailure* (producer) | Option to disable throwing the HttpOperationFailedException in case of failed responses from the remote server. This allows you to get all responses regardless of the HTTP
[camel] 02/02: 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
commit 1a32473d36aa9532aa0d1c008e23498b34396b6e
Author: Claus Ibsen
AuthorDate: Thu Oct 31 14:46:01 2019 +0100
Polished
---
.../main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java| 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
index 01404a6..bc1542b 100644
---
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
+++
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
@@ -64,8 +64,7 @@ import static
org.apache.camel.maven.packaging.PackageHelper.findCamelDirectory;
import static org.apache.camel.maven.packaging.PackageHelper.loadText;
/**
- * Generate Spring Boot auto configuration files for Camel components and data
- * formats.
+ * Generate Endpoint DSL source files for Components.
*/
@Mojo(name = "generate-endpoint-dsl", threadSafe = true,
requiresDependencyResolution = ResolutionScope.COMPILE_PLUS_RUNTIME,
defaultPhase = LifecyclePhase.PROCESS_CLASSES)
public class EndpointDslMojo extends AbstractMojo {
[camel] branch master updated (354c301 -> 1a32473)
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 354c301 CAMEL-14116: Ignore non-ascii chars from description and move javaDocs into the setters instead of fields (#3303) new 0ecbe53 Polished new 1a32473 Polished 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: components/camel-http/src/main/docs/http-component.adoc | 8 .../main/java/org/apache/camel/component/http/HttpEndpoint.java | 8 .../java/org/apache/camel/maven/packaging/EndpointDslMojo.java| 3 +-- 3 files changed, 9 insertions(+), 10 deletions(-)
[GitHub] [camel-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341121532
##
File path: pkg/trait/environment.go
##
@@ -63,7 +64,12 @@ func (t *environmentTrait) Apply(e *Environment) error {
envvar.SetValFrom(, envVarPodName, "metadata.name")
}
- envvar.SetVal(, envVarMainClass, defaultMainClass)
+ quarkus := e.Catalog.GetTrait("quarkus").(*quarkusTrait)
+ if quarkus.isEnabled() {
+ quarkus.addContainerEnvironment(e)
+ } else {
+ envvar.SetVal(, envVarMainClass, defaultMainClass)
Review comment:
Absolutely. It's all scattered so it should be done 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-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341125379 ## File path: deploy/camel-catalog-quarkus-0.3.0-1.0.7-SNAPSHOT.yaml ## @@ -0,0 +1,337 @@ +# --- +# 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. +# --- + +apiVersion: camel.apache.org/v1alpha1 +kind: CamelCatalog +metadata: + name: camel-catalog-quarkus-0.3.0-1.0.7-snapshot + labels: +app: camel-k +camel.apache.org/catalog.version: 3.0.0-RC3 +camel.apache.org/catalog.loader.version: 3.0.0-RC3 +camel.apache.org/runtime.version: 1.0.7-SNAPSHOT +camel.apache.org/runtime.provider: quarkus Review comment: yep, it is just for consistency 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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341123862 ## File path: docs/modules/ROOT/pages/traits.adoc ## @@ -394,6 +394,24 @@ It's disabled by default. !=== +| quarkus +| All +| Activates the Quarkus runtime + + + + +It's disabled by default. + +[cols="m,"] +!=== + +! quarkus.camel-quarkus-version +! The Camel Quarkus version to use for the integration + +! quarkus.quarkus-version +! The Quarkus version to use for the integration Review comment: I see then that the documentation should reflect the above :) Or get rid of the `quarkus-version` 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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341123862 ## File path: docs/modules/ROOT/pages/traits.adoc ## @@ -394,6 +394,24 @@ It's disabled by default. !=== +| quarkus +| All +| Activates the Quarkus runtime + + + + +It's disabled by default. + +[cols="m,"] +!=== + +! quarkus.camel-quarkus-version +! The Camel Quarkus version to use for the integration + +! quarkus.quarkus-version +! The Quarkus version to use for the integration Review comment: I see then that the documentation should reflect the above :) 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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341123066 ## File path: deploy/camel-catalog-quarkus-0.3.0-1.0.7-SNAPSHOT.yaml ## @@ -0,0 +1,337 @@ +# --- +# 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. +# --- + +apiVersion: camel.apache.org/v1alpha1 +kind: CamelCatalog +metadata: + name: camel-catalog-quarkus-0.3.0-1.0.7-snapshot + labels: +app: camel-k +camel.apache.org/catalog.version: 3.0.0-RC3 +camel.apache.org/catalog.loader.version: 3.0.0-RC3 +camel.apache.org/runtime.version: 1.0.7-SNAPSHOT +camel.apache.org/runtime.provider: quarkus Review comment: The Quarkus versions are included in the `.spec.RuntimeProvider.Quarkus` struct, but we can add extra annotations for them. 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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341122502 ## File path: docs/modules/ROOT/pages/traits.adoc ## @@ -394,6 +394,24 @@ It's disabled by default. !=== +| quarkus +| All +| Activates the Quarkus runtime + + + + +It's disabled by default. + +[cols="m,"] +!=== + +! quarkus.camel-quarkus-version +! The Camel Quarkus version to use for the integration + +! quarkus.quarkus-version +! The Quarkus version to use for the integration Review comment: Ultimately, the relation is captured in the matching catalog and taken from there. These parameters are only used to select the match the Camel catalog to be used and that is prescriptive in term of compatibility. 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-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341122744
##
File path: pkg/builder/runtime/quarkus.go
##
@@ -0,0 +1,148 @@
+package runtime
+
+import (
+ "bufio"
+ "bytes"
+ "fmt"
+ "os"
+ "path"
+
+ "github.com/pkg/errors"
+
+ "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+ "github.com/apache/camel-k/pkg/builder"
+ "github.com/apache/camel-k/pkg/util/camel"
+ "github.com/apache/camel-k/pkg/util/defaults"
+ "github.com/apache/camel-k/pkg/util/maven"
+)
+
+// QuarkusSteps --
+var QuarkusSteps = []builder.Step{
+ Steps.LoadCamelQuarkusCatalog,
+ Steps.GenerateQuarkusProject,
+ Steps.ComputeQuarkusDependencies,
+}
+
+func loadCamelQuarkusCatalog(ctx *builder.Context) error {
+ catalog, err := camel.LoadCatalog(ctx.C, ctx.Client,
ctx.Build.Meta.Namespace, ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus)
+ if err != nil {
+ return err
+ }
+
+ if catalog == nil {
+ return fmt.Errorf("unable to find catalog matching version
requirement: camel=%s, runtime=%s, camel-quarkus=%s, quarkus=%s",
+ ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion)
+ }
+
+ ctx.Catalog = catalog
+
+ return nil
+}
+
+func generateQuarkusProject(ctx *builder.Context) error {
+ p := maven.NewProjectWithGAV("org.apache.camel.k.integration",
"camel-k-integration", defaults.Version)
+ p.Properties = ctx.Build.Platform.Build.Properties
+ p.DependencyManagement = {Dependencies:
make([]maven.Dependency, 0)}
+ p.Dependencies = make([]maven.Dependency, 0)
+ p.Build = {Plugins: make([]maven.Plugin, 0)}
+
+ // DependencyManagement
+ p.DependencyManagement.Dependencies =
append(p.DependencyManagement.Dependencies,
+ maven.Dependency{
+ GroupID:"org.apache.camel.quarkus",
+ ArtifactID: "camel-quarkus-bom",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ maven.Dependency{
+ GroupID:"org.apache.camel.k",
+ ArtifactID: "camel-k-runtime-bom",
+ Version:ctx.Build.RuntimeVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ )
+
+ // Plugins
+ p.Build.Plugins = append(p.Build.Plugins,
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-bootstrap-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ },
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ Executions: []maven.Execution{
+ {
+ Goals: []string{
+ "build",
+ },
+ },
+ },
+ },
+ )
+
+ ctx.Maven.Project = p
+
+ return nil
+}
+
+func computeQuarkusDependencies(ctx *builder.Context) error {
+ mc := maven.NewContext(path.Join(ctx.Path, "maven"), ctx.Maven.Project)
+ mc.SettingsContent = ctx.Maven.SettingsData
+ mc.LocalRepository = ctx.Build.Platform.Build.Maven.LocalRepository
+ mc.Timeout = ctx.Build.Platform.Build.Maven.Timeout.Duration
+
+ // Build the project, as the quarkus-bootstrap plugin build-tree goal
+ // requires the artifact to be installed
+ mc.AddArgument("install")
+ if err := maven.Run(mc); err != nil {
+ return errors.Wrap(err, "failure while building project")
+ }
+
+ // Call the Quarkus dependencies plugin
+ mc.AdditionalArguments = nil
+ mc.AddArguments("quarkus-bootstrap:build-tree")
Review comment:
Theoretically everything should be derived from the runtime artefacts, but
need to be proven :)
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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341121532
##
File path: pkg/trait/environment.go
##
@@ -63,7 +64,12 @@ func (t *environmentTrait) Apply(e *Environment) error {
envvar.SetValFrom(, envVarPodName, "metadata.name")
}
- envvar.SetVal(, envVarMainClass, defaultMainClass)
+ quarkus := e.Catalog.GetTrait("quarkus").(*quarkusTrait)
+ if quarkus.isEnabled() {
+ quarkus.addContainerEnvironment(e)
+ } else {
+ envvar.SetVal(, envVarMainClass, defaultMainClass)
Review comment:
Absolutely. It's all scattered so I should be done 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-k] astefanutti commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
astefanutti commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341121310
##
File path: pkg/builder/runtime/quarkus.go
##
@@ -0,0 +1,148 @@
+package runtime
+
+import (
+ "bufio"
+ "bytes"
+ "fmt"
+ "os"
+ "path"
+
+ "github.com/pkg/errors"
+
+ "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+ "github.com/apache/camel-k/pkg/builder"
+ "github.com/apache/camel-k/pkg/util/camel"
+ "github.com/apache/camel-k/pkg/util/defaults"
+ "github.com/apache/camel-k/pkg/util/maven"
+)
+
+// QuarkusSteps --
+var QuarkusSteps = []builder.Step{
+ Steps.LoadCamelQuarkusCatalog,
+ Steps.GenerateQuarkusProject,
+ Steps.ComputeQuarkusDependencies,
+}
+
+func loadCamelQuarkusCatalog(ctx *builder.Context) error {
+ catalog, err := camel.LoadCatalog(ctx.C, ctx.Client,
ctx.Build.Meta.Namespace, ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus)
+ if err != nil {
+ return err
+ }
+
+ if catalog == nil {
+ return fmt.Errorf("unable to find catalog matching version
requirement: camel=%s, runtime=%s, camel-quarkus=%s, quarkus=%s",
+ ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion)
+ }
+
+ ctx.Catalog = catalog
+
+ return nil
+}
+
+func generateQuarkusProject(ctx *builder.Context) error {
+ p := maven.NewProjectWithGAV("org.apache.camel.k.integration",
"camel-k-integration", defaults.Version)
+ p.Properties = ctx.Build.Platform.Build.Properties
+ p.DependencyManagement = {Dependencies:
make([]maven.Dependency, 0)}
+ p.Dependencies = make([]maven.Dependency, 0)
+ p.Build = {Plugins: make([]maven.Plugin, 0)}
+
+ // DependencyManagement
+ p.DependencyManagement.Dependencies =
append(p.DependencyManagement.Dependencies,
+ maven.Dependency{
+ GroupID:"org.apache.camel.quarkus",
+ ArtifactID: "camel-quarkus-bom",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ maven.Dependency{
+ GroupID:"org.apache.camel.k",
+ ArtifactID: "camel-k-runtime-bom",
+ Version:ctx.Build.RuntimeVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ )
+
+ // Plugins
+ p.Build.Plugins = append(p.Build.Plugins,
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-bootstrap-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ },
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ Executions: []maven.Execution{
+ {
+ Goals: []string{
+ "build",
+ },
+ },
+ },
+ },
+ )
+
+ ctx.Maven.Project = p
+
+ return nil
+}
+
+func computeQuarkusDependencies(ctx *builder.Context) error {
+ mc := maven.NewContext(path.Join(ctx.Path, "maven"), ctx.Maven.Project)
+ mc.SettingsContent = ctx.Maven.SettingsData
+ mc.LocalRepository = ctx.Build.Platform.Build.Maven.LocalRepository
+ mc.Timeout = ctx.Build.Platform.Build.Maven.Timeout.Duration
+
+ // Build the project, as the quarkus-bootstrap plugin build-tree goal
+ // requires the artifact to be installed
+ mc.AddArgument("install")
+ if err := maven.Run(mc); err != nil {
+ return errors.Wrap(err, "failure while building project")
+ }
+
+ // Call the Quarkus dependencies plugin
+ mc.AdditionalArguments = nil
+ mc.AddArguments("quarkus-bootstrap:build-tree")
Review comment:
Ah my understanding was that the augmentation process could add runtime
dependencies as well. Let me try using the standard dependency plugin then!
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:
[camel] branch master updated: Fixed failing link in 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 d2337bc Fixed failing link in doc d2337bc is described below commit d2337bca1f88f800cce9f24d6231803a5318df8a Author: Claus Ibsen AuthorDate: Thu Oct 31 13:57:17 2019 +0100 Fixed failing link in doc --- docs/user-manual/modules/ROOT/pages/faq/how-do-i-edit-the-website.adoc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/user-manual/modules/ROOT/pages/faq/how-do-i-edit-the-website.adoc b/docs/user-manual/modules/ROOT/pages/faq/how-do-i-edit-the-website.adoc index 99bd70a0..9b33f26 100644 --- a/docs/user-manual/modules/ROOT/pages/faq/how-do-i-edit-the-website.adoc +++ b/docs/user-manual/modules/ROOT/pages/faq/how-do-i-edit-the-website.adoc @@ -23,7 +23,7 @@ Steps to edit a file: == In local repositories -To edit files locally, it's important to understand how the website is generated and where the files are located. The https://github.com/apache/camel-website[site generator] is located in its own repository. The documentation is located in the main https://github.com/apache/camel[Apache Camel] repository and sub-project repositories, such as https://github.com/apache/camel-k[Camel-K] and https://github.com/apache/camel-quarkus[Camel Quarkus]. Have a look at the xref:how-does-the-website- [...] +To edit files locally, it's important to understand how the website is generated and where the files are located. The https://github.com/apache/camel-website[site generator] is located in its own repository. The documentation is located in the main https://github.com/apache/camel[Apache Camel] repository and sub-project repositories, such as https://github.com/apache/camel-k[Camel-K] and https://github.com/apache/camel-quarkus[Camel Quarkus]. Have a look at the xref:faq/how-does-the-webs [...] Steps to edit a file:
[camel] branch master updated: Improve summary for camel-jetty
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 e3af268 Improve summary for camel-jetty
e3af268 is described below
commit e3af2685d52196f2c33bf0acf2a22c855cbc6cd4
Author: Claus Ibsen
AuthorDate: Thu Oct 31 13:55:46 2019 +0100
Improve summary for camel-jetty
---
.../java/org/apache/camel/component/jetty9/JettyHttpEndpoint9.java | 3 +--
.../builder/endpoint/dsl/JettyHttpEndpointBuilder9Factory.java | 6 ++
docs/components/modules/ROOT/pages/http-component.adoc | 5 +++--
docs/components/modules/ROOT/pages/index.adoc | 2 +-
.../jetty9/springboot/JettyHttpComponentConfiguration9.java | 3 +--
5 files changed, 8 insertions(+), 11 deletions(-)
diff --git
a/components/camel-jetty/src/main/java/org/apache/camel/component/jetty9/JettyHttpEndpoint9.java
b/components/camel-jetty/src/main/java/org/apache/camel/component/jetty9/JettyHttpEndpoint9.java
index 68808f1..27e8873 100644
---
a/components/camel-jetty/src/main/java/org/apache/camel/component/jetty9/JettyHttpEndpoint9.java
+++
b/components/camel-jetty/src/main/java/org/apache/camel/component/jetty9/JettyHttpEndpoint9.java
@@ -27,8 +27,7 @@ import org.apache.camel.http.common.HttpBinding;
import org.apache.camel.spi.UriEndpoint;
/**
- * The jetty component provides HTTP-based endpoints for consuming and
producing
- * HTTP requests.
+ * To use Jetty as a HTTP server as consumer for Camel routes.
*/
@UriEndpoint(firstVersion = "1.2.0", scheme = "jetty", extendsScheme = "http",
title = "Jetty", syntax = "jetty:httpUri", label = "http", consumerOnly = true,
lenientProperties = true,
excludeProperties =
"authMethod,authMethodPriority,authUsername,authPassword,authDomain,authHost"
diff --git
a/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/JettyHttpEndpointBuilder9Factory.java
b/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/JettyHttpEndpointBuilder9Factory.java
index 071f459..0786fb2 100644
---
a/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/JettyHttpEndpointBuilder9Factory.java
+++
b/core/camel-endpointdsl/src/main/java/org/apache/camel/builder/endpoint/dsl/JettyHttpEndpointBuilder9Factory.java
@@ -23,8 +23,7 @@ import
org.apache.camel.builder.endpoint.AbstractEndpointBuilder;
import org.apache.camel.spi.HeaderFilterStrategy;
/**
- * The jetty component provides HTTP-based endpoints for consuming and
producing
- * HTTP requests.
+ * To use Jetty as a HTTP server as consumer for Camel routes.
*
* Generated by camel-package-maven-plugin - do not edit this file!
*/
@@ -416,8 +415,7 @@ public interface JettyHttpEndpointBuilder9Factory {
}
/**
* Jetty (camel-jetty)
- * The jetty component provides HTTP-based endpoints for consuming and
- * producing HTTP requests.
+ * To use Jetty as a HTTP server as consumer for Camel routes.
*
* Category: http
* Available as of version: 1.2
diff --git a/docs/components/modules/ROOT/pages/http-component.adoc
b/docs/components/modules/ROOT/pages/http-component.adoc
index 237bc23..80e2266 100644
--- a/docs/components/modules/ROOT/pages/http-component.adoc
+++ b/docs/components/modules/ROOT/pages/http-component.adoc
@@ -102,7 +102,7 @@ with the following path and query parameters:
|===
-=== Query Parameters (50 parameters):
+=== Query Parameters (51 parameters):
[width="100%",cols="2,5,^1,2",options="header"]
@@ -118,7 +118,8 @@ with the following path and query parameters:
| *connectionClose* (producer) | Specifies whether a Connection Close header
must be added to HTTP Request. By default connectionClose is false. | false |
boolean
| *cookieStore* (producer) | To use a custom CookieStore. By default the
BasicCookieStore is used which is an in-memory only cookie store. Notice if
bridgeEndpoint=true then the cookie store is forced to be a noop cookie store
as cookie shouldn't be stored as we are just bridging (eg acting as a proxy).
If a cookieHandler is set then the cookie store is also forced to be a noop
cookie store as cookie handling is then performed by the cookieHandler. | |
CookieStore
| *copyHeaders* (producer) | If this option is true then IN exchange headers
will be copied to OUT exchange headers according to copy strategy. Setting this
to false, allows to only include the headers from the HTTP response (not
propagating IN headers). | true | boolean
-| *deleteWithBody* (producer) | Whether the HTTP DELETE should include the
message body or not. By default HTTP DELETE do not include any HTTP message.
However in some rare cases users may need to be able to include the message
body. | false | boolean
+| *deleteWithBody* (producer) | Whether the HTTP DELETE should include
[GitHub] [camel-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341109327 ## File path: docs/modules/ROOT/pages/traits.adoc ## @@ -394,6 +394,24 @@ It's disabled by default. !=== +| quarkus +| All +| Activates the Quarkus runtime + + + + +It's disabled by default. + +[cols="m,"] +!=== + +! quarkus.camel-quarkus-version +! The Camel Quarkus version to use for the integration + +! quarkus.quarkus-version +! The Quarkus version to use for the integration Review comment: Not very sure if we need to let people change this value as the relation between camel and quarkus is really thigh at the moment so I'd remove it or add some version validation. 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-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341112564
##
File path: pkg/builder/runtime/quarkus.go
##
@@ -0,0 +1,148 @@
+package runtime
+
+import (
+ "bufio"
+ "bytes"
+ "fmt"
+ "os"
+ "path"
+
+ "github.com/pkg/errors"
+
+ "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+ "github.com/apache/camel-k/pkg/builder"
+ "github.com/apache/camel-k/pkg/util/camel"
+ "github.com/apache/camel-k/pkg/util/defaults"
+ "github.com/apache/camel-k/pkg/util/maven"
+)
+
+// QuarkusSteps --
+var QuarkusSteps = []builder.Step{
+ Steps.LoadCamelQuarkusCatalog,
+ Steps.GenerateQuarkusProject,
+ Steps.ComputeQuarkusDependencies,
+}
+
+func loadCamelQuarkusCatalog(ctx *builder.Context) error {
+ catalog, err := camel.LoadCatalog(ctx.C, ctx.Client,
ctx.Build.Meta.Namespace, ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus)
+ if err != nil {
+ return err
+ }
+
+ if catalog == nil {
+ return fmt.Errorf("unable to find catalog matching version
requirement: camel=%s, runtime=%s, camel-quarkus=%s, quarkus=%s",
+ ctx.Build.CamelVersion, ctx.Build.RuntimeVersion,
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion)
+ }
+
+ ctx.Catalog = catalog
+
+ return nil
+}
+
+func generateQuarkusProject(ctx *builder.Context) error {
+ p := maven.NewProjectWithGAV("org.apache.camel.k.integration",
"camel-k-integration", defaults.Version)
+ p.Properties = ctx.Build.Platform.Build.Properties
+ p.DependencyManagement = {Dependencies:
make([]maven.Dependency, 0)}
+ p.Dependencies = make([]maven.Dependency, 0)
+ p.Build = {Plugins: make([]maven.Plugin, 0)}
+
+ // DependencyManagement
+ p.DependencyManagement.Dependencies =
append(p.DependencyManagement.Dependencies,
+ maven.Dependency{
+ GroupID:"org.apache.camel.quarkus",
+ ArtifactID: "camel-quarkus-bom",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.CamelQuarkusVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ maven.Dependency{
+ GroupID:"org.apache.camel.k",
+ ArtifactID: "camel-k-runtime-bom",
+ Version:ctx.Build.RuntimeVersion,
+ Type: "pom",
+ Scope: "import",
+ },
+ )
+
+ // Plugins
+ p.Build.Plugins = append(p.Build.Plugins,
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-bootstrap-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ },
+ maven.Plugin{
+ GroupID:"io.quarkus",
+ ArtifactID: "quarkus-maven-plugin",
+ Version:
ctx.Build.RuntimeProvider.Quarkus.QuarkusVersion,
+ Executions: []maven.Execution{
+ {
+ Goals: []string{
+ "build",
+ },
+ },
+ },
+ },
+ )
+
+ ctx.Maven.Project = p
+
+ return nil
+}
+
+func computeQuarkusDependencies(ctx *builder.Context) error {
+ mc := maven.NewContext(path.Join(ctx.Path, "maven"), ctx.Maven.Project)
+ mc.SettingsContent = ctx.Maven.SettingsData
+ mc.LocalRepository = ctx.Build.Platform.Build.Maven.LocalRepository
+ mc.Timeout = ctx.Build.Platform.Build.Maven.Timeout.Duration
+
+ // Build the project, as the quarkus-bootstrap plugin build-tree goal
+ // requires the artifact to be installed
+ mc.AddArgument("install")
+ if err := maven.Run(mc); err != nil {
+ return errors.Wrap(err, "failure while building project")
+ }
+
+ // Call the Quarkus dependencies plugin
+ mc.AdditionalArguments = nil
+ mc.AddArguments("quarkus-bootstrap:build-tree")
Review comment:
do we need that ? as far as I know it should be enough to add runtime
dependencies so it should be enough to use the standard dependency plugin
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:
[GitHub] [camel-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial
Quarkus support
URL: https://github.com/apache/camel-k/pull/1013#discussion_r341114757
##
File path: pkg/trait/environment.go
##
@@ -63,7 +64,12 @@ func (t *environmentTrait) Apply(e *Environment) error {
envvar.SetValFrom(, envVarPodName, "metadata.name")
}
- envvar.SetVal(, envVarMainClass, defaultMainClass)
+ quarkus := e.Catalog.GetTrait("quarkus").(*quarkusTrait)
+ if quarkus.isEnabled() {
+ quarkus.addContainerEnvironment(e)
+ } else {
+ envvar.SetVal(, envVarMainClass, defaultMainClass)
Review comment:
not something for now but we may think about creating a `runtime-main` trait
and handle things homogeneously
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-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341107568 ## File path: deploy/camel-catalog-quarkus-0.3.0-1.0.7-SNAPSHOT.yaml ## @@ -0,0 +1,337 @@ +# --- +# 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. +# --- + +apiVersion: camel.apache.org/v1alpha1 +kind: CamelCatalog +metadata: + name: camel-catalog-quarkus-0.3.0-1.0.7-snapshot + labels: +app: camel-k +camel.apache.org/catalog.version: 3.0.0-RC3 +camel.apache.org/catalog.loader.version: 3.0.0-RC3 +camel.apache.org/runtime.version: 1.0.7-SNAPSHOT +camel.apache.org/runtime.provider: quarkus Review comment: I know it related to camel-k-runtime but I guess we need to include information about the quarkus version 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-14118: Add getWithBody option to camel-http and also fix so an explicit GET wont attempt to read body as its not in use, this helps CAMEL-14115
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 cf8c9a6 CAMEL-14118: Add getWithBody option to camel-http and also
fix so an explicit GET wont attempt to read body as its not in use, this helps
CAMEL-14115
cf8c9a6 is described below
commit cf8c9a6505556e0df7ab426d787de4c1ced24131
Author: Claus Ibsen
AuthorDate: Thu Oct 31 13:41:59 2019 +0100
CAMEL-14118: Add getWithBody option to camel-http and also fix so an
explicit GET wont attempt to read body as its not in use, this helps CAMEL-14115
---
.../camel-http/src/main/docs/http-component.adoc | 5 +--
.../apache/camel/component/http/HttpEndpoint.java | 22 +++--
.../component/http/HttpGetWithBodyMethod.java | 37 ++
.../apache/camel/component/http/HttpProducer.java | 11 +--
.../component/http/helper/HttpMethodHelper.java| 10 ++
.../camel/component/http/HttpMethodsTest.java | 15 +
6 files changed, 85 insertions(+), 15 deletions(-)
diff --git a/components/camel-http/src/main/docs/http-component.adoc
b/components/camel-http/src/main/docs/http-component.adoc
index e965f86..8b1dcf0 100644
--- a/components/camel-http/src/main/docs/http-component.adoc
+++ b/components/camel-http/src/main/docs/http-component.adoc
@@ -101,7 +101,7 @@ with the following path and query parameters:
|===
-=== Query Parameters (50 parameters):
+=== Query Parameters (51 parameters):
[width="100%",cols="2,5,^1,2",options="header"]
@@ -117,7 +117,8 @@ with the following path and query parameters:
| *connectionClose* (producer) | Specifies whether a Connection Close header
must be added to HTTP Request. By default connectionClose is false. | false |
boolean
| *cookieStore* (producer) | To use a custom CookieStore. By default the
BasicCookieStore is used which is an in-memory only cookie store. Notice if
bridgeEndpoint=true then the cookie store is forced to be a noop cookie store
as cookie shouldn't be stored as we are just bridging (eg acting as a proxy).
If a cookieHandler is set then the cookie store is also forced to be a noop
cookie store as cookie handling is then performed by the cookieHandler. | |
CookieStore
| *copyHeaders* (producer) | If this option is true then IN exchange headers
will be copied to OUT exchange headers according to copy strategy. Setting this
to false, allows to only include the headers from the HTTP response (not
propagating IN headers). | true | boolean
-| *deleteWithBody* (producer) | Whether the HTTP DELETE should include the
message body or not. By default HTTP DELETE do not include any HTTP message.
However in some rare cases users may need to be able to include the message
body. | false | boolean
+| *deleteWithBody* (producer) | Whether the HTTP DELETE should include the
message body or not. By default HTTP DELETE do not include any HTTP body.
However in some rare cases users may need to be able to include the message
body. | false | boolean
+| *getWithBody* (producer) | Whether the HTTP GET should include the message
body or not. By default HTTP GET do not include any HTTP body. However in some
rare cases users may need to be able to include the message body. | false |
boolean
| *httpMethod* (producer) | Configure the HTTP method to use. The HttpMethod
header cannot override this option if set. | | HttpMethods
| *ignoreResponseBody* (producer) | If this option is true, The http producer
won't read response body and cache the input stream | false | boolean
| *lazyStartProducer* (producer) | Whether the producer should be started lazy
(on the first message). By starting lazy you can use this to allow CamelContext
and routes to startup in situations where a producer may otherwise fail during
starting and cause the route to fail being started. By deferring this startup
to be lazy then the startup failure can be handled during routing messages via
Camel's routing error handlers. Beware that when the first message is processed
then creating and [...]
diff --git
a/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java
b/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java
index c48d9e8..82d894e 100644
---
a/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java
+++
b/components/camel-http/src/main/java/org/apache/camel/component/http/HttpEndpoint.java
@@ -105,10 +105,12 @@ public class HttpEndpoint extends HttpCommonEndpoint {
private boolean clearExpiredCookies = true;
@UriParam(label = "producer", description = "If this option is true,
camel-http sends preemptive basic authentication to the server.")
private boolean authenticationPreemptive;
+@UriParam(label = "producer", description = "Whether the HTTP GET should
[GitHub] [camel-k] lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support
lburgazzoli commented on a change in pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013#discussion_r341108212 ## File path: deploy/camel-catalog-quarkus-0.3.0-1.0.7-SNAPSHOT.yaml ## @@ -0,0 +1,337 @@ +# --- +# 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. +# --- + +apiVersion: camel.apache.org/v1alpha1 +kind: CamelCatalog +metadata: + name: camel-catalog-quarkus-0.3.0-1.0.7-snapshot + labels: +app: camel-k +camel.apache.org/catalog.version: 3.0.0-RC3 +camel.apache.org/catalog.loader.version: 3.0.0-RC3 +camel.apache.org/runtime.version: 1.0.7-SNAPSHOT +camel.apache.org/runtime.provider: quarkus +spec: + version: 3.0.0-RC3 + runtimeVersion: 1.0.7-SNAPSHOT + runtimeProvider: +quarkus: + camelQuarkusVersion: 0.3.0 + quarkusVersion: 0.26.1 + artifacts: +camel-k-loader-groovy: + groupId: org.apache.camel.k + artifactId: camel-k-loader-groovy Review comment: Also related to camel-k-runtime: there are quarkus specific extension for those runtime so we probably need to replace the artefact coordinates 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-k] lburgazzoli opened a new issue #1037: Get rid of json schema generation
lburgazzoli opened a new issue #1037: Get rid of json schema generation URL: https://github.com/apache/camel-k/issues/1037 We now generates some json schema in `assets/json-schema` but as far as I remember they are not more used so we can get rid of them 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-k] nicolaferraro opened a new issue #1036: Generate full k8s client for provided APIs
nicolaferraro opened a new issue #1036: Generate full k8s client for provided APIs URL: https://github.com/apache/camel-k/issues/1036 So that other Go applications can use the client to directly manage Integrations. 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-14113] camel-elasticsearch-rest: NullPointer exception if there is no body in exchange (#3300)
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 9bff1b6 [CAMEL-14113] camel-elasticsearch-rest: NullPointer exception
if there is no body in exchange (#3300)
9bff1b6 is described below
commit 9bff1b6ed57441804b594c5e61a13987c276f271
Author: JiriOndrusek
AuthorDate: Thu Oct 31 12:58:20 2019 +0100
[CAMEL-14113] camel-elasticsearch-rest: NullPointer exception if there is
no body in exchange (#3300)
---
.../elasticsearch/ElasticsearchProducer.java | 27 ++
.../ElasticsearchActionRequestConverter.java | 16 ++---
2 files changed, 35 insertions(+), 8 deletions(-)
diff --git
a/components/camel-elasticsearch-rest/src/main/java/org/apache/camel/component/elasticsearch/ElasticsearchProducer.java
b/components/camel-elasticsearch-rest/src/main/java/org/apache/camel/component/elasticsearch/ElasticsearchProducer.java
index 66fc65b..31889d8 100644
---
a/components/camel-elasticsearch-rest/src/main/java/org/apache/camel/component/elasticsearch/ElasticsearchProducer.java
+++
b/components/camel-elasticsearch-rest/src/main/java/org/apache/camel/component/elasticsearch/ElasticsearchProducer.java
@@ -149,24 +149,45 @@ public class ElasticsearchProducer extends
DefaultProducer {
if (operation == ElasticsearchOperation.Index) {
IndexRequest indexRequest = message.getBody(IndexRequest.class);
+if (indexRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only Map,
String, byte[], XContentBuilder or IndexRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.index(indexRequest,
RequestOptions.DEFAULT).getId());
} else if (operation == ElasticsearchOperation.Update) {
UpdateRequest updateRequest = message.getBody(UpdateRequest.class);
+if (updateRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only Map,
String, byte[], XContentBuilder or UpdateRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.update(updateRequest,
RequestOptions.DEFAULT).getId());
} else if (operation == ElasticsearchOperation.GetById) {
GetRequest getRequest = message.getBody(GetRequest.class);
+if (getRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only
String or GetRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.get(getRequest,
RequestOptions.DEFAULT));
} else if (operation == ElasticsearchOperation.Bulk) {
BulkRequest bulkRequest = message.getBody(BulkRequest.class);
+if (bulkRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only
List, Collection or BulkRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.bulk(bulkRequest,
RequestOptions.DEFAULT).getItems());
} else if (operation == ElasticsearchOperation.BulkIndex) {
BulkRequest bulkRequest = message.getBody(BulkRequest.class);
+if (bulkRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only
List, Collection or BulkRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.bulk(bulkRequest,
RequestOptions.DEFAULT).getItems());
} else if (operation == ElasticsearchOperation.Delete) {
DeleteRequest deleteRequest = message.getBody(DeleteRequest.class);
+if (deleteRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only
String or DeleteRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.delete(deleteRequest,
RequestOptions.DEFAULT).getResult());
} else if (operation == ElasticsearchOperation.DeleteIndex) {
DeleteIndexRequest deleteIndexRequest =
message.getBody(DeleteIndexRequest.class);
+if (deleteIndexRequest == null) {
+throw new IllegalArgumentException("Wrong body type. Only
String or DeleteIndexRequest is allowed as a type");
+}
message.setBody(restHighLevelClient.indices().delete(deleteIndexRequest,
RequestOptions.DEFAULT).isAcknowledged());
} else if (operation == ElasticsearchOperation.Exists) {
// ExistsRequest API is deprecated, using SearchRequest instead
with size=0 and terminate_after=1
@@ -188,6 +209,9 @@ public class ElasticsearchProducer extends DefaultProducer {
}
} else if (operation == ElasticsearchOperation.Search) {
SearchRequest searchRequest = message.getBody(SearchRequest.class);
+if
[camel] branch master updated (4bf6728 -> ef20a2e)
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 4bf6728 camel-main: add fluent setter for autowireComponentPropertiesNonNullOnly add ef20a2e Regen No new revisions were added by this update. Summary of changes: .../src/main/resources/META-INF/camel-main-configuration-metadata.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[GitHub] [camel-quarkus] ppalaga opened a new issue #357: Make netty-http dependent on netty and remove the duplications
ppalaga opened a new issue #357: Make netty-http dependent on netty and remove the duplications URL: https://github.com/apache/camel-quarkus/issues/357 There will be some duplications between netty and netty-http components once https://github.com/apache/camel-quarkus/pull/353#pullrequestreview-309788273 gets merged. In a follow up step, we should make netty-http dependent on netty and remove the dups. We should also document in the netty.adoc that the native transport is not supported in the native 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
[camel] branch master updated (182f796 -> 4bf6728)
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.git. from 182f796 remove superfluous super() calls which compiler would do for free (#3301) add 56719cd camel-main: add an option to control autowiring of non null properties add 581af67 properties-binding: do not auto wire camel context from registry add af92bc8 Fix findings add 4bf6728 camel-main: add fluent setter for autowireComponentPropertiesNonNullOnly No new revisions were added by this update. Summary of changes: .../org/apache/camel/main/BaseMainSupport.java | 6 +- .../camel/main/MainConfigurationProperties.java| 24 +++ .../camel-main-configuration-metadata.json | 6 + .../org/apache/camel/main/MainIoCAutowireTest.java | 172 + .../camel-main/src/test/resources/empty.properties | 2 +- .../camel/support/PropertyBindingSupport.java | 10 ++ 6 files changed, 216 insertions(+), 4 deletions(-) create mode 100644 core/camel-main/src/test/java/org/apache/camel/main/MainIoCAutowireTest.java copy tooling/maven/camel-package-maven-plugin/src/main/resources/camel-package-maven-plugin.properties => core/camel-main/src/test/resources/empty.properties (98%)
[camel] branch superfluous-super-call deleted (was 0b87036)
This is an automated email from the ASF dual-hosted git repository. bvahdat pushed a change to branch superfluous-super-call in repository https://gitbox.apache.org/repos/asf/camel.git. was 0b87036 remove superfluous super() calls which compiler would do for free The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #353: Camel quarkus netty
ppalaga commented on a change in pull request #353: Camel quarkus netty
URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341070813
##
File path: extensions/netty/runtime/pom.xml
##
@@ -0,0 +1,104 @@
+
+
+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-netty-parent
+0.3.2-SNAPSHOT
+../pom.xml
+
+
+camel-quarkus-netty
+Camel Quarkus :: Netty :: Runtime
+
+
+0.4.0
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-bom
+${project.version}
+pom
+import
+
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-core
+
+
+io.quarkus
+quarkus-netty
+
+
+io.quarkus
+quarkus-caffeine
+
+
+org.apache.camel
+camel-endpointdsl
+
Review comment:
`camel-endpointdsl` can be removed too. The problems you saw were perhaps
cased by the missing deps in the deployment module - see my comments there.
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 #353: Camel quarkus netty
ppalaga commented on a change in pull request #353: Camel quarkus netty
URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341068398
##
File path: extensions/netty/runtime/pom.xml
##
@@ -0,0 +1,104 @@
+
+
+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-netty-parent
+0.3.2-SNAPSHOT
+../pom.xml
+
+
+camel-quarkus-netty
+Camel Quarkus :: Netty :: Runtime
+
+
+0.4.0
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-bom
+${project.version}
+pom
+import
+
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-core
+
+
+io.quarkus
+quarkus-netty
+
+
+io.quarkus
+quarkus-caffeine
+
Review comment:
`quarkus-caffeine` dependency can be removed. It`s transitively there via
camel-quarkus-core and camel-netty does not depend on 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 #353: Camel quarkus netty
ppalaga commented on a change in pull request #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341073366 ## File path: integration-tests/netty/pom.xml ## @@ -0,0 +1,132 @@ + + +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-integration-tests +0.3.2-SNAPSHOT + + +camel-quarkus-integration-test-netty +Camel Quarkus :: Integration Tests :: Netty +Integration tests for Camel Quarkus Netty extension + + + +org.apache.camel.quarkus +camel-quarkus-netty + + +org.apache.camel.quarkus +camel-quarkus-log + Review comment: You do not use `camel-quarkus-log`, plz remove 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 #353: Camel quarkus netty
ppalaga commented on a change in pull request #353: Camel quarkus netty URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341074315 ## File path: integration-tests/netty/pom.xml ## @@ -0,0 +1,132 @@ + + +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-integration-tests +0.3.2-SNAPSHOT + + +camel-quarkus-integration-test-netty +Camel Quarkus :: Integration Tests :: Netty +Integration tests for Camel Quarkus Netty extension + + + +org.apache.camel.quarkus +camel-quarkus-netty + + +org.apache.camel.quarkus +camel-quarkus-log + + +io.quarkus +quarkus-resteasy + Review comment: You do not use `quarkus-resteasy` either, but the test runner needs it transitive dep `quarkus-undertow` to be able to detect that the sever has started. So plese replace resteasy with the following: ``` io.quarkus quarkus-undertow ``` 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 #353: Camel quarkus netty
ppalaga commented on a change in pull request #353: Camel quarkus netty
URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341072903
##
File path: extensions/netty/deployment/pom.xml
##
@@ -0,0 +1,75 @@
+
+
+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-netty-parent
+0.3.2-SNAPSHOT
+../pom.xml
+
+
+camel-quarkus-netty-deployment
+Camel Quarkus :: Netty :: Deployment
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-bom-deployment
+${project.version}
+pom
+import
+
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-core-deployment
+
Review comment:
The -deployment counterparts of all (quarkus or camel-quarkus) extensions
you depend on in the runtime module must be here - so you need to add
```
io.quarkus
quarkus-netty-deployment
```
(see also my comments in the runtime pom before you start considering to add
more)
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 #353: Camel quarkus netty
ppalaga commented on a change in pull request #353: Camel quarkus netty
URL: https://github.com/apache/camel-quarkus/pull/353#discussion_r341071125
##
File path: extensions/netty/runtime/pom.xml
##
@@ -0,0 +1,104 @@
+
+
+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-netty-parent
+0.3.2-SNAPSHOT
+../pom.xml
+
+
+camel-quarkus-netty
+Camel Quarkus :: Netty :: Runtime
+
+
+0.4.0
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-bom
+${project.version}
+pom
+import
+
+
+
+
+
+
+org.apache.camel.quarkus
+camel-quarkus-core
+
+
+io.quarkus
+quarkus-netty
+
+
+io.quarkus
+quarkus-caffeine
+
+
+org.apache.camel
+camel-endpointdsl
+
+
+org.apache.camel
+camel-netty
+
+
+org.apache.camel
+camel-caffeine-lrucache
+
+
Review comment:
camel-netty does not depend on caffeine. The exclusions can be removed.
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 (9cbe43e -> 182f796)
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 9cbe43e Polished add 182f796 remove superfluous super() calls which compiler would do for free (#3301) No new revisions were added by this update. Summary of changes: .../java/org/apache/camel/component/activemq/ActiveMQComponent.java| 1 - .../org/apache/camel/component/as2/api/InvalidAS2NameException.java| 1 - .../as2/api/entity/AS2MessageDispositionNotificationEntity.java| 1 - .../java/org/apache/camel/dataformat/avro/SpecificDataNoCache.java | 1 - .../java/org/apache/camel/component/aws/ddb/AmazonDDBClientMock.java | 1 - .../java/org/apache/camel/component/aws/ecs/AmazonECSClientMock.java | 1 - .../java/org/apache/camel/component/aws/eks/AmazonEKSClientMock.java | 1 - .../camel/component/aws/kinesis/ReachedClosedStatusException.java | 1 - .../java/org/apache/camel/component/aws/kms/AmazonKMSClientMock.java | 1 - .../org/apache/camel/component/aws/lambda/AmazonLambdaClientMock.java | 1 - .../java/org/apache/camel/component/aws/mq/AmazonMQClientMock.java | 1 - .../java/org/apache/camel/component/aws/msk/AmazonMSKClientMock.java | 1 - .../java/org/apache/camel/component/aws/s3/AmazonS3ClientMock.java | 1 - .../apache/camel/component/aws/s3/AmazonS3EncryptionClientMock.java| 1 - .../java/org/apache/camel/component/aws/sdb/AmazonSDBClientMock.java | 1 - .../java/org/apache/camel/component/aws/ses/AmazonSESClientMock.java | 1 - .../java/org/apache/camel/component/aws/sns/AmazonSNSClientMock.java | 1 - .../java/org/apache/camel/component/aws/sqs/AmazonSQSClientMock.java | 1 - .../java/org/apache/camel/component/aws/sqs/AmazonSQSClientMock.java | 1 - .../src/main/java/org/apache/camel/component/bonita/api/BonitaAPI.java | 1 - .../java/org/apache/camel/component/bonita/api/model/FileInput.java| 1 - .../camel/component/bonita/api/model/ProcessDefinitionResponse.java| 1 - .../org/apache/camel/component/bonita/api/util/BonitaAPIConfig.java| 1 - .../test/java/org/apache/camel/processor/aggregate/util/HeaderDto.java | 1 - .../main/java/org/apache/camel/component/consul/ConsulRegistry.java| 1 - .../component/crypto/cms/common/CryptoCmsMarshallerConfiguration.java | 1 - .../crypto/cms/common/DefaultCryptoCmsUnMarshallerConfiguration.java | 1 - .../crypto/cms/crypt/DefaultEnvelopedDataDecryptorConfiguration.java | 1 - .../java/org/apache/camel/component/debezium/DebeziumEndpoint.java | 1 - .../org/apache/camel/component/debezium/DebeziumTestComponent.java | 1 - .../java/org/apache/camel/component/debezium/DebeziumTestEndpoint.java | 1 - .../org/apache/camel/component/debezium/DebeziumMySqlComponent.java| 1 - .../org/apache/camel/component/debezium/DebeziumMySqlEndpoint.java | 1 - .../org/apache/camel/component/debezium/DebeziumPostgresComponent.java | 1 - .../org/apache/camel/component/debezium/DebeziumPostgresEndpoint.java | 1 - .../camel/component/google/bigquery/GoogleBigQueryConstants.java | 1 - .../component/google/bigquery/sql/GoogleBigQuerySQLComponent.java | 1 - .../component/google/calendar/internal/GoogleCalendarConstants.java| 1 - .../google/calendar/stream/GoogleCalendarStreamConstants.java | 1 - .../camel/component/google/drive/internal/GoogleDriveConstants.java| 1 - .../camel/component/google/mail/internal/GoogleMailConstants.java | 1 - .../camel/component/google/mail/stream/GoogleMailStreamConstants.java | 1 - .../camel/component/google/pubsub/consumer/PubsubAcknowledgement.java | 1 - .../camel/component/google/sheets/internal/GoogleSheetsConstants.java | 1 - .../component/google/sheets/stream/GoogleSheetsStreamConstants.java| 1 - .../component/google/sheets/server/GoogleSheetsApiTestServer.java | 2 -- .../apache/camel/component/hazelcast/HazelcastDefaultComponent.java| 1 - .../org/apache/camel/http/common/HttpRestHeaderFilterStrategy.java | 1 - .../src/main/java/org/apache/camel/component/http/HttpComponent.java | 2 -- .../src/main/java/org/apache/camel/component/ironmq/GsonUtil.java | 1 - .../camel/component/jbpm/workitem/InOnlyCamelWorkItemHandler.java | 1 - .../camel/component/jbpm/workitem/InOutCamelWorkItemHandler.java | 1 - .../src/main/java/org/apache/camel/component/jt400/Jt400Component.java | 1 - .../src/test/java/org/apache/camel/component/util/HeaderDto.java | 1 - .../main/java/org/apache/camel/component/lucene/LuceneComponent.java | 1 - .../org/apache/camel/component/lumberjack/LumberjackComponent.java | 1 - .../src/main/java/org/apache/camel/component/mail/MailComponent.java | 1 - .../micrometer/eventnotifier/AbstractMicrometerEventNotifier.java | 1 - .../event/notifier/AbstractMicroProfileMetricsEventNotifier.java | 1 -
[camel] 01/01: remove superfluous super() calls which compiler would do for free
This is an automated email from the ASF dual-hosted git repository. bvahdat pushed a commit to branch superfluous-super-call in repository https://gitbox.apache.org/repos/asf/camel.git commit 0b87036c3d9a0cb8ad40863efd2aa852cd0a8d6d Author: Babak Vahdat AuthorDate: Thu Oct 31 09:58:11 2019 +0100 remove superfluous super() calls which compiler would do for free --- .../java/org/apache/camel/component/activemq/ActiveMQComponent.java| 1 - .../org/apache/camel/component/as2/api/InvalidAS2NameException.java| 1 - .../as2/api/entity/AS2MessageDispositionNotificationEntity.java| 1 - .../java/org/apache/camel/dataformat/avro/SpecificDataNoCache.java | 1 - .../java/org/apache/camel/component/aws/ddb/AmazonDDBClientMock.java | 1 - .../java/org/apache/camel/component/aws/ecs/AmazonECSClientMock.java | 1 - .../java/org/apache/camel/component/aws/eks/AmazonEKSClientMock.java | 1 - .../camel/component/aws/kinesis/ReachedClosedStatusException.java | 1 - .../java/org/apache/camel/component/aws/kms/AmazonKMSClientMock.java | 1 - .../org/apache/camel/component/aws/lambda/AmazonLambdaClientMock.java | 1 - .../java/org/apache/camel/component/aws/mq/AmazonMQClientMock.java | 1 - .../java/org/apache/camel/component/aws/msk/AmazonMSKClientMock.java | 1 - .../java/org/apache/camel/component/aws/s3/AmazonS3ClientMock.java | 1 - .../apache/camel/component/aws/s3/AmazonS3EncryptionClientMock.java| 1 - .../java/org/apache/camel/component/aws/sdb/AmazonSDBClientMock.java | 1 - .../java/org/apache/camel/component/aws/ses/AmazonSESClientMock.java | 1 - .../java/org/apache/camel/component/aws/sns/AmazonSNSClientMock.java | 1 - .../java/org/apache/camel/component/aws/sqs/AmazonSQSClientMock.java | 1 - .../java/org/apache/camel/component/aws/sqs/AmazonSQSClientMock.java | 1 - .../src/main/java/org/apache/camel/component/bonita/api/BonitaAPI.java | 1 - .../java/org/apache/camel/component/bonita/api/model/FileInput.java| 1 - .../camel/component/bonita/api/model/ProcessDefinitionResponse.java| 1 - .../org/apache/camel/component/bonita/api/util/BonitaAPIConfig.java| 1 - .../test/java/org/apache/camel/processor/aggregate/util/HeaderDto.java | 1 - .../main/java/org/apache/camel/component/consul/ConsulRegistry.java| 1 - .../component/crypto/cms/common/CryptoCmsMarshallerConfiguration.java | 1 - .../crypto/cms/common/DefaultCryptoCmsUnMarshallerConfiguration.java | 1 - .../crypto/cms/crypt/DefaultEnvelopedDataDecryptorConfiguration.java | 1 - .../java/org/apache/camel/component/debezium/DebeziumEndpoint.java | 1 - .../org/apache/camel/component/debezium/DebeziumTestComponent.java | 1 - .../java/org/apache/camel/component/debezium/DebeziumTestEndpoint.java | 1 - .../org/apache/camel/component/debezium/DebeziumMySqlComponent.java| 1 - .../org/apache/camel/component/debezium/DebeziumMySqlEndpoint.java | 1 - .../org/apache/camel/component/debezium/DebeziumPostgresComponent.java | 1 - .../org/apache/camel/component/debezium/DebeziumPostgresEndpoint.java | 1 - .../camel/component/google/bigquery/GoogleBigQueryConstants.java | 1 - .../component/google/bigquery/sql/GoogleBigQuerySQLComponent.java | 1 - .../component/google/calendar/internal/GoogleCalendarConstants.java| 1 - .../google/calendar/stream/GoogleCalendarStreamConstants.java | 1 - .../camel/component/google/drive/internal/GoogleDriveConstants.java| 1 - .../camel/component/google/mail/internal/GoogleMailConstants.java | 1 - .../camel/component/google/mail/stream/GoogleMailStreamConstants.java | 1 - .../camel/component/google/pubsub/consumer/PubsubAcknowledgement.java | 1 - .../camel/component/google/sheets/internal/GoogleSheetsConstants.java | 1 - .../component/google/sheets/stream/GoogleSheetsStreamConstants.java| 1 - .../component/google/sheets/server/GoogleSheetsApiTestServer.java | 2 -- .../apache/camel/component/hazelcast/HazelcastDefaultComponent.java| 1 - .../org/apache/camel/http/common/HttpRestHeaderFilterStrategy.java | 1 - .../src/main/java/org/apache/camel/component/http/HttpComponent.java | 2 -- .../src/main/java/org/apache/camel/component/ironmq/GsonUtil.java | 1 - .../camel/component/jbpm/workitem/InOnlyCamelWorkItemHandler.java | 1 - .../camel/component/jbpm/workitem/InOutCamelWorkItemHandler.java | 1 - .../src/main/java/org/apache/camel/component/jt400/Jt400Component.java | 1 - .../src/test/java/org/apache/camel/component/util/HeaderDto.java | 1 - .../main/java/org/apache/camel/component/lucene/LuceneComponent.java | 1 - .../org/apache/camel/component/lumberjack/LumberjackComponent.java | 1 - .../src/main/java/org/apache/camel/component/mail/MailComponent.java | 1 - .../micrometer/eventnotifier/AbstractMicrometerEventNotifier.java | 1 - .../event/notifier/AbstractMicroProfileMetricsEventNotifier.java | 1 -
[camel] branch superfluous-super-call created (now 0b87036)
This is an automated email from the ASF dual-hosted git repository. bvahdat pushed a change to branch superfluous-super-call in repository https://gitbox.apache.org/repos/asf/camel.git. at 0b87036 remove superfluous super() calls which compiler would do for free This branch includes the following new commits: new 0b87036 remove superfluous super() calls which compiler would do for free 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.
[camel] branch master updated (80716bf -> 9cbe43e)
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 80716bf Regen add 9cbe43e Polished No new revisions were added by this update. Summary of changes: .../java/org/apache/camel/component/validator/ValidatorEndpoint.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
[GitHub] [camel-quarkus] ppalaga opened a new issue #356: Issues in the List of extensions
ppalaga opened a new issue #356: Issues in the List of extensions URL: https://github.com/apache/camel-quarkus/issues/356 * Some entries have bad Since version. The plugin should not assume 0.2 as a default. I think it should rather fail if `` is not set for the given extension. * The description of camel-quarkus-attachments and camel-quarkus-core-xml `Maven plugins configuration` is apparently inherited from the build parent. The plugin should not use the inherited value and I think it should rather fail if the description is not there in the runtime module. * Not sure how the extensions are are assigned to the Misc. category, but at least opentracing, MP health and Platform HTTP seem to rather belong to Camel components category. Fixing this should be coordinated with whoever works on the potentially conflicting #336 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
