[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/666 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-175825325 merging if there is no more discussion --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-26 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-175465193 It looks good to me. Thanks @Karuppayya for the improvement. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-26 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-175448598 @Leemoonsoo what do you think? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your pr

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-173821304 Travis still not happy, not sure why, but Travis is passing in other PRs, like 664 & 665 --- If your project is set up for it, you can reply to this ema

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-21 Thread Karuppayya
Github user Karuppayya commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-173816608 @felixcheung @Leemoonsoo The branch used in old PR got messed up due to a bad merge, hence creating this new PR. Link to old one https://github.com/apa

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-21 Thread Karuppayya
GitHub user Karuppayya opened a pull request: https://github.com/apache/incubator-zeppelin/pull/666 Shell interpreter improvements Creating new PR with the changes from https://github.com/apache/incubator-zeppelin/pull/615 Please check the above PR for prior discussions.

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/615#issuecomment-173765158 I'm not sure about the error, too. I'll have some time in this weekend to investigate this error more closely. --- If your project is set up for it, y

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/615#issuecomment-173759652 yes, this is weird. I've tested your changes and they seem to run fine. @Leemoonsoo any idea? @Karuppayya don't forget to open this PR again - it's ve

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-21 Thread Karuppayya
Github user Karuppayya closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/615 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-20 Thread Karuppayya
Github user Karuppayya commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/615#issuecomment-173249137 I get the following error Running org.apache.zeppelin.interpreter.remote.RemoteAngularObjectTest 15:34:27,697 INFO org.apache.zeppelin.interpreter

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-20 Thread Karuppayya
GitHub user Karuppayya reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/615 Shell interpreter improvements ### What is this PR for? *Provide ability to to run shell commands in parallel *Provide ability to cancel shell command *Propagate the err

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-20 Thread Karuppayya
Github user Karuppayya closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/615 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-19 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/615#issuecomment-173106887 @Karuppayya could you trigger travis again? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-13 Thread Karuppayya
GitHub user Karuppayya reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/615 Shell interpreter improvements ### What is this PR for? *Provide ability to to run shell commands in parallel *Provide ability to cancel shell command *Propagate the err

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-13 Thread Karuppayya
Github user Karuppayya closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/615 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-11 Thread Karuppayya
GitHub user Karuppayya reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/615 Shell interpreter improvements ### What is this PR for? *Provide ability to to run shell commands in parallel *Provide ability to cancel shell command *Propagate the err

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-08 Thread Karuppayya
Github user Karuppayya closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/615 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-08 Thread Karuppayya
GitHub user Karuppayya opened a pull request: https://github.com/apache/incubator-zeppelin/pull/615 Shell interpreter improvements ### What is this PR for? *Provide ability to to run shell commands in parallel *Provide ability to cancel shell command *Propagate the error