GitHub user NicolasRouquette opened a pull request: https://github.com/apache/zeppelin/pull/3216
[ZEPPELIN-3167] ### What is this PR for? Split the R assignment in two steps, evaluation and assignment, such that the tryCatch in the R eval loop can detect errors and report them to Zeppelin. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/ * https://issues.apache.org/jira/browse/ZEPPELIN-3167 ### How should this be tested? * First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration * Strongly recommended: add automated unit tests for any new or changed behavior * Outline any manual steps to test the PR here. ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? You can merge this pull request into a Git repository by running: $ git pull https://github.com/NicolasRouquette/zeppelin master-ZEPPELIN-3167 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zeppelin/pull/3216.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 #3216 ---- commit 83ada910d19d6f7ca94a90f9ebf3ffa83f65eb6b Author: Nicolas F Rouquette <nicolas.f.rouquette@...> Date: 2018-11-03T18:19:35Z Split the assignment into two steps, evaluation and assignment, such that the tryCatch in the eval loop can detect and report errors back to Zeppelin ---- ---