Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-12 Thread via GitHub
zbendhiba merged PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738 -- 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. To unsubscribe, e-mail:

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-12 Thread via GitHub
zbendhiba commented on PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#issuecomment-1938878668 I pushed again because of format error and squashed commits -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-12 Thread via GitHub
zbendhiba commented on code in PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#discussion_r1486328420 ## integration-tests-jvm/elasticsearch-rest-client/src/main/java/org/apache/camel/quarkus/component/elasticsearch/rest/client/it/ElasticsearchRestClientResource.java:

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-12 Thread via GitHub
zbendhiba commented on PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#issuecomment-1938871924 > LGTM, the failure is probably related to re-formatting need only. > > @zbendhiba If one is currently implementing an example that transform data and load it into

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-09 Thread via GitHub
aldettinger commented on code in PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#discussion_r1484318580 ##

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-09 Thread via GitHub
zbendhiba commented on PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#issuecomment-1935494520 thanks @jamesnetherton ! -- 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

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-08 Thread via GitHub
jamesnetherton commented on PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#issuecomment-1935450271 > The test failure seems unrelated. Is master-openshift considered a flaky test ? It got enabled yesterday after being disabled for a long time. I will try to take a

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-08 Thread via GitHub
jamesnetherton commented on code in PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#discussion_r1483951773 ## integration-tests-jvm/elasticsearch-rest-client/src/test/java/org/apache/camel/quarkus/component/elasticsearch/rest/client/it/ElasticsearchTestResource.java:

Re: [PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-08 Thread via GitHub
zbendhiba commented on PR #5738: URL: https://github.com/apache/camel-quarkus/pull/5738#issuecomment-1935409231 The test failure seems unrelated. Is master-openshift considered a flaky test ? -- This is an automated message from the Apache Git Service. To respond to the message, please

[PR] Add Elasticsearch Low Level Rest Client [camel-quarkus]

2024-02-08 Thread via GitHub
zbendhiba opened a new pull request, #5738: URL: https://github.com/apache/camel-quarkus/pull/5738 Fixes #5736 I created a follow up issue to add more test coverage. https://github.com/apache/camel-quarkus/issues/5737 -- This is an automated message from the Apache Git Service.