camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Flemming Harms (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (CAMEL-11868) Migrate camel-elasticsearch5 java transport client to the new high level rest client
Date Mon, 02 Oct 2017 16:55:00 GMT

    [ https://issues.apache.org/jira/browse/CAMEL-11868?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16188355#comment-16188355
] 

Flemming Harms edited comment on CAMEL-11868 at 10/2/17 4:54 PM:
-----------------------------------------------------------------

It might be a good idea, since not all features is supported in the high level rest client.

I already got it to work with the rest client, but then I would suggest we create a super
class for ElasticsearchProducer and a ElasticsearchTransportProducer and ElasticsearchRestProducer.
In the ElasticsearchEndpoint we decide which one we should instantiate based on the type client
on the class path. This of course mean the client dependency is not included in the the camel-elasticsearch5
component.

Regarding the camel-elasticsearch, it's not possible to use the Hight level REST client since
it only support ELK from 5.6.0 and forward.

One think to be aware of with the REST API that it is slightly different, which mean various
request and response object is not called the same with the REST client. So either we have
to document this in the manual or we have to come up with a common request / response object
for both clients



was (Author: flemming.harms@gmail.com):
It might be a good idea, since not all features is supported in the high level rest client.

I already got it to work with the rest client, but then I would suggest we create a super
class for ElasticsearchProducer and a ElasticsearchTransportProducer and ElasticsearchRestProducer.
In the ElasticsearchEndpoint we decide which one we should instantiate based on the type client
on the class path. This of course mean the client dependency is not included in the the camel-elasticsearch5
component.

Regarding the camel-elasticsearch, it's not possible to use the Hight level REST client since
it only support ELK from 5.6.0 and forward.


> Migrate camel-elasticsearch5 java transport client to the new high level rest client
> ------------------------------------------------------------------------------------
>
>                 Key: CAMEL-11868
>                 URL: https://issues.apache.org/jira/browse/CAMEL-11868
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-elasticsearch5
>         Environment: any
>            Reporter: Flemming Harms
>            Priority: Critical
>
> The current java transport client is due EOL in near future and it will be a good idea
we switch the the new high level rest client instead. The new high level rest client is only
released from version 5.6.0 and towards so an general upgrade of the dependencies is required
for the ELK component.
> https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/java-rest-high.html
> This also add the support for basic authentication without the need of x-pack. This is
usefull when running ELK behind a reverse proxy. 
> https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/_basic_authentication.html



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message