[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-08-02 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/3066 will merge it soon ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-08-02 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/3066 looks like everything passed https://travis-ci.org/TinkoffCreditSystems/zeppelin/builds/407590408 ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-08-01 Thread egorklimov
Github user egorklimov commented on the issue: https://github.com/apache/zeppelin/pull/3066 @felixcheung are there still any defects? ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-24 Thread egorklimov
Github user egorklimov commented on the issue: https://github.com/apache/zeppelin/pull/3066 @jongyoul CI is green https://travis-ci.org/TinkoffCreditSystems/zeppelin/builds/407590408 ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-23 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/3066 Your branch is not configured travis. Please enable travis for your branch. ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-23 Thread egorklimov
Github user egorklimov commented on the issue: https://github.com/apache/zeppelin/pull/3066 @felixcheung I added `TException` and now it seems to be ok, thank you. * Zeppelin log for z.run("WrongParagraphId") and z.runNote("WrongNoteId"): ``` ERROR [2018-07-23

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-23 Thread egorklimov
Github user egorklimov commented on the issue: https://github.com/apache/zeppelin/pull/3066 @jongyoul result is unfortunately the same https://travis-ci.org/TinkoffCreditSystems/zeppelin/builds/407075673 ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-23 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/3066 @egorklimov You could make CI gree if you rebased from the current master ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/3066 can you comment on my comment: `this doesn't seem quite right, we are ignoring all IOException from running the paragraph?`? ---

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-20 Thread egorklimov
Github user egorklimov commented on the issue: https://github.com/apache/zeppelin/pull/3066 @felixcheung look again please. Here is some troubles with CI: tests 6 & 7 always fail, even after cleaning cache: https://travis-ci.org/TinkoffCreditSystems/zeppelin/builds/404476903

[GitHub] zeppelin issue #3066: [ZEPPELIN-3618] ZeppelinContext methods z.run and z.ru...

2018-07-12 Thread egorklimov
Github user egorklimov commented on the issue: https://github.com/apache/zeppelin/pull/3066 I didn't find out how to send error message in paragraph result, my best try also removed other output in that paragraph, It would be nice if you could suggest an easy way to make it. ---