ZEPPELIN-501: validate notes before update index

bzz · bzz · commit da69c0773728 · 2015-12-22T19:23:17.000+09:00
diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/search/SearchService.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/search/SearchService.java
@@ -180,10 +180,21 @@ public void updateIndexDoc(Note note) throws IOException {
   }
 
   private void updateIndexNoteName(Note note) throws IOException {
-    updateDoc(note.getId(), note.getName(), null);
+    String noteName = note.getName();
+    String noteId = note.getId();
+    LOG.debug("Indexing Notebook {}, '{}'", noteId, noteName);
+    if (null == noteName || noteName.isEmpty()) {
+      LOG.debug("Skipping empty notebook name");
+      return;
+    }
+    updateDoc(noteId, noteName, null);
   }
 
   private void updateIndexParagraph(Note note, Paragraph p) throws IOException {
+    if (p.getText() == null) {
+      LOG.debug("Skipping empty paragraph");
+      return;
+    }
     updateDoc(note.getId(), note.getName(), p);
   }
 
