[GitHub] zeppelin issue #1661: [ZEPPELIN-1605] zeppelin-interpreter: don't publish sh...

2016-12-05 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1661 Sounds awesome, thank you! Let's merge then, if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #1661: [ZEPPELIN-1605] zeppelin-interpreter: don't publish sh...

2016-12-01 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1661 @bzz Thank you for review, I just rebased. And yes, the way you described is correct way to test this PR, I updated it in description. --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #1661: [ZEPPELIN-1605] zeppelin-interpreter: don't publish sh...

2016-11-30 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1661 @minahlee Looks great to me! Could you please rebase on latest master? I assume that for testing, it should be enough to build `mvn clean package -Pbuild-distr -DskipTests` and make