[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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 (instead of table)**
5. set owners and writers field set user1
6. set readers field sets user2
7. logout user1 and login user2.
8. back to note and check whether popup is appeared or not.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
5. set `readers` field sets user2
6. logout user1 and login user2.
7. back to note and check whether popup is appeared or not.

Is it correct?






---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 Feature
  Components: front-end
Reporter: Tinkoff DWH
Assignee: Tinkoff DWH
Priority: Minor


Add ability to find and replace code in note.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[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 appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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.
After noticing out-of-date documentation, I updated it
Also noticed issues on missing try/catch expressions and missing info 
messages on enable/disable packages.

### What type of PR is it?
[Bug Fix | Documentation | Refactoring]

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-2699

### How should this be tested?
Run "curl -X POST http://localhost:8080/api/helium/enable/{packageName}";
Check that the package gets enabled

### Questions:
* Does the licenses files need update? N
* Is there breaking changes for older versions? N
* Does this needs documentation? Y (Done)


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/nokia/zeppelin zeppelin2699

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2446.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2446


commit a003ea2e5fa7d4ad8bd006f0c17d9901333ce875
Author: Nelson Costa 
Date:   2017-06-30T09:00:23Z

[ZEPPELIN-2699] Helium REST API bug-fixes. Also updated documentation

commit 5589a22fdfdb010e7e48495b4992ae90099c44ab
Author: Nelson Costa 
Date:   2017-07-03T21:02:32Z

[ZEPPELIN-2699] Changes after code-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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 same behaviour when this property does not exist or when it 
exists and it's set to true.

### What type of PR is it?
[ Improvement]

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-2714

### How should this be tested?
1. Open Zeppelin, go to Interpreter > Spark
1. Click spark ui > msg: "No spark application running"
1. Go to a paragraph, run "sc.parallelize(1 to 100).count()", check "SPARK 
JOB" button
1. Go to Interpreter > Spark > Set "spark.ui.enabled" to "false"
1. Rerun paragraph > "SPARK JOB" button not visible
1. Go to Interpreter > Spark > "spark ui" button not visible
Also, test with "spark.ui.enabled"="true" and other workflow combinations

### Questions:
* Does the licenses files need update? N
* Is there breaking changes for older versions? N
* Does this needs documentation? N


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/nokia/zeppelin zeppelin2714

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2456.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2456


commit 4d7658f423d9e90dd50137edd2f2010952619985
Author: Nelson Costa 
Date:   2017-06-30T09:17:59Z

[ZEPPELIN-2714] Soft-code Spark UI button visualization

commit b0767c2173ae4e3529846388f9617aab291729f1
Author: Nelson Costa 
Date:   2017-07-03T20:38:01Z

[ZEPPELIN-2714] Improvements to change request




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 Zeppelin own 
format

### What type of PR is it?
Improvement | Feature

### Todos
* [x] - initial flow
* [x] - add rename tests

### What is the Jira issue?
[ZEPPELIN-2702](https://issues.apache.org/jira/browse/ZEPPELIN-2702)

### How should this be tested?
1. go create new notes or rename title of old ones
2. see the notebook directory, under [NOTE_ID]/title.zpln

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? maybe


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/khalidhuseynov/zeppelin feat/zpln-extension

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2448.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2448


commit 60b3d8bc31872d05818d28d476c83d3a3b1d775b
Author: Khalid Huseynov 
Date:   2017-06-27T12:36:27Z

read note from zpln and json files

commit 4d02c8f24fae27bfa01c330d34e0371be9d57855
Author: Khalid Huseynov 
Date:   2017-06-28T15:05:53Z

add file conversion routine

commit b469cc4003524a446488670153017c5b4f29985d
Author: Khalid Huseynov 
Date:   2017-06-28T18:14:14Z

fix create note

commit e14889c7f7446b2e4b4bf1c5f9a6fb3447bbae02
Author: Khalid Huseynov 
Date:   2017-06-28T18:14:48Z

add convert util test

commit d7ce2b38ae4b6ec27d332266822ee1e4e40bab6b
Author: Khalid Huseynov 
Date:   2017-07-02T04:20:46Z

remove version info

commit 9785855b76c702b2227afbe9c729bec1de5c6b07
Author: Khalid Huseynov 
Date:   2017-07-02T04:21:28Z

change filename max length to 100

commit 74ad5fa49b1d04dc06ba1c355c9714a5bc6db08f
Author: Khalid Huseynov 
Date:   2017-07-03T01:54:20Z

add rename method to notebook repo

commit c4d471e6f24f01d803c69f80902783654f98a645
Author: Khalid Huseynov 
Date:   2017-07-03T17:42:42Z

add rename test

commit fca6c70309cf5d425316e7aca1a61976955fabd8
Author: Khalid Huseynov 
Date:   2017-07-03T20:32:03Z

fix createNote case

commit 98918c20d9da2a6dd6836be00b1d1a283ae1fd81
Author: Khalid Huseynov 
Date:   2017-07-03T20:32:58Z

add create note-> rename test




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 
and debug, so here propose to enable "zeppelin.livy.displayAppInfo" to true as 
default.

### What type of PR is it?

Improvement

### Todos

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2716

### How should this be tested?

Manual verification.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jerryshao/zeppelin ZEPPELIN-2716

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2459.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2459


commit d28b251300c29aa4d39b3c7acef15b539e94ccf7
Author: jerryshao 
Date:   2017-07-03T06:06:16Z

Change the default value of zeppelin.livy.displayAppInfo to true




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 
and debug, so here propose to enable "zeppelin.livy.displayAppInfo" to true as 
default.

### What type of PR is it?

Improvement

### Todos

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2716

### How should this be tested?

Manual verification.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jerryshao/zeppelin ZEPPELIN-2716

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2459.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2459


commit d28b251300c29aa4d39b3c7acef15b539e94ccf7
Author: jerryshao 
Date:   2017-07-03T06:06:16Z

Change the default value of zeppelin.livy.displayAppInfo to true




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 it doesn't matter. I've gone 
through interpreter_binding_mode.md, tidying up the English some more. I'm done 
with my edits now, 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 notes. Фny user can execute and 
everyone will have their `precode`.  `set search_path='#{user}`  #{user} will 
be replaced with the username from the context


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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.

Well, multiple and pluggable communication between frontend and 
interpreters would be great and useful, but I am aware of the impact on code 
level.

If we have to move to something else than thrift, what would be the added 
value of netty (just asking).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 server and 
interpreter process, so I don't think it is necessary to use kerberos here. But 
definitely, we need to make sure this channel secure, I think both thrift and 
netty support security channel.

>> Do we target a single implementation dropping the other or something 
that can be configured with the burden to maintain multiple implementations?

We would use just one, that's why I  mention in the above comment that 
before we choose thrift or netty, we need to consider the whole picture. 

>> What about REST (for more easy and universal clients) or WebSocket (for 
push) as more alternatives?

I don't see much advantages of using websocket. Netty and thrift both could 
do the push.  And the communication channel here is one-to-one, not one-to-many.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 paragraph. e.g.

```
%spark.init

set property_1=value_1
set property_2=value_2
```
So that the note could be shared to other people, and other people can 
reproduce the note result easily. My point is that note should be 
self-contained thing that could be reproduced when it is shared to other people


However it is still very immature idea. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 hover doesn't work properly when you touch whatever `spark` or `sql` or 
`python`.

### What type of PR is it?
[Bug Fix | Improvement]

### What is the Jira issue?
* None

### How should this be tested?
This way is mobile test.
1. check your internal IP, (for example my case is 192.168.x.x)
2. run `bundle exec jekyll serve --watch --host=0.0.0.0` in your terminal
3. connect your local server(192.168.x.x:4000) and check work well

### Screenshots (if appropriate)
(Sorry, I don't know how I can record that in iphone..)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? Yes, this is


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/soralee/zeppelin gh-pate_improve_mobile

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2460.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2460


commit 2f0f6f35d6a37453a58fc43d0462da9c6eb13a48
Author: soralee 
Date:   2017-07-03T10:19:03Z

[gh-pages] fix hover event in technologies section on mobile




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.8.0
Reporter: Guillaume Van Delsen
Priority: Blocker


It's been few months now we are experiencing frequent crashes when restarting 
Zeppelin interpreter. We first thought this was due to zeppelin own spark 
interpreter so we switched to Livy one. But yet same kind of crashes. After an 
interpreter restart we are unable to run cells and the interpreter's list shows 
nothing. We have to restart Zeppelin each time.

This maybe related to https://issues.apache.org/jira/browse/ZEPPELIN-2547






--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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) ?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 target a single implementation dropping the other or something that 
can be configured with the burden to maintain multiple implementations?
- What about REST (for more easy and universal clients) or WebSocket (for 
push) as more alternatives? 



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---