This is an automated email from the ASF dual-hosted git repository.
ilgrosso pushed a commit to branch 2_1_X
in repository https://gitbox.apache.org/repos/asf/syncope.git
The following commit(s) were added to refs/heads/2_1_X by this push:
new 58b1bbb027 Upgrading Elasticsearch (keeping RestHighLevelClient)
58b1bbb027 is described below
commit 58b1bbb0270ef9bc8ebe8c3f2cb9cf8914cf1810
Author: Francesco Chicchiriccò <[email protected]>
AuthorDate: Sat May 7 12:26:49 2022 +0200
Upgrading Elasticsearch (keeping RestHighLevelClient)
---
ext/elasticsearch/client-elasticsearch/pom.xml | 17 --------------
.../client/ElasticsearchClientFactoryBean.java | 16 +++++++------
.../client/ElasticsearchIndexManager.java | 10 ++++-----
.../elasticsearch/client/ElasticsearchUtils.java | 5 +++--
.../jpa/dao/ElasticsearchAnySearchDAO.java | 4 ++--
.../java/job/ElasticsearchReindex.java | 6 ++---
pom.xml | 26 +---------------------
7 files changed, 23 insertions(+), 61 deletions(-)
diff --git a/ext/elasticsearch/client-elasticsearch/pom.xml
b/ext/elasticsearch/client-elasticsearch/pom.xml
index 22d65201aa..d1945a98a7 100644
--- a/ext/elasticsearch/client-elasticsearch/pom.xml
+++ b/ext/elasticsearch/client-elasticsearch/pom.xml
@@ -48,23 +48,6 @@ under the License.
<groupId>org.elasticsearch.client</groupId>
<artifactId>elasticsearch-rest-high-level-client</artifactId>
</dependency>
- <dependency>
- <groupId>org.elasticsearch.client</groupId>
- <artifactId>elasticsearch-rest-client</artifactId>
- </dependency>
- <dependency>
- <groupId>org.elasticsearch</groupId>
- <artifactId>elasticsearch</artifactId>
- </dependency>
-
- <dependency>
- <groupId>com.fasterxml.jackson.dataformat</groupId>
- <artifactId>jackson-dataformat-smile</artifactId>
- </dependency>
- <dependency>
- <groupId>com.fasterxml.jackson.dataformat</groupId>
- <artifactId>jackson-dataformat-cbor</artifactId>
- </dependency>
</dependencies>
<build>
diff --git
a/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchClientFactoryBean.java
b/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchClientFactoryBean.java
index 39246a931b..df68c6ea85 100644
---
a/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchClientFactoryBean.java
+++
b/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchClientFactoryBean.java
@@ -31,14 +31,16 @@ import org.apache.http.impl.client.BasicCredentialsProvider;
import org.apache.http.message.BasicHeader;
import org.elasticsearch.client.RestClient;
import org.elasticsearch.client.RestClientBuilder;
-import org.elasticsearch.client.RestHighLevelClient;
import org.springframework.beans.factory.DisposableBean;
import org.springframework.beans.factory.FactoryBean;
/**
- * Spring {@link FactoryBean} for getting the Elasticsearch's {@link
RestHighLevelClient} singleton instance.
+ * Spring {@link FactoryBean} for getting the Elasticsearch's
+ * {@link org.elasticsearch.client.RestHighLevelClient} singleton instance.
*/
-public class ElasticsearchClientFactoryBean implements
FactoryBean<RestHighLevelClient>, DisposableBean {
+@SuppressWarnings("deprecation")
+public class ElasticsearchClientFactoryBean
+ implements FactoryBean<org.elasticsearch.client.RestHighLevelClient>,
DisposableBean {
private final List<HttpHost> hosts;
@@ -52,7 +54,7 @@ public class ElasticsearchClientFactoryBean implements
FactoryBean<RestHighLevel
private String apiKeySecret;
- private RestHighLevelClient client;
+ private org.elasticsearch.client.RestHighLevelClient client;
public ElasticsearchClientFactoryBean(final List<HttpHost> hosts) {
this.hosts = hosts;
@@ -91,7 +93,7 @@ public class ElasticsearchClientFactoryBean implements
FactoryBean<RestHighLevel
}
@Override
- public RestHighLevelClient getObject() throws Exception {
+ public org.elasticsearch.client.RestHighLevelClient getObject() throws
Exception {
synchronized (this) {
if (client == null) {
RestClientBuilder restClient =
RestClient.builder(hosts.toArray(new HttpHost[0]));
@@ -109,7 +111,7 @@ public class ElasticsearchClientFactoryBean implements
FactoryBean<RestHighLevel
restClient.setDefaultHeaders(
new Header[] { new
BasicHeader(HttpHeaders.AUTHORIZATION, "ApiKey " + apiKeyAuth) });
}
- client = new RestHighLevelClient(restClient);
+ client = new
org.elasticsearch.client.RestHighLevelClient(restClient);
}
}
return client;
@@ -117,7 +119,7 @@ public class ElasticsearchClientFactoryBean implements
FactoryBean<RestHighLevel
@Override
public Class<?> getObjectType() {
- return RestHighLevelClient.class;
+ return org.elasticsearch.client.RestHighLevelClient.class;
}
@Override
diff --git
a/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchIndexManager.java
b/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchIndexManager.java
index b313dca022..284831ff1e 100644
---
a/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchIndexManager.java
+++
b/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchIndexManager.java
@@ -36,12 +36,11 @@ import
org.elasticsearch.action.support.master.AcknowledgedResponse;
import org.elasticsearch.action.update.UpdateRequest;
import org.elasticsearch.action.update.UpdateResponse;
import org.elasticsearch.client.RequestOptions;
-import org.elasticsearch.client.RestHighLevelClient;
import org.elasticsearch.client.indices.CreateIndexRequest;
import org.elasticsearch.client.indices.CreateIndexResponse;
import org.elasticsearch.client.indices.GetIndexRequest;
-import org.elasticsearch.common.xcontent.XContentBuilder;
-import org.elasticsearch.common.xcontent.XContentFactory;
+import org.elasticsearch.xcontent.XContentBuilder;
+import org.elasticsearch.xcontent.XContentFactory;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.transaction.event.TransactionalEventListener;
@@ -49,16 +48,17 @@ import
org.springframework.transaction.event.TransactionalEventListener;
/**
* Listen to any create / update and delete in order to keep the Elasticsearch
indexes consistent.
*/
+@SuppressWarnings("deprecation")
public class ElasticsearchIndexManager {
private static final Logger LOG =
LoggerFactory.getLogger(ElasticsearchIndexManager.class);
- protected final RestHighLevelClient client;
+ protected final org.elasticsearch.client.RestHighLevelClient client;
protected final ElasticsearchUtils elasticsearchUtils;
public ElasticsearchIndexManager(
- final RestHighLevelClient client,
+ final org.elasticsearch.client.RestHighLevelClient client,
final ElasticsearchUtils elasticsearchUtils) {
this.client = client;
diff --git
a/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchUtils.java
b/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchUtils.java
index a411ec1990..3a030cc096 100644
---
a/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchUtils.java
+++
b/ext/elasticsearch/client-elasticsearch/src/main/java/org/apache/syncope/ext/elasticsearch/client/ElasticsearchUtils.java
@@ -36,14 +36,15 @@ import
org.apache.syncope.core.persistence.api.entity.anyobject.AnyObject;
import org.apache.syncope.core.persistence.api.entity.group.Group;
import org.apache.syncope.core.persistence.api.entity.user.User;
import org.apache.syncope.core.spring.security.AuthContextUtils;
-import org.elasticsearch.common.xcontent.XContentBuilder;
-import org.elasticsearch.common.xcontent.XContentFactory;
+import org.elasticsearch.xcontent.XContentBuilder;
+import org.elasticsearch.xcontent.XContentFactory;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.transaction.annotation.Transactional;
/**
* Utility methods for usage with Elasticsearch.
*/
+@SuppressWarnings("deprecation")
public class ElasticsearchUtils {
public static String getContextDomainName(final String domain, final
AnyTypeKind kind) {
diff --git
a/ext/elasticsearch/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/ElasticsearchAnySearchDAO.java
b/ext/elasticsearch/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/ElasticsearchAnySearchDAO.java
index c11bb4d2bd..6843a02bd8 100644
---
a/ext/elasticsearch/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/ElasticsearchAnySearchDAO.java
+++
b/ext/elasticsearch/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/ElasticsearchAnySearchDAO.java
@@ -63,7 +63,6 @@ import
org.apache.syncope.ext.elasticsearch.client.ElasticsearchUtils;
import org.elasticsearch.action.search.SearchRequest;
import org.elasticsearch.action.search.SearchType;
import org.elasticsearch.client.RequestOptions;
-import org.elasticsearch.client.RestHighLevelClient;
import org.elasticsearch.client.core.CountRequest;
import org.elasticsearch.index.query.DisMaxQueryBuilder;
import org.elasticsearch.index.query.QueryBuilder;
@@ -78,6 +77,7 @@ import org.springframework.beans.factory.annotation.Autowired;
/**
* Search engine implementation for users, groups and any objects, based on
Elasticsearch.
*/
+@SuppressWarnings("deprecation")
public class ElasticsearchAnySearchDAO extends AbstractAnySearchDAO {
protected static final char[] ELASTICSEARCH_REGEX_CHARS = new char[] {
@@ -96,7 +96,7 @@ public class ElasticsearchAnySearchDAO extends
AbstractAnySearchDAO {
}
@Autowired
- protected RestHighLevelClient client;
+ protected org.elasticsearch.client.RestHighLevelClient client;
@Autowired
protected ElasticsearchUtils elasticsearchUtils;
diff --git
a/ext/elasticsearch/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/job/ElasticsearchReindex.java
b/ext/elasticsearch/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/job/ElasticsearchReindex.java
index edc9dbc144..c2e691e101 100644
---
a/ext/elasticsearch/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/job/ElasticsearchReindex.java
+++
b/ext/elasticsearch/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/job/ElasticsearchReindex.java
@@ -31,8 +31,7 @@ import
org.apache.syncope.ext.elasticsearch.client.ElasticsearchUtils;
import org.elasticsearch.action.index.IndexRequest;
import org.elasticsearch.action.index.IndexResponse;
import org.elasticsearch.client.RequestOptions;
-import org.elasticsearch.client.RestHighLevelClient;
-import org.elasticsearch.common.xcontent.XContentBuilder;
+import org.elasticsearch.xcontent.XContentBuilder;
import org.quartz.JobExecutionContext;
import org.quartz.JobExecutionException;
import org.springframework.beans.factory.annotation.Autowired;
@@ -40,10 +39,11 @@ import
org.springframework.beans.factory.annotation.Autowired;
/**
* Remove and rebuild all Elasticsearch indexes with information from existing
users, groups and any objects.
*/
+@SuppressWarnings("deprecation")
public class ElasticsearchReindex extends AbstractSchedTaskJobDelegate {
@Autowired
- protected RestHighLevelClient client;
+ protected org.elasticsearch.client.RestHighLevelClient client;
@Autowired
protected ElasticsearchIndexManager indexManager;
diff --git a/pom.xml b/pom.xml
index 708f820fa9..4f7a86ed68 100644
--- a/pom.xml
+++ b/pom.xml
@@ -433,7 +433,7 @@ under the License.
<slf4j.version>1.7.36</slf4j.version>
<opensaml.version>3.3.1</opensaml.version>
- <elasticsearch.version>7.15.2</elasticsearch.version>
+ <elasticsearch.version>7.16.3</elasticsearch.version>
<apacheds.version>2.0.0.AM26</apacheds.version>
<apachedirapi.version>2.0.0</apachedirapi.version>
@@ -805,30 +805,6 @@ under the License.
<groupId>org.elasticsearch.client</groupId>
<artifactId>elasticsearch-rest-high-level-client</artifactId>
<version>${elasticsearch.version}</version>
- <exclusions>
- <exclusion>
- <groupId>com.fasterxml.jackson.datatype</groupId>
- <artifactId>jackson-dataformat-smile</artifactId>
- </exclusion>
- <exclusion>
- <groupId>com.fasterxml.jackson.datatype</groupId>
- <artifactId>jackson-dataformat-yaml</artifactId>
- </exclusion>
- <exclusion>
- <groupId>com.fasterxml.jackson.datatype</groupId>
- <artifactId>jackson-dataformat-cbor</artifactId>
- </exclusion>
- </exclusions>
- </dependency>
- <dependency>
- <groupId>org.elasticsearch.client</groupId>
- <artifactId>elasticsearch-rest-client</artifactId>
- <version>${elasticsearch.version}</version>
- </dependency>
- <dependency>
- <groupId>org.elasticsearch</groupId>
- <artifactId>elasticsearch</artifactId>
- <version>${elasticsearch.version}</version>
</dependency>
<dependency>