 20d981e1d9
			
		
	
	20d981e1d9
	
	
	
		
			
			We recently added a project index to Gerrit. This complements the existing set of indices (change, account, group). For this new index we want to offer a REST endpoint as well to reindex a project with the option to recusively reindex the children as well. Unfortunately, we previously bound /index to the endpoint that reindexes all changes in the project. This commit changes that binding to /index.changes. We make an effort to keep the REST API stable, but this is just an administrative endpoint and it seems to be a better fit to just repurpose the endpoint name. This commit adds tests and methods for the Java extension API. Change-Id: I2b3bb9f38c94938b98059da193e80289996a4a41
		
			
				
	
	
		
			73 lines
		
	
	
		
			3.1 KiB
		
	
	
	
		
			Java
		
	
	
	
	
	
			
		
		
	
	
			73 lines
		
	
	
		
			3.1 KiB
		
	
	
	
		
			Java
		
	
	
	
	
	
| // Copyright (C) 2017 The Android Open Source Project
 | |
| //
 | |
| // Licensed under the Apache License, Version 2.0 (the "License");
 | |
| // you may not use this file except in compliance with the License.
 | |
| // You may obtain a copy of the License at
 | |
| //
 | |
| // http://www.apache.org/licenses/LICENSE-2.0
 | |
| //
 | |
| // Unless required by applicable law or agreed to in writing, software
 | |
| // distributed under the License is distributed on an "AS IS" BASIS,
 | |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 | |
| // See the License for the specific language governing permissions and
 | |
| // limitations under the License.
 | |
| 
 | |
| package com.google.gerrit.server.restapi.project;
 | |
| 
 | |
| import static com.google.gerrit.server.git.QueueProvider.QueueType.BATCH;
 | |
| 
 | |
| import com.google.common.io.ByteStreams;
 | |
| import com.google.common.util.concurrent.ListeningExecutorService;
 | |
| import com.google.gerrit.common.data.GlobalCapability;
 | |
| import com.google.gerrit.extensions.annotations.RequiresCapability;
 | |
| import com.google.gerrit.extensions.api.projects.ProjectInput;
 | |
| import com.google.gerrit.extensions.restapi.Response;
 | |
| import com.google.gerrit.extensions.restapi.RestModifyView;
 | |
| import com.google.gerrit.reviewdb.client.Project;
 | |
| import com.google.gerrit.server.git.MultiProgressMonitor;
 | |
| import com.google.gerrit.server.git.MultiProgressMonitor.Task;
 | |
| import com.google.gerrit.server.index.IndexExecutor;
 | |
| import com.google.gerrit.server.index.change.AllChangesIndexer;
 | |
| import com.google.gerrit.server.index.change.ChangeIndexer;
 | |
| import com.google.gerrit.server.project.ProjectResource;
 | |
| 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)
 | |
| @Singleton
 | |
| public class IndexChanges implements RestModifyView<ProjectResource, ProjectInput> {
 | |
| 
 | |
|   private final Provider<AllChangesIndexer> allChangesIndexerProvider;
 | |
|   private final ChangeIndexer indexer;
 | |
|   private final ListeningExecutorService executor;
 | |
| 
 | |
|   @Inject
 | |
|   IndexChanges(
 | |
|       Provider<AllChangesIndexer> allChangesIndexerProvider,
 | |
|       ChangeIndexer indexer,
 | |
|       @IndexExecutor(BATCH) ListeningExecutorService executor) {
 | |
|     this.allChangesIndexerProvider = allChangesIndexerProvider;
 | |
|     this.indexer = indexer;
 | |
|     this.executor = executor;
 | |
|   }
 | |
| 
 | |
|   @Override
 | |
|   public Response.Accepted apply(ProjectResource resource, ProjectInput input) {
 | |
|     Project.NameKey project = resource.getNameKey();
 | |
|     Task mpt =
 | |
|         new MultiProgressMonitor(ByteStreams.nullOutputStream(), "Reindexing project")
 | |
|             .beginSubTask("", MultiProgressMonitor.UNKNOWN);
 | |
|     AllChangesIndexer allChangesIndexer = allChangesIndexerProvider.get();
 | |
|     allChangesIndexer.setVerboseOut(NullOutputStream.INSTANCE);
 | |
|     // The REST call is just a trigger for async reindexing, so it is safe to ignore the future's
 | |
|     // return value.
 | |
|     @SuppressWarnings("unused")
 | |
|     Future<?> possiblyIgnoredError =
 | |
|         executor.submit(allChangesIndexer.reindexProject(indexer, project, mpt, mpt));
 | |
|     return Response.accepted("Project " + project + " submitted for reindexing");
 | |
|   }
 | |
| }
 |