]> source.dussan.org Git - sonarqube.git/commitdiff
Remove useless usage of ES UpsertRequest
authorSimon Brandhof <simon.brandhof@sonarsource.com>
Tue, 23 Feb 2016 17:57:41 +0000 (18:57 +0100)
committerSimon Brandhof <simon.brandhof@sonarsource.com>
Fri, 26 Feb 2016 13:29:39 +0000 (14:29 +0100)
server/sonar-server/src/main/java/org/sonar/server/user/index/UserIndexer.java
server/sonar-server/src/main/java/org/sonar/server/view/index/ViewIndexer.java

index 8e606de2c5c56b5a124dd9d21c783ab10c49a098..6cb8f5fe74c1ef72635e3f5656567a60120886ed 100644 (file)
@@ -20,6 +20,7 @@
 package org.sonar.server.user.index;
 
 import java.util.Iterator;
+import org.elasticsearch.action.index.IndexRequest;
 import org.elasticsearch.action.update.UpdateRequest;
 import org.sonar.db.DbClient;
 import org.sonar.db.DbSession;
@@ -57,17 +58,16 @@ public class UserIndexer extends BaseIndexer {
     bulk.start();
     while (users.hasNext()) {
       UserDoc user = users.next();
-      bulk.add(newUpsertRequest(user));
+      bulk.add(newIndexRequest(user));
       maxUpdatedAt = Math.max(maxUpdatedAt, user.updatedAt());
     }
     bulk.stop();
     return maxUpdatedAt;
   }
 
-  private static UpdateRequest newUpsertRequest(UserDoc user) {
-    return new UpdateRequest(UserIndexDefinition.INDEX, UserIndexDefinition.TYPE_USER, user.login())
-      .doc(user.getFields())
-      .upsert(user.getFields());
+  private static IndexRequest newIndexRequest(UserDoc user) {
+    return new IndexRequest(UserIndexDefinition.INDEX, UserIndexDefinition.TYPE_USER, user.login())
+      .source(user.getFields());
   }
 
 }
index 5c721ce77d703bdb19c8f73ca5ae3d82fa3f9ea5..296a9cccaf88f0377693d0c1630488f5d600c927 100644 (file)
@@ -21,7 +21,7 @@ package org.sonar.server.view.index;
 
 import java.util.List;
 import java.util.Map;
-import org.elasticsearch.action.update.UpdateRequest;
+import org.elasticsearch.action.index.IndexRequest;
 import org.sonar.db.DbClient;
 import org.sonar.db.DbSession;
 import org.sonar.db.component.ComponentDto;
@@ -113,16 +113,15 @@ public class ViewIndexer extends BaseIndexer {
   }
 
   private void doIndex(BulkIndexer bulk, ViewDoc viewDoc, boolean needClearCache) {
-    bulk.add(newUpsertRequest(viewDoc));
+    bulk.add(newIndexRequest(viewDoc));
     if (needClearCache) {
       clearLookupCache(viewDoc.uuid());
     }
   }
 
-  private static UpdateRequest newUpsertRequest(ViewDoc doc) {
-    return new UpdateRequest(ViewIndexDefinition.INDEX, ViewIndexDefinition.TYPE_VIEW, doc.uuid())
-      .doc(doc.getFields())
-      .upsert(doc.getFields());
+  private static IndexRequest newIndexRequest(ViewDoc doc) {
+    return new IndexRequest(ViewIndexDefinition.INDEX, ViewIndexDefinition.TYPE_VIEW, doc.uuid())
+      .source(doc.getFields());
   }
 
   private void clearLookupCache(String viewUuid) {