[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-03 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2439 Oh, Thanks for quick answer and check! Let me test as your comment and come back here! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-03 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2439 @soralee thanks for taking the time :-) there's one more step to your list: 1. login as user1 2. create a note 3. add a paragraph **4. change the visualization to a chart (instea

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-03 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2439 I have a question quickly. For reproduce, I just make sure test flow clearly. 1. login as user1 2. create a note 3. add a paragraph 4. set `owners` and `writers` field set user1

[GitHub] zeppelin issue #2268: [ZEPPELIN-2403] interpreter property widgets

2017-07-03 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2268 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

[jira] [Created] (ZEPPELIN-2718) Find and replace code in note

2017-07-03 Thread Tinkoff DWH (JIRA)
Tinkoff DWH created ZEPPELIN-2718: - Summary: Find and replace code in note Key: ZEPPELIN-2718 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2718 Project: Zeppelin Issue Type: New Featur

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-03 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2439 Hi, @herval. Thanks for contribution and sorry for late response. Let me take a look this one. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-03 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2439 any thoughts on 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 enabled and wishes s

[GitHub] zeppelin issue #2440: [ZEPPELIN-2587] allow logging in if you're anonymous

2017-07-03 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2440 any additional thoughts on this? Is it mergeable? @jongyoul @cfries --- 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

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-07-03 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2428 any additional thoughts on this? Is it mergeable? @khalidhuseynov @tinkoff-dwh @felixcheung --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin pull request #2446: [ZEPPELIN-2699] Helium REST API bug-fixes. Also...

2017-07-03 Thread necosta
Github user necosta closed the pull request at: https://github.com/apache/zeppelin/pull/2446 --- 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 pull request #2446: [ZEPPELIN-2699] Helium REST API bug-fixes. Also...

2017-07-03 Thread necosta
GitHub user necosta reopened a pull request: https://github.com/apache/zeppelin/pull/2446 [ZEPPELIN-2699] Helium REST API bug-fixes. Also updated documentation ### What is this PR for? Fixing bug on enabling Helium plugins (artifact is empty, not null) when sending curl command.

[GitHub] zeppelin pull request #2456: [ZEPPELIN-2714] Soft-code Spark UI button visua...

2017-07-03 Thread necosta
GitHub user necosta reopened a pull request: https://github.com/apache/zeppelin/pull/2456 [ZEPPELIN-2714] Soft-code Spark UI button visualization ### What is this PR for? When "spark.ui.enabled" property is set to "false" we should not show the Spark UI button. We keep the s

[GitHub] zeppelin pull request #2456: [ZEPPELIN-2714] Soft-code Spark UI button visua...

2017-07-03 Thread necosta
Github user necosta closed the pull request at: https://github.com/apache/zeppelin/pull/2456 --- 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 #2458: Zeppelin 2701 - Set HTML title for notebook page

2017-07-03 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2458 Hi, @ebuildy. Thanks for improving it. But Travis is fail, could you check this? --- 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 #2443: [ZEPPELIN-2698] Apply KerberosInterpreter to JDBCInter...

2017-07-03 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2443 @zjffdu I've made the suggested changes. Let me know if I'm missing something else as well. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin pull request #2448: [ZEPPELIN-2702] save notes in reader-friendly f...

2017-07-03 Thread khalidhuseynov
GitHub user khalidhuseynov reopened a pull request: https://github.com/apache/zeppelin/pull/2448 [ZEPPELIN-2702] save notes in reader-friendly format with zpln extension for VFS and Git repo ### What is this PR for? first task in converting note files into human readable and Zep

[GitHub] zeppelin pull request #2448: [ZEPPELIN-2702] save notes in reader-friendly f...

2017-07-03 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at: https://github.com/apache/zeppelin/pull/2448 --- 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 #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-03 Thread jerryshao
GitHub user jerryshao reopened a pull request: https://github.com/apache/zeppelin/pull/2459 [ZEPPELIN-2716] Change the default value of zeppelin.livy.displayAppInfo to true ### What is this PR for? Since it is quite useful to expose the application info for user to monitor

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-03 Thread jerryshao
Github user jerryshao closed the pull request at: https://github.com/apache/zeppelin/pull/2459 --- 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 i

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-03 Thread jerryshao
GitHub user jerryshao reopened a pull request: https://github.com/apache/zeppelin/pull/2459 [ZEPPELIN-2716] Change the default value of zeppelin.livy.displayAppInfo to true ### What is this PR for? Since it is quite useful to expose the application info for user to monitor

[GitHub] zeppelin pull request #2459: [ZEPPELIN-2716] Change the default value of zep...

2017-07-03 Thread jerryshao
Github user jerryshao closed the pull request at: https://github.com/apache/zeppelin/pull/2459 --- 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 i

[GitHub] zeppelin pull request #2457: [ZEPPELIN-2713] Fix NPE of notebook creat rest ...

2017-07-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2457 --- 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 #2456: [ZEPPELIN-2714] Soft-code Spark UI button visualizatio...

2017-07-03 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2456 @felixcheung , @Leemoonsoo , ready for re-review. 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 ha

[GitHub] zeppelin issue #2448: [ZEPPELIN-2702] save notes in reader-friendly format w...

2017-07-03 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2448 @zjffdu addressed the feedback on adding new `rename` method and having only one fileInfo, please help review --- If your project is set up for it, you can reply to this email and have yo

[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-07-03 Thread cacti77
Github user cacti77 commented on the issue: https://github.com/apache/zeppelin/pull/2437 @1ambda Thanks for finishing the diagrams. I would probably put the SparkContext inside the interpreter process but outside the InterpreterGroup; but if @zjffdu is happy with the diagrams I guess

[GitHub] zeppelin issue #2442: [ZEPPELIN-2679] JDBC. precode for session

2017-07-03 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2442 it is JDBC interpretator, set variables in first paragraph will not work because we have connection pool and the paragraphs will be executed in different statemnets. What problem with shared

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-03 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @echarles Hi, I saw Twill and met one of committers at Apache BigData NA. I agree with @zjffdu's opinion. After merging this PR, I'll take a look into that project. --- If your project is set u

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-03 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/2418 In case of SNPEGO Web Auth (not yet supported by Zeppelin0,, you could consider transfert the Kerberos ticket from front-end to back-end and use it to , this the usecase I was thinking to.

[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-07-03 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2437 Thanks for the review. @felixcheung Resolved all comments. @zjffdu @cacti77 Update imgs and description. See the last 2 commits. --- If your project is set up for it, you can reply to

[GitHub] zeppelin pull request #2447: [ZEPPELIN-2700] Provide a link to interpreter b...

2017-07-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2447 --- 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 #2451: [ZEPPELIN-2708][DOCS] feat: Add v scroll to set...

2017-07-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2451 --- 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 #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-03 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2418 >> Do we need to support kerberos-like auth in the communication protocol (you can achieve this with Thrift, not sure with Netty)? This is internal communication channel between zeppelin se

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-03 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 Thanks @echarles I also thought about it before, but not sure whether these 2 project still active. Maybe we need do some investigation. --- If your project is set up for it, you can reply to thi

[GitHub] zeppelin issue #2442: [ZEPPELIN-2679] JDBC. precode for session

2017-07-03 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2442 I am thinking that every interpreter could have one special interpreter for initializing interpreter setting and custom enviroment requirement for his note. And do the initialization at the first p

[GitHub] zeppelin issue #2459: [ZEPPELIN-2716] Change the default value of zeppelin.l...

2017-07-03 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2459 Thanks @jerryshao 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

[GitHub] zeppelin pull request #2460: [gh-pages] fix hover event in technologies sect...

2017-07-03 Thread soralee
GitHub user soralee opened a pull request: https://github.com/apache/zeppelin/pull/2460 [gh-pages] fix hover event in technologies section on mobile ### What is this PR for? This is for fixing for `hover event` in the technologies section on mobile (I'm using Iphone 6). A ho

[GitHub] zeppelin issue #2442: [ZEPPELIN-2679] JDBC. precode for session

2017-07-03 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2442 @zjffdu another instance? user == analitic. It is real task from production. The user does't configure interpreter, it is work of administrator. --- If your project is set up for it, you

[jira] [Created] (ZEPPELIN-2717) Crashes after interpreter restart

2017-07-03 Thread Guillaume Van Delsen (JIRA)
Guillaume Van Delsen created ZEPPELIN-2717: -- Summary: Crashes after interpreter restart Key: ZEPPELIN-2717 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2717 Project: Zeppelin

[GitHub] zeppelin issue #2268: [ZEPPELIN-2403] interpreter property widgets

2017-07-03 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2268 1,2 done @1ambda thx for review --- 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 e

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-03 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/2329 Late question: This implementation uses the raw Yarn APIs. Did you consider using higher-level APIs such as Twill (http://twill.apache.org) or Slider (https://slider.incubator.apache.org) ? ---

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-07-03 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/2418 I'd like adding more points to the discussion: - Do we need to support kerberos-like auth in the communication protocol (you can achieve this with Thrift, not sure with Netty)? - Do we tar