[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2238 Thanks @alourie and Rob Marshall. This PR has merged. but @asfgit didn't automatically closed it. @alourie Could you close this PR manually? --- If your project is set up for it, you

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-13 Thread alourie
Github user alourie commented on the issue: https://github.com/apache/zeppelin/pull/2238 @Leemoonsoo my Jira ID is alourie, but please assign it to Rob Marshall, who opened the ticket. He did all the development, I just helped with the PR/CI :-) --- If your project is set up for it,

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2238 LGTM. Merge this PR to master if no further discussions. @alourie Could you let me know your JIRA id? i'd like to mark https://issues.apache.org/jira/browse/ZEPPELIN-1501 resolved with

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-13 Thread alourie
Github user alourie commented on the issue: https://github.com/apache/zeppelin/pull/2238 @Leemoonsoo oh wow, who knew. So now it works. All checks passed :-) Thanks for the help! --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2238 Looks like code need to handle null return of `getProperty(CASSANDRA_WITH_SSL)` at `org.apache.zeppelin.cassandra.CassandraInterpreter.open(CassandraInterpreter.java:200)` --- If your project

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-12 Thread alourie
Github user alourie commented on the issue: https://github.com/apache/zeppelin/pull/2238 @Leemoonsoo I've tried to rerun them few times. Only one now fails - build #4; fails with NPE in C* setup/tearDown: ```Tests run: 2, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 40.22

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2238 They might be flaky tests. Could you retry failed jobs and see if it goes green? --- 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 #2238: 0.7.1 with ssl

2017-07-11 Thread instaclustr-oss
Github user instaclustr-oss commented on the issue: https://github.com/apache/zeppelin/pull/2238 Hi @Leemoonsoo Sorry for taking awhile to setup the CI. Now that I have, I see some jobs fail. What should I do about it? Thanks! --- If your project is set up for it,

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-04-22 Thread alourie
Github user alourie commented on the issue: https://github.com/apache/zeppelin/pull/2238 Hi @Leemoonsoo! I have it in my task list, will get to it probably next week. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-04-21 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2238 Thanks @alourie for the contribution Could you configure CI? Following message from Jekins might help. ``` Looks like travis-ci is not configured for your fork. Please setup