Github user HyukjinKwon commented on the issue:
https://github.com/apache/zeppelin/pull/3206
Thank you guys!
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/3206
yes, everything is out (except for the announcement)
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/3217
LGTM
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/3220
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/3220
Merge if no more comments
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3220
LGTM
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/3219
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/3218
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/3219
Thanks @felixcheung for review. Merge to master.
---
Github user NicolasRouquette commented on the issue:
https://github.com/apache/zeppelin/pull/3215
There's already unit tests in `SparkRInterpreterTest.java`; what
specifically do you suggest adding?
---
10 matches
Mail list logo