[GitHub] [zeppelin] liuxunorg opened a new pull request #3334: [ZEPPELIN-4072] Fix CreateNote() throw Note existed exception caused travis to fail

[GitBox]

liuxunorg opened a new pull request #3334: [ZEPPELIN-4072] Fix CreateNote() 
throw Note existed exception caused travis to fail
URL: https://github.com/apache/zeppelin/pull/3334
 
 
   ### What is this PR for?
   Fix CreateNote() function throw Note existed IOException caused travis to 
fail.
   When using trivis-ci, The exception will be triggered randomly.
   
   #### **Cause Analysis:**
   1. Recently, Notebook added a new one in createNote() to determine if the 
note name already exists. If it exists, an exception is thrown.
   2. In the test case, the code
   TestUtils.getInstance(Notebook.class) creates a global single instance 
object.
   
   3. Program processing flow
   
   + TestUtils.getInstance(Notebook.class).createNote("note1", anonymous);
   + After executing some test code, eg: spark, python, etc
   + TestUtils.getInstance(Notebook.class).removeNote(note.getId(), anonymous);
   + Delete this note.
   + If the spark, python exception is thrown before removeNote, then the note 
still exists. Causes other test cases to be abnormal.
   
   
   ### What type of PR is it?
   [Bug Fix]
   
   ### Todos
   * [ ] - Task
   
   ### What is the Jira issue?
   https://issues.apache.org/jira/browse/ZEPPELIN-4072
   
   ### How should this be tested?
   * travis pass
   
   ### Screenshots (if appropriate)
   <img width="1013" alt="WechatIMG1" 
src="https://user-images.githubusercontent.com/3677382/54428195-ab1c5b80-4757-11e9-9c8a-232847d8b2b3.png";>
   <img width="1223" alt="WechatIMG2" 
src="https://user-images.githubusercontent.com/3677382/54428213-b7a0b400-4757-11e9-982c-0fbd85dd84bb.png";>
   
   ### Questions:
   * Does the licenses files need update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services