You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by rc...@apache.org on 2019/11/25 02:19:30 UTC

[james-project] 02/10: Fix logging for already created ElasticSearch index

This is an automated email from the ASF dual-hosted git repository.

rcordier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit 7840311c0361983a4ffdd180a09e1246b39e8c5f
Author: Raphael Ouazana <ra...@linagora.com>
AuthorDate: Thu Nov 21 16:46:58 2019 +0100

    Fix logging for already created ElasticSearch index
---
 .../java/org/apache/james/backends/es/IndexCreationFactory.java  | 2 +-
 .../src/main/java/org/apache/james/backends/es/IndexName.java    | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexCreationFactory.java b/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexCreationFactory.java
index 2910c91..9d6f834 100644
--- a/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexCreationFactory.java
+++ b/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexCreationFactory.java
@@ -120,7 +120,7 @@ public class IndexCreationFactory {
                             .source(settings), RequestOptions.DEFAULT);
             } catch (ElasticsearchStatusException exception) {
                 if (exception.getMessage().contains(INDEX_ALREADY_EXISTS_EXCEPTION_MESSAGE)) {
-                    LOGGER.info("Index [{}] already exist", indexName);
+                    LOGGER.info("Index [{}] already exists", indexName.getValue());
                 } else {
                     throw exception;
                 }
diff --git a/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexName.java b/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexName.java
index 4541f48..bfdddba 100644
--- a/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexName.java
+++ b/backends-common/elasticsearch/src/main/java/org/apache/james/backends/es/IndexName.java
@@ -21,6 +21,8 @@ package org.apache.james.backends.es;
 
 import java.util.Objects;
 
+import com.google.common.base.MoreObjects;
+
 public class IndexName {
     private final String value;
 
@@ -46,4 +48,11 @@ public class IndexName {
     public final int hashCode() {
         return Objects.hash(value);
     }
+
+    @Override
+    public String toString() {
+        return MoreObjects.toStringHelper(this)
+            .add("value", value)
+            .toString();
+    }
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org