GitHub user egorklimov opened a pull request: https://github.com/apache/zeppelin/pull/3066
[Zeppelin-3618] ZeppelinContext methods z.run and z.runNote fall after passing wrong argument ### What is this PR for? Passing wrong argument to z.run() or z.runNote() causes error: * Zeppelin log: ``` ERROR [2018-07-12 18:19:05,110] ({pool-5-thread-1} TThreadPoolServer.java[run]:297) - Error occurred during processing of message. java.lang.RuntimeException: Not existed noteId: WrongNoteId at org.apache.zeppelin.interpreter.RemoteInterpreterEventServer.runParagraphs(RemoteInterpreterEventServer.java:250) ... ``` * Interpreter log: ``` WARN [2018-07-12 18:19:05,113] ({pool-2-thread-5} RemoteInterpreterEventClient.java[runParagraphs]:259) - Fail to runParagraphs: RunParagraphsEvent(noteId:broadcast, paragraphIds:[], paragraphIndices:[], curParagraphId:20180709-113817_1103600568) org.apache.thrift.transport.TTransportException ... INFO [2018-07-12 18:19:05,223] ({pool-2-thread-5} SchedulerFactory.java[jobFinished]:115) - Job 20180709-113817_1103600568 finished by scheduler interpreter_434750169 WARN [2018-07-12 18:20:10,756] ({pool-1-thread-1} RemoteInterpreterEventClient.java[onInterpreterOutputUpdateAll]:234) - Fail to updateAllOutput org.apache.thrift.transport.TTransportException ... ``` After that user couldn't use z.run() and z.runNote() until interpreter restart. ### What type of PR is it? Bug Fix ### What is the Jira issue? Issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN-3618 ### How should this be tested? * CI pass * Zeppelin log for `z.run("WrongParagraphId")` and `z.runNote("WrongNoteId")`: ``` ERROR [2018-07-12 19:32:49,125] ({pool-5-thread-1} RemoteInterpreterEventServer.java[runParagraphs]:250) - Not existed paragraphId: WrongParagraphId ERROR [2018-07-12 19:31:45,746] ({pool-5-thread-1} RemoteInterpreterEventServer.java[runParagraphs]:250) - Not existed noteId: WrongNoteId ``` * No error messages in interpreter log ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No You can merge this pull request into a Git repository by running: $ git pull https://github.com/TinkoffCreditSystems/zeppelin ZEPPELIN-3618 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3066.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #3066 ---- commit a336b8c41d8d43868486449a5b6ec726989d0bfd Author: egorklimov <klim.electronicmail@...> Date: 2018-07-12T18:05:13Z Fix failing runParagraphs ---- ---