[GitHub] zeppelin issue #2381: [WIP] [gh-pages] [ZEPPELIN-2597] Updating offcial main...

2017-06-01 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 one more bug problem, if I escape `download` menu, color still leaves active color. ![hover bug](https://cloud.githubusercontent.com/assets/8110458/26714144/26e9d3ea-47ab-11e7-9496-d1072b1aca2

[GitHub] zeppelin issue #2388: [ZEPPELIN-2492] Use single Dockerfile for each tag (br...

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2388 @Leemoonsoo This is created for branch-0.7. For master, https://github.com/apache/zeppelin/pull/2318 was already merged. Could you help to review this? --- If your project is set up for i

[GitHub] zeppelin pull request #2388: [ZEPPELIN-2492] Use single Dockerfile for each ...

2017-06-01 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2388 [ZEPPELIN-2492] Use single Dockerfile for each tag (branch-0.7) ### What is this PR for? **This PR is created for branch-0.7. for master, https://github.com/apache/zeppelin/pull/2318 was

[GitHub] zeppelin pull request #2387: [ZEPPELIN-2573] fix: Small font-size for navbar...

2017-06-01 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2387 [ZEPPELIN-2573] fix: Small font-size for navbar menus ### What is this PR for? Navbar style was changed by https://github.com/apache/zeppelin/pull/2355 But font-size for navbar menu is

[GitHub] zeppelin issue #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

2017-06-01 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2373 @1ambda To be honest I'm not such an expert and I'm just integrating from a technical POV a solution that other experts in team found. I will love to have such documentation too, but I ca

[GitHub] zeppelin pull request #2386: [ZEPPELIN-2603] fix: Broken Paragraph Title

2017-06-01 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2386 --- 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 e

[GitHub] zeppelin pull request #2386: [ZEPPELIN-2603] fix: Broken Paragraph Title

2017-06-01 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2386 [ZEPPELIN-2603] fix: Broken Paragraph Title ### What is this PR for? Paragraph titles are split into multiple lines when they are long. - let's applying text-overflow like the n

[GitHub] zeppelin pull request #2386: [ZEPPELIN-2603] fix: Broken Paragraph Title

2017-06-01 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2386 [ZEPPELIN-2603] fix: Broken Paragraph Title ### What is this PR for? Paragraph titles are split into multiple lines when they are long. - let's applying text-overflow like the not

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2384 Regarding to Promsie polyfill, we need to turn on the `useBuiltIn` option in babel-preset-env - https://github.com/babel/babel-preset-env#usebuiltins - https://github.com/babel/babel-pr

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2384 https://github.com/apache/zeppelin/pull/2273 was merged into master only. --- 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 pro

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

2017-06-01 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2354 @sohaibiftikhar i guess we can rebase it from master now --- 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

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2384 Tested the Master branch in IE11. and got different error. ![image](https://cloud.githubusercontent.com/assets/1540981/26710534/607ee4ca-470f-11e7-8adb-59c0d0954ac6.png) @1ambd

[GitHub] zeppelin issue #2326: [ZEPPELIN-2519] Disable watchers not in viewport

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2326 Hi @vipul1409, This PR occurs an error like ```js TypeError: Cannot read property 'create' of undefined at Object.link (http://localhost:9001/app.bundle.js:9662:52) at ht

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2384 @Leemoonsoo I think we don't need to merge this PR into master. because the master branch supports IE9+ ``` "browsers": ["last 5 version", "> 5%", "edge >= 12", "ie >= 9", "safari 7",

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2384 Thanks @1ambda @prabhjyotsingh for the review. Merge to master and branch-0.7 if no further comments. --- If your project is set up for it, you can reply to this email and have your reply appe

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

2017-06-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/932 --- 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 en

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

2017-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 Tested and LGTM. Sorry for the late reply. I'll merge it now. --- 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 n

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2384 LGTM! --- 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

[GitHub] zeppelin issue #2376: Branch 0.7

2017-06-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2376 @simulight , Could you close this PR and create a new one ? This PR is mixed with other commits. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[jira] [Created] (ZEPPELIN-2611) `Notebook Search` page doesn't display searched contents

2017-06-01 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2611: --- Summary: `Notebook Search` page doesn't display searched contents Key: ZEPPELIN-2611 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2611 Project: Zeppelin

[jira] [Created] (ZEPPELIN-2610) `Link to this paragraph` feature doesn't work at all (0.8.0-SNAPSHOT)

2017-06-01 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2610: --- Summary: `Link to this paragraph` feature doesn't work at all (0.8.0-SNAPSHOT) Key: ZEPPELIN-2610 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2610 Project: Zepp

[GitHub] zeppelin pull request #2385: [ZEPPELIN-2609] Add missing shortcut descriptio...

2017-06-01 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2385 [ZEPPELIN-2609] Add missing shortcut descriptions ### What is this PR for? Add missing shortcut descriptions. Refer the *TODOs* section and attached screenshots for detail. ###

[GitHub] zeppelin pull request #2385: [ZEPPELIN-2609] Add missing shortcut descriptio...

2017-06-01 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2385 --- 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 e

[GitHub] zeppelin issue #2385: [ZEPPELIN-2609] Add missing shortcut descriptions

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2385 @soralee Could you help review this? --- 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 enabl

[GitHub] zeppelin pull request #2385: [ZEPPELIN-2609] Add missing shortcut descriptio...

2017-06-01 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2385 [ZEPPELIN-2609] Add missing shortcut descriptions ### What is this PR for? Add missing shortcut descriptions. Refer the *TODOs* section and attached screenshots for detail. ### W

[jira] [Created] (ZEPPELIN-2609) Add missing editor shortcut description.

2017-06-01 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2609: --- Summary: Add missing editor shortcut description. Key: ZEPPELIN-2609 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2609 Project: Zeppelin Issue Type: Imp

[GitHub] zeppelin issue #2368: Improved comment, removed admin user definition per de...

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2368 @cfries Hi, could you restart or fix the failed job? - https://travis-ci.org/cfries/zeppelin/builds/236343303 --- If your project is set up for it, you can reply to this email and have you

[GitHub] zeppelin issue #2377: User does not land on Zeppelin UI Login page after cli...

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2377 Tested locally and works as described. LGTM except for a comment. --- 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 doe

[GitHub] zeppelin issue #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2384 LGTM. --- 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] zeppelin issue #2382: [ZEPPELIN-2185] No tooltip displayed on chart type but...

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2382 @0q Hi! Any update? :) --- 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 wis

[GitHub] zeppelin issue #2370: change the spark version of dockerfile

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2370 Hi @sven0726! Any update? :) --- 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 w

[GitHub] zeppelin issue #2379: [MINOR] fix: Remove newlines in spark tutorial output

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2379 @Leemoonsoo @felixcheung Thanks for the review. - I removed all the things which are not related to the output. - and the paragraph in the spark tutorial now looks ![image](htt

[GitHub] zeppelin issue #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

2017-06-01 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2373 Then can we add detailed documentation for "How to use this feature?" That's because users are having trouble when they setup security things and there is no proper documentation. LDAP is

[GitHub] zeppelin pull request #2383: [ZEPPELIN-2606] Fix compilation with R interpre...

2017-06-01 Thread andreaTP
GitHub user andreaTP reopened a pull request: https://github.com/apache/zeppelin/pull/2383 [ZEPPELIN-2606] Fix compilation with R interpreter enabled ### What is this PR for? `mvn -DskipTests clean package -Pr` fails ### What type of PR is it? Bug Fix ### Wh

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-01 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2383 Since R is under spark we need to enable it differently AFAICS now it is enabled at least in one test (where all other interpreters are compiled). Travis is happy I close and re-open this.

[GitHub] zeppelin pull request #2383: [ZEPPELIN-2606] Fix compilation with R interpre...

2017-06-01 Thread andreaTP
Github user andreaTP closed the pull request at: https://github.com/apache/zeppelin/pull/2383 --- 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 #2369: [ZEPPELIN-2591] Show user info in spark job descriptio...

2017-06-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2369 Thanks @karuppayya for the improvement. LGTM --- 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 f

[GitHub] zeppelin pull request #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-01 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2384 [ZEPPELIN-2577] No graphs visible on IE11 ### What is this PR for? This PR fixes issue described in https://issues.apache.org/jira/browse/ZEPPELIN-2577 by adding .endsWith method.

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

2017-06-01 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/932 Tested and LGTM. Merge to master if no further discussions. --- 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 proje

[GitHub] zeppelin pull request #2378: [ZEPPELIN-2602] fix: broken navbar-title render...

2017-06-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2378 --- 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 e

[GitHub] zeppelin pull request #2366: [ZEPPELIN-2471] - NotebookTest fails on travis

2017-06-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2366 --- 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 e

[jira] [Created] (ZEPPELIN-2608) Zeppelin not displaying d3 visualisation in cell

2017-06-01 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2608: Summary: Zeppelin not displaying d3 visualisation in cell Key: ZEPPELIN-2608 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2608 Project: Zeppelin Issue Type: B

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-01 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2383 good point, so I added it to the list, I think it fits the scope of this PR (let see what travis says) --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2383 @andreaTP That's is intended. It would be converted to inclusive. ``` MODULES="-pl $(echo .,zeppelin-interpreter,${INTERPRETERS} | sed 's/!//g')" ``` --- If your project is set up

[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 #2381: [WIP] [gh-pages] [ZEPPELIN-2597] Updating offcial main...

2017-06-01 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 I'm improving navigation bar but, I found two bug in navigation bar. so we should resolve those issues together. 1. when resizing window, drop-toggle is appeared. ![bug1](https://cloud

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-01 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2383 `INTERPRETERS` is surprisingly an exclusion list @zjffdu https://github.com/apache/zeppelin/blob/master/.travis.yml#L40 --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-01 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2383 I think `r` is missing in `INTERPRETERS` of `.travis.yml` --- 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 ha

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-01 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2383 LGTM --- 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

[GitHub] zeppelin pull request #2367: [ZEPPELIN-2584]. Livy doesn't work under kerber...

2017-06-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2367 --- 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 e

[GitHub] zeppelin issue #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

2017-06-01 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2373 @1ambda potentially yes @khalidhuseynov thanks for pointing out a public server I can test! I will go through later on B.t.w. I realized that the MVP to get this working is just