opendev/gerrit
Code Issues Proposed changes

Merge "Assign and ignore unused future in project/Index.java"

Browse Source
This commit is contained in:
David Pursehouse
2017-09-04 07:36:04 +00:00
committed by Gerrit Code Review
parent 3647c00835 b3852b954c
commit e8819efe02
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-server/src/main/java/com/google/gerrit/server/project/Index.java
View File

@@ -32,6 +32,7 @@ import com.google.gerrit.server.index.change.ChangeIndexer;
import com.google.inject.Inject;
import com.google.inject.Provider;
import com.google.inject.Singleton;
import java.util.concurrent.Future;
import org.eclipse.jgit.util.io.NullOutputStream;
@RequiresCapability(GlobalCapability.ADMINISTRATE_SERVER)
@@ -60,7 +61,11 @@ public class Index implements RestModifyView<ProjectResource, ProjectInput> {
.beginSubTask("", MultiProgressMonitor.UNKNOWN);
AllChangesIndexer allChangesIndexer = allChangesIndexerProvider.get();
allChangesIndexer.setVerboseOut(NullOutputStream.INSTANCE);
executor.submit(allChangesIndexer.reindexProject(indexer, project, mpt, mpt));
// The REST call is just a trigger for async reindexing, so it is safe to ignore the future's
// return value.
@SuppressWarnings("unused")
Future<Void> ignored =
executor.submit(allChangesIndexer.reindexProject(indexer, project, mpt, mpt));
return Response.accepted("Project " + project + " submitted for reindexing");
}
}