[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2017-06-01 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/932 Hi all, tested it and it works. I understood that https://github.com/apache/zeppelin/pull/2182 is on hold because this one is also going to fix it but in a better way? Can we move forward

[GitHub] zeppelin issue #2354: Allow group/role based authentication using LdapRealm ...

2017-05-29 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2354 Hi @khalidhuseynov! Thanks for looking into it. Is there a way to support you? --- 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 #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-04-03 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Is there something I can do to accelerate the merge process here? Any open questions? --- 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 #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-31 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2176 I get what you are saying and I agree. My only point was that you need to be aware that this change only does the following thing: **Before**: Who persists the local changes earlier, wins

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-31 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2176 I am not sure if this change helps. We still have a race condition and conflict resolution is still "forceful overwrite and don't notify". If I see it correctly: *Before

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-30 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2177 Thanks @1ambda. I'm quite busy at this moment so if someone else (e.g. @AhyoungRyu) can also take a look that would be greatly appreciated. --- If your project is set up for it, you can rep

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-30 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2176 Sorry I was traveling @1ambda :( I understand what you are saying but I still don't understand exactly how it will work after this PR. Consider the following situation: - U

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-23 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2177 Gotcha @1ambda, The gifs make it clear now, thanks! Not sure about the exact code changes as I'm not that familiar with the code base you're touching. Do you have any speci

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-23 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2176 Understood @1ambda . Thanks for updating the gifs, now it is clear to me except one thing. When the second notebook had `user1` written and the first notebook executed the paragraph with `admin

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Looking at the discussion and the different issues I see what you mean @1ambda. We are actually having two issues: 1. Local changes are overwritten when the notebook gets updates from the

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2177 I am not sure I understand the exact difference between this and #2176. Can you try to upload bigger versions of the gif or elaborate a bit on the difference? Are the two PR (#2176 and #2177

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2176 Hi @1ambda. What are the implications of this change? In the scenario you describe, it makes sense. So if person A is editing a cell and person B is running it at the same time

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Here you go @1ambda ![zeppelin-1492-1](https://cloud.githubusercontent.com/assets/3427394/24218616/2cf9cd0c-0f44-11e7-93bb-5ae375c0296b.gif) --- If your project is set up for it, you

[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-16 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2137 Works like a charm @Leemoonsoo, thanks! --- 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

[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2137 Ok @Leemoonsoo. So what does this PR actually do if it still waits 2 minutes now at a different stage? Or can I somehow prevent Zeppelin from getting the available helium package list? --- If

[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2137 Hi @Leemoonsoo, I built your branch (https://travis-ci.org/FRosner/docker-zeppelin/builds/211271444) and tried it. However, it still waits for 2 minutes, now with a different '

[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2137 LGTM. I'm gonna build it today and give it a shot. I'll let you know how it went :) Thanks @Leemoonsoo --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin pull request #2120: [ZEPPELIN-1492] fixing the issue where updating...

2017-03-14 Thread FRosner
GitHub user FRosner reopened a pull request: https://github.com/apache/zeppelin/pull/2120 [ZEPPELIN-1492] fixing the issue where updating a paragraph was not propagated correctly ### What is this PR for? This pull request fixes two issues regarding paragraphs not being

[GitHub] zeppelin pull request #2120: [ZEPPELIN-1492] fixing the issue where updating...

2017-03-14 Thread FRosner
Github user FRosner closed the pull request at: https://github.com/apache/zeppelin/pull/2120 --- 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 feature is

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-13 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Do you want me to recreate the fork? I forked it long time ago :D -- Sent from my phone. Please excuse my brevity. On 03/13/2017, 16:05 Lee moon soo wrote: @FRosner

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-13 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 @1ambda it's building fine at travis: https://travis-ci.org/FRosner/incubator-zeppelin Anything else I need to do? Why is Jenkins complaining? --- If your project is set up for it

[GitHub] zeppelin issue #2120: WIP: ZEPPELIN-1492

2017-03-10 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2120 Will do 👍 --- 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

[GitHub] zeppelin pull request #2120: WIP ZEPPELIN-1492

2017-03-10 Thread FRosner
GitHub user FRosner opened a pull request: https://github.com/apache/zeppelin/pull/2120 WIP ZEPPELIN-1492 ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-09 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2095 That would be amazing. :) -- Sent from my phone. Please excuse my brevity. --- 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 #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2095 @NohSeho I have the problem that we are behind a proxy and the proxy is not configured. I don't want npm and I don't want node.js but Zeppelin tries to download it for 2 minutes. I th

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2095 Thanks for clarifying @1ambda. Is there a Jira issue for the first point already? Can you point me to the part of the code where this is configured? --- If your project is set up for it, you can

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2095 @1ambda you can see it at the time (2 minutes difference): ``` INFO [2017-03-07 12:52:34,074] ({main} FileDownloader.java[execute]:91) - No proxy was configured, downloading directly

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/2095 Hi @NohSeho I built Zeppelin after your fix and checked again. Now it is still hanging for two minutes before actually moving to the part that was changed. The situation we have is that we have a

[GitHub] zeppelin pull request #:

2017-03-01 Thread FRosner
Github user FRosner commented on the pull request: https://github.com/apache/zeppelin/commit/300f7532342d1ea47b85d3b777a8797a3e2248d4#commitcomment-21101395 In zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/HeliumVisualizationFactory.java: In zeppelin-zengine/src/main

[GitHub] zeppelin pull request #:

2017-02-28 Thread FRosner
Github user FRosner commented on the pull request: https://github.com/apache/zeppelin/commit/300f7532342d1ea47b85d3b777a8797a3e2248d4#commitcomment-21095991 In zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/HeliumVisualizationFactory.java: In zeppelin-zengine/src/main

[GitHub] zeppelin pull request #:

2017-02-28 Thread FRosner
Github user FRosner commented on the pull request: https://github.com/apache/zeppelin/commit/300f7532342d1ea47b85d3b777a8797a3e2248d4#commitcomment-21090241 In zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/HeliumVisualizationFactory.java: In zeppelin-zengine/src/main

[GitHub] zeppelin issue #1422: Allow matplotlib plot display in pyspark context

2016-09-15 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/1422 Can we have some integration test to verify this one? --- 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

[GitHub] zeppelin issue #673: [Zeppelin-630] Introduce new way of dependency loading ...

2016-08-29 Thread FRosner
Github user FRosner commented on the issue: https://github.com/apache/zeppelin/pull/673 Haven't tried @nihakue. Can you try to load a jar through z.load? Maybe it's some issue related to Spark packages + %dep? --- If your project is set up for it, you can reply to this