SONAR-6089 Add java doc

diff --git a/server/sonar-server/src/main/java/org/sonar/server/view/index/ViewIndexer.java b/server/sonar-server/src/main/java/org/sonar/server/view/index/ViewIndexer.java
index 953aedf4dbc8bfa3f34be7be4878fbea81e78fb6..3f00775cbf3e15d096766ee1820b15230b6e2dde 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/view/index/ViewIndexer.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/view/index/ViewIndexer.java
@@ -47,8 +47,10 @@ public class ViewIndexer extends BaseIndexer {
   }
 
   /**
-   * Index all views if the index is empty.
-   * Only used on startup.
+   * Index all views if the index is empty (Only used on startup).
+   * It's currently not possible to index only data from db that are not existing in the index because we have no way to last when the structure of a view is changed :
+   * - Either the definition has changed -> No updated at column in the projects table,
+   * - Either the view is defined by a regex -> A new analysed project automatically steps into the view.
    *
    * The views lookup cache will not be cleared
    */

diff --git a/server/sonar-server/src/test/java/org/sonar/server/es/EsTester.java b/server/sonar-server/src/test/java/org/sonar/server/es/EsTester.java
index 186722ff74b8f32d2c2dcf1fe632c26879f04ad3..2c01714b687a09253c3512f36b0a7d4fd6f0b742 100644 (file)
--- a/server/sonar-server/src/test/java/org/sonar/server/es/EsTester.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/es/EsTester.java
@@ -51,6 +51,7 @@ import java.io.File;
 import java.io.FileInputStream;
 import java.util.Collections;
 import java.util.List;
+import java.util.Map;
 
 import static com.google.common.collect.Lists.newArrayList;
 import static org.assertj.core.api.Assertions.assertThat;
@@ -165,6 +166,14 @@ public class EsTester extends ExternalResource {
     bulk.get();
   }
 
+  public void putDocuments(String index, String type, Map<String, Object>... docs) throws Exception {
+    BulkRequestBuilder bulk = client.prepareBulk().setRefresh(true);
+    for (Map<String, Object> doc : docs) {
+      bulk.add(new IndexRequest(index, type).source(doc));
+    }
+    bulk.get();
+  }
+
   public long countDocuments(String indexName, String typeName) {
     return client().prepareCount(indexName).setTypes(typeName).get().getCount();
   }

diff --git a/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerMediumTest.java b/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerMediumTest.java
index 3be8d4e82a351e692692bb857790184f4359c491..ff806db5e704cc8e43450b933c7a799d538eae68 100644 (file)
--- a/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerMediumTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerMediumTest.java
@@ -53,7 +53,8 @@ import static org.assertj.core.api.Assertions.assertThat;
 /**
  * It's not possible to only used EsTester as IssueIndex does not support it yet.
  *
- * This class
+ * Only clear of the views lookup cache is tested here.
+ * See {@link ViewIndexerTest} for tests on common use cases.
  */
 public class ViewIndexerMediumTest {