[GitHub] zeppelin issue #1809: ZEPPELIN-1866. No meaningful exception for pig interpr...

2017-01-03 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1809 Cool. ~~It also looks like CI is passing.~~ Hmm, I guess not.. Merging if no more comment. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1809: ZEPPELIN-1866. No meaningful exception for pig interpr...

2017-01-02 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1809 Cool. It also looks like CI is passing. Merging if no more comment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] zeppelin issue #1809: ZEPPELIN-1866. No meaningful exception for pig interpr...

2017-01-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1809 I think these are flaky test. Because all the failed test are not pig related. Changes in this PR is only on pig interpreter. ``` Failed tests:

[GitHub] zeppelin issue #1809: ZEPPELIN-1866. No meaningful exception for pig interpr...

2016-12-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1809 @felixcheung Please help review. CI failure is not relevant. ``` AngularElem - should provide onclick method *** FAILED *** The code passed to eventually never returned normally.