logging the error

ramaswamydevarajan · ramaswamydevarajan · commit 869b48f42843 · 2016-01-21T13:22:45.000-08:00
diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Notebook.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Notebook.java
@@ -167,7 +167,7 @@ public String exportNote(String noteId) throws IOException, IllegalArgumentExcep
     Gson gson = gsonBuilder.create();
     Note note = getNote(noteId);
     if (note == null) {
-      throw new IllegalArgumentException(noteId + "not found");
+      throw new IllegalArgumentException(noteId + " not found");
     }
     return gson.toJson(note);
   }
@@ -200,7 +200,8 @@ public Note importNote(String sourceJson, String noteName) throws IOException {
 
       newNote.persist();
     } catch (IOException e) {
-      throw new IOException(e);
+      logger.error(e.toString(), e);
+      throw e;
     }
     
     return newNote;

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,7 @@ public String exportNote(String noteId) throws IOException, IllegalArgumentExcep`
`167`	`167`	`Gson gson = gsonBuilder.create();`
`168`	`168`	`Note note = getNote(noteId);`
`169`	`169`	`if (note == null) {`
`170`		`- throw new IllegalArgumentException(noteId + "not found");`
	`170`	`+ throw new IllegalArgumentException(noteId + " not found");`
`171`	`171`	`}`
`172`	`172`	`return gson.toJson(note);`
`173`	`173`	`}`
`@@ -200,7 +200,8 @@ public Note importNote(String sourceJson, String noteName) throws IOException {`
`200`	`200`
`201`	`201`	`newNote.persist();`
`202`	`202`	`} catch (IOException e) {`
`203`		`- throw new IOException(e);`
	`203`	`+ logger.error(e.toString(), e);`
	`204`	`+ throw e;`
`204`	`205`	`}`
`205`	`206`
`206`	`207`	`return newNote;`