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 bt...@apache.org on 2019/05/30 07:22:24 UTC
[james-project] 13/19: MAILBOX-351 ReIndexer should allow me to
reIndex previous tasks failures
This is an automated email from the ASF dual-hosted git repository.
btellier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git
commit 7643548684b672b5c58ac3629277b56eeedfe832
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Tue May 28 10:43:30 2019 +0700
MAILBOX-351 ReIndexer should allow me to reIndex previous tasks failures
---
...iluresReIndexationTask.java => ErrorRecoveryIndexationTask.java} | 6 +++---
.../main/java/org/apache/mailbox/tools/indexer/ReIndexerImpl.java | 2 +-
.../java/org/apache/james/webadmin/routes/ReindexingRoutesTest.java | 6 +++---
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/PreviousFailuresReIndexationTask.java b/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ErrorRecoveryIndexationTask.java
similarity index 89%
rename from mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/PreviousFailuresReIndexationTask.java
rename to mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ErrorRecoveryIndexationTask.java
index faf513c..cc6fb16 100644
--- a/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/PreviousFailuresReIndexationTask.java
+++ b/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ErrorRecoveryIndexationTask.java
@@ -25,15 +25,15 @@ import org.apache.james.mailbox.indexer.ReIndexingExecutionFailures;
import org.apache.james.task.Task;
import org.apache.james.task.TaskExecutionDetails;
-public class PreviousFailuresReIndexationTask implements Task {
- private static final String PREVIOUS_FAILURES_INDEXING = "ReIndexPreviousFailures";
+public class ErrorRecoveryIndexationTask implements Task {
+ private static final String PREVIOUS_FAILURES_INDEXING = "ErrorRecoveryIndexation";
private final ReIndexerPerformer reIndexerPerformer;
private final ReprocessingContextInformation additionalInformation;
private final ReprocessingContext reprocessingContext;
private final ReIndexingExecutionFailures previousFailures;
- public PreviousFailuresReIndexationTask(ReIndexerPerformer reIndexerPerformer, ReIndexingExecutionFailures previousFailures) {
+ public ErrorRecoveryIndexationTask(ReIndexerPerformer reIndexerPerformer, ReIndexingExecutionFailures previousFailures) {
this.reIndexerPerformer = reIndexerPerformer;
this.previousFailures = previousFailures;
this.reprocessingContext = new ReprocessingContext();
diff --git a/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ReIndexerImpl.java b/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ReIndexerImpl.java
index 6b1cbd8..9244b91 100644
--- a/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ReIndexerImpl.java
+++ b/mailbox/tools/indexer/src/main/java/org/apache/mailbox/tools/indexer/ReIndexerImpl.java
@@ -99,7 +99,7 @@ public class ReIndexerImpl implements ReIndexer {
@Override
public Task reIndex(ReIndexingExecutionFailures previousFailures) {
- return new PreviousFailuresReIndexationTask(reIndexerPerformer, previousFailures);
+ return new ErrorRecoveryIndexationTask(reIndexerPerformer, previousFailures);
}
private void validateIdExists(MailboxId mailboxId) throws MailboxException {
diff --git a/server/protocols/webadmin/webadmin-mailbox/src/test/java/org/apache/james/webadmin/routes/ReindexingRoutesTest.java b/server/protocols/webadmin/webadmin-mailbox/src/test/java/org/apache/james/webadmin/routes/ReindexingRoutesTest.java
index fc3c701..2d9e100 100644
--- a/server/protocols/webadmin/webadmin-mailbox/src/test/java/org/apache/james/webadmin/routes/ReindexingRoutesTest.java
+++ b/server/protocols/webadmin/webadmin-mailbox/src/test/java/org/apache/james/webadmin/routes/ReindexingRoutesTest.java
@@ -1053,7 +1053,7 @@ class ReindexingRoutesTest {
.then()
.body("status", is("completed"))
.body("taskId", is(notNullValue()))
- .body("type", is("ReIndexPreviousFailures"))
+ .body("type", is("ErrorRecoveryIndexation"))
.body("additionalInformation.successfullyReprocessMailCount", is(0))
.body("additionalInformation.failedReprocessedMailCount", is(0))
.body("startedDate", is(notNullValue()))
@@ -1097,7 +1097,7 @@ class ReindexingRoutesTest {
.then()
.body("status", is("completed"))
.body("taskId", is(notNullValue()))
- .body("type", is("ReIndexPreviousFailures"))
+ .body("type", is("ErrorRecoveryIndexation"))
.body("additionalInformation.successfullyReprocessMailCount", is(1))
.body("additionalInformation.failedReprocessedMailCount", is(0))
.body("startedDate", is(notNullValue()))
@@ -1142,7 +1142,7 @@ class ReindexingRoutesTest {
.then()
.body("status", is("failed"))
.body("taskId", is(notNullValue()))
- .body("type", is("ReIndexPreviousFailures"))
+ .body("type", is("ErrorRecoveryIndexation"))
.body("additionalInformation.successfullyReprocessMailCount", is(0))
.body("additionalInformation.failedReprocessedMailCount", is(1))
.body("additionalInformation.failures.\"" + mailboxId.serialize() + "\"[0].uid", is(Long.valueOf(uidAsLong).intValue()))
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org