[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2518 Thanks @SachinJanani for the fix! 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 appear on

[GitHub] zeppelin issue #2502: [ZEPPELIN-2807] Passing Z variables to SQL Interpreter...

2017-08-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2502 @sanjaydasgupta Thanks for addressing newline issue. If we take a look dynamic form, each interpreter can choose whether interpreter wants to use Dynamic form or not by overriding [Inte

[GitHub] zeppelin issue #2495: [HOTFIX] JDBC completions. fix cursor position

2017-08-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2495 Merge to master if no further comments --- 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 #2532: [ZEPPELIN-2850]

2017-08-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2532 Thanks for @imnotkind addressing BOM. Tested and LGTM. CI failure is not related to this change. Merge to master if no further comment. --- If your project is set up for it, you ca

[GitHub] zeppelin issue #2526: [ZEPPELIN-2848] Added new type of user to only run not...

2017-08-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2526 ok, sorry for dragging it on... then I'll flip the question, are Writers also Runners then? In the case someone is upgrading from the current version of Zeppelin with an existing notebook

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

2017-08-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2373 Hi! Please redirect your questions to dev@zeppelin.apache.org or open a JIRA if there is a bug? --- If your project is set up for it, you can reply to this email and have your reply appear o

[jira] [Created] (ZEPPELIN-2870) Could Not Start Zeppelin

2017-08-17 Thread Umair Saleem (JIRA)
Umair Saleem created ZEPPELIN-2870: -- Summary: Could Not Start Zeppelin Key: ZEPPELIN-2870 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2870 Project: Zeppelin Issue Type: Bug Affec

[jira] [Created] (ZEPPELIN-2869) .take(1) method on Dataset[String] fails due to inability to deserialize (NoSuchMethodError)

2017-08-17 Thread Shafique Jamal (JIRA)
Shafique Jamal created ZEPPELIN-2869: Summary: .take(1) method on Dataset[String] fails due to inability to deserialize (NoSuchMethodError) Key: ZEPPELIN-2869 URL: https://issues.apache.org/jira/browse/ZEPPELI

[GitHub] zeppelin issue #2520: ZEPPELIN-2833 JDBC Interpreter doesn't follow JDBC spe...

2017-08-17 Thread ss77892
Github user ss77892 commented on the issue: https://github.com/apache/zeppelin/pull/2520 @felixcheung As I mentioned in the JIRA the default jdbc driver that is used in tests has different behavior ( allows to make call next() even if the result set is already at the end). Not sure ho

[GitHub] zeppelin issue #2526: [ZEPPELIN-2848] Added new type of user to only run not...

2017-08-17 Thread gfalcone
Github user gfalcone commented on the issue: https://github.com/apache/zeppelin/pull/2526 @felixcheung nothing would change for them because readers cannot run the notebook. The only rights allowed to run a notebook are writers and owners --- If your project is set up for it, you ca

[GitHub] zeppelin pull request #2536: [ZEPPELIN-2861] Use OpenJDK in docker image.

2017-08-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2536 --- 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 #2526: [ZEPPELIN-2848] Added new type of user to only run not...

2017-08-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2526 I see. In the case someone is upgrading from the current version of Zeppelin with an existing notebook with Readers role set, would they run into trouble that those users can no longer run the

[GitHub] zeppelin issue #2543: [ZEPPELIN-2860] Invalid last job execution time

2017-08-17 Thread tae-jun
Github user tae-jun commented on the issue: https://github.com/apache/zeppelin/pull/2543 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

Re: What about the release of Zeppelin 0.7.3

2017-08-17 Thread moon soo Lee
Hi, 0.7.3 is a maintenance release, so in terms of new feature, I prefer to ship it in 0.8.0, so we can keep 0.7.3 as a stable release. We have a discussion thread for release 0.8.0 [1]. And I think we can discuss there, more about 0.8.0 release which includes all the new functionalities. Does i

Re: What about the release of Zeppelin 0.7.3

2017-08-17 Thread Polyakov Valeriy
Hi, Moon! Is there any possibility to include in 0.7.3 some critical functionality (for Tinkoff) out of security scope? It will help us to upgrade our platform in more stable mode. Now we are forced to use unstable version built from development snapshot in production. On 2017-08-16 18

[GitHub] zeppelin issue #2541: [ZEPPELIN-2865] upgrade Beam interpreter to latest ver...

2017-08-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2541 LGTM. Merge if 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 your project does not have this feature enabled

[GitHub] zeppelin issue #2502: [ZEPPELIN-2807] Passing Z variables to SQL Interpreter...

2017-08-17 Thread sanjaydasgupta
Github user sanjaydasgupta commented on the issue: https://github.com/apache/zeppelin/pull/2502 Thanks for the catch # @Leemoonsoo I have fixed and committed the issue. --- 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 #2532: [ZEPPELIN-2850]

2017-08-17 Thread imnotkind
Github user imnotkind commented on the issue: https://github.com/apache/zeppelin/pull/2532 oh i didn't realize there were certain issues with excel. i included BOM, and i added revokeObjectURL (decrementing reference count) to enable garbage collecting. now unicode works fine! --- I

[GitHub] zeppelin pull request #2502: [ZEPPELIN-2807] Passing Z variables to SQL Inte...

2017-08-17 Thread sanjaydasgupta
Github user sanjaydasgupta closed the pull request at: https://github.com/apache/zeppelin/pull/2502 --- 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 feat

[GitHub] zeppelin pull request #2502: [ZEPPELIN-2807] Passing Z variables to SQL Inte...

2017-08-17 Thread sanjaydasgupta
GitHub user sanjaydasgupta reopened a pull request: https://github.com/apache/zeppelin/pull/2502 [ZEPPELIN-2807] Passing Z variables to SQL Interpreter (One part of ZEPPELIN-1967) ### What is this PR for? This feature allows embedding/interpolating Z variables into SQL stat