[jira] [Commented] (CALCITE-2376) Unify ES2 and ES2 adapters. Migrate to low-level rest client

2018-07-11 Thread Andrei Sereda (JIRA)


[ 
https://issues.apache.org/jira/browse/CALCITE-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16540917#comment-16540917
 ] 

Andrei Sereda commented on CALCITE-2376:


[~julianhyde] [PR 758|https://github.com/apache/calcite/pull/758] 

> Unify ES2 and ES2 adapters. Migrate to low-level rest client
> 
>
> Key: CALCITE-2376
> URL: https://issues.apache.org/jira/browse/CALCITE-2376
> Project: Calcite
>  Issue Type: Improvement
>  Components: elasticsearch-adapter
>Reporter: Andrei Sereda
>Assignee: Michael Mior
>Priority: Major
> Fix For: 1.17.0
>
>
> This is an effort to migrate Elastic Search adapters to use [low-level rest 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/java-rest-low.html].
>  Current [native transport 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-api/6.3/client.html]
>  has been deprecated in 7.0 (to be removed in 8.x).
> Another advantage of low-level client is that it is compatible with any ES 
> server version and has few (non-core) dependencies. As part of this 
> improvement, both ES2 and ES5 adapters will be unified and reside under new 
> maven module {{elasticsearch}} (contrary to separate {{elasticsearch2}} and 
> {{elasticsearch5}}).
> h3. Breaking changes
> Using new {{artifactId}} and {{SchemaFactory}} implementation will force 
> clients to change their configuration and dependencies. No other breaking 
> changes (except configuration) are expected. 
>   



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CALCITE-2376) Unify ES2 and ES2 adapters. Migrate to low-level rest client

2018-07-11 Thread Andrei Sereda (JIRA)


[ 
https://issues.apache.org/jira/browse/CALCITE-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16540902#comment-16540902
 ] 

Andrei Sereda commented on CALCITE-2376:


Will do

> Unify ES2 and ES2 adapters. Migrate to low-level rest client
> 
>
> Key: CALCITE-2376
> URL: https://issues.apache.org/jira/browse/CALCITE-2376
> Project: Calcite
>  Issue Type: Improvement
>  Components: elasticsearch-adapter
>Reporter: Andrei Sereda
>Assignee: Michael Mior
>Priority: Major
> Fix For: 1.17.0
>
>
> This is an effort to migrate Elastic Search adapters to use [low-level rest 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/java-rest-low.html].
>  Current [native transport 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-api/6.3/client.html]
>  has been deprecated in 7.0 (to be removed in 8.x).
> Another advantage of low-level client is that it is compatible with any ES 
> server version and has few (non-core) dependencies. As part of this 
> improvement, both ES2 and ES5 adapters will be unified and reside under new 
> maven module {{elasticsearch}} (contrary to separate {{elasticsearch2}} and 
> {{elasticsearch5}}).
> h3. Breaking changes
> Using new {{artifactId}} and {{SchemaFactory}} implementation will force 
> clients to change their configuration and dependencies. No other breaking 
> changes (except configuration) are expected. 
>   



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CALCITE-2376) Unify ES2 and ES2 adapters. Migrate to low-level rest client

2018-07-11 Thread Julian Hyde (JIRA)


[ 
https://issues.apache.org/jira/browse/CALCITE-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16540853#comment-16540853
 ] 

Julian Hyde commented on CALCITE-2376:
--

[~asereda] Can you make log4j2.xml consistent in format to other .xml files? In 
particular the header.

Also, when your javadoc has paragraphs you must use {{}}, otherwise html 
will flow them together.  This applies in ElasticsearchNode and 
ElasticsearchPolicy.

> Unify ES2 and ES2 adapters. Migrate to low-level rest client
> 
>
> Key: CALCITE-2376
> URL: https://issues.apache.org/jira/browse/CALCITE-2376
> Project: Calcite
>  Issue Type: Improvement
>  Components: elasticsearch-adapter
>Reporter: Andrei Sereda
>Assignee: Michael Mior
>Priority: Major
> Fix For: 1.17.0
>
>
> This is an effort to migrate Elastic Search adapters to use [low-level rest 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/java-rest-low.html].
>  Current [native transport 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-api/6.3/client.html]
>  has been deprecated in 7.0 (to be removed in 8.x).
> Another advantage of low-level client is that it is compatible with any ES 
> server version and has few (non-core) dependencies. As part of this 
> improvement, both ES2 and ES5 adapters will be unified and reside under new 
> maven module {{elasticsearch}} (contrary to separate {{elasticsearch2}} and 
> {{elasticsearch5}}).
> h3. Breaking changes
> Using new {{artifactId}} and {{SchemaFactory}} implementation will force 
> clients to change their configuration and dependencies. No other breaking 
> changes (except configuration) are expected. 
>   



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CALCITE-2376) Unify ES2 and ES2 adapters. Migrate to low-level rest client

2018-07-09 Thread Michael Mior (JIRA)


[ 
https://issues.apache.org/jira/browse/CALCITE-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16537173#comment-16537173
 ] 

Michael Mior commented on CALCITE-2376:
---

Fixed in 
[0204f2860|https://github.com/apache/calcite/commit/0204f2860c8c83915c62085a47d42a78a10aac46].

> Unify ES2 and ES2 adapters. Migrate to low-level rest client
> 
>
> Key: CALCITE-2376
> URL: https://issues.apache.org/jira/browse/CALCITE-2376
> Project: Calcite
>  Issue Type: Improvement
>  Components: elasticsearch-adapter
>Reporter: Andrei Sereda
>Assignee: Michael Mior
>Priority: Major
> Fix For: 1.17.0
>
>
> This is an effort to migrate Elastic Search adapters to use [low-level rest 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/java-rest-low.html].
>  Current [native transport 
> client|https://www.elastic.co/guide/en/elasticsearch/client/java-api/6.3/client.html]
>  has been deprecated in 7.0 (to be removed in 8.x).
> Another advantage of low-level client is that it is compatible with any ES 
> server version and has few (non-core) dependencies. As part of this 
> improvement, both ES2 and ES5 adapters will be unified and reside under new 
> maven module {{elasticsearch}} (contrary to separate {{elasticsearch2}} and 
> {{elasticsearch5}}).
> h3. Breaking changes
> Using new {{artifactId}} and {{SchemaFactory}} implementation will force 
> clients to change their configuration and dependencies. No other breaking 
> changes (except configuration) are expected. 
>   



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)