[GitHub] zeppelin issue #2248: [ZEPPELIN-2396] eliminate the 'ctrl+L' shortcut keyboa...

2017-04-12 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2248
  
Awesome! It was really annoying. Let me test and comment soon.


---
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 #2248: [ZEPPELIN-2396] eliminate the 'ctrl+L' shortcut...

2017-04-12 Thread soralee
GitHub user soralee opened a pull request:

https://github.com/apache/zeppelin/pull/2248

[ZEPPELIN-2396] eliminate the 'ctrl+L' shortcut keyboard

### What is this PR for?
This PR is for that eliminate the `ctrl+L` shortcut keyboard. This function 
is what the ace editor defined.

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

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

### How should this be tested?
* Press `Ctrl + L` key and check if showing the dialog or not

### Screenshots (if appropriate)
[Before]

![image](https://cloud.githubusercontent.com/assets/8110458/24990819/8529312a-2051-11e7-8849-e00803310752.png)

[After]

![z_zeppelin-2396](https://cloud.githubusercontent.com/assets/8110458/24990966/9764e108-2052-11e7-9387-560f9d587782.gif)


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


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

$ git pull https://github.com/soralee/zeppelin ZEPPELIN-2396_ctrl_l

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

https://github.com/apache/zeppelin/pull/2248.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 #2248


commit 4e1b8465a84099efff077e6767d643ee148a4fad
Author: soralee 
Date:   2017-04-13T04:49:11Z

ZEPPELIN-2396 drop the Ctrl+L shortcut keyboard




---
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 #2247: [ZEPPELIN-2318][branch-0.7] Fix proxy configura...

2017-04-12 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request:

https://github.com/apache/zeppelin/pull/2247

[ZEPPELIN-2318][branch-0.7] Fix proxy configuration for http client of 
zeppelinhub storage layer

### What is this PR for?
this is to resolve this issue for `branch-0.7` since original PR #2198 had 
conflicts with the branch. for more details check the original pr

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

### Todos
* [x] - Task

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

### How should this be tested?
check in #2198 

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/khalidhuseynov/zeppelin 
fix/branch-0.7-ZEPPELIN-2318

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

https://github.com/apache/zeppelin/pull/2247.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 #2247


commit 6add668ff50528fc3ef92beca51c3842c7679171
Author: Khalid Huseynov 
Date:   2017-04-13T04:36:37Z

add dependency in pom, resolve coflict

commit a65f735566d644c1d0d3b6a8f4ab93423bc2560b
Author: Khalid Huseynov 
Date:   2017-03-28T12:58:59Z

add proxy client with asynclient library

commit 7f3cd5040f5b8dca85c14eb5141692c081efd68a
Author: Khalid Huseynov 
Date:   2017-03-28T13:00:21Z

jetty client relay to asyncclient when proxy on

commit 8f8109eaaa6420d3c0ff3281cab01d2f00d1fa4f
Author: Khalid Huseynov 
Date:   2017-03-28T13:01:01Z

add close routine

commit e6622398d1dcf8ae9f9f491b2d56da1dd137ea66
Author: Khalid Huseynov 
Date:   2017-03-28T15:40:03Z

add ssl setup

commit d8efb462faf6066767b21fe69ed1179921b8bac0
Author: Khalid Huseynov 
Date:   2017-03-28T16:13:09Z

fix websocket timing

commit e4db79ca0608e6e2f86b2598b8440770fd7006fc
Author: Khalid Huseynov 
Date:   2017-03-28T16:36:21Z

fix log

commit ec23c958a03a6ada958f263b19db18f98a120cdf
Author: Khalid Huseynov 
Date:   2017-03-29T07:54:38Z

edge case logs from error -> warn




---
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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
hmm, right. Seems document is not needed 😄 


---
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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
@zjffdu 
How to use is the same as the existing one.
If I add a document, should I add something like __zeppelin__ or __spark__?
What do you think? :)


---
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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
@cloverhearts  I left one minor comment. Besides,  we need to add document 
for 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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
I would like to merge this.
This solves the problem of usability.
I will merge this if I no longer have any opinion.
Thank you :)


---
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 #2198: [ZEPPELIN-2318] Fix proxy configuration for htt...

2017-04-12 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2198


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


Re: shiro.ini : Zeppelin interpreter authorization: lock to one user

2017-04-12 Thread Arun Khetarpal
Hi -

I think the ask is that currently it seems that activeDirectoryGroupRealm
maps each active directory group to a shiro role. Later we can authorize
urls with roles in our urls section of shiro file.

Our problem is that we don't really have an admin group handy in our AD -
due to which there is a bit of a problem in setting the correct
authorization of some urls. We were able to play around with
ActiveDirectoryGroupRealm and modify in such a way that an internal role is
created for systemUser and then authorize urls for him based on the
internal role.

Later we placed this modified ActiveDirectoryGroupRealm in Class path and
everything works as expected. So my question is -

a. Is there a better way provided by zeppelin for users to provide their
own implementation of ActiveDirectoryGroupRealm ?
b. Is the community interested in modifying ActiveDirectoryGroupRealm with
our tweaks ?


Regards,
Arun




On 12 April 2017 at 15:55, Vartul Gupta 
wrote:

> Hi Team,
>
> I am trying to setup authentication and authorization for Zeppelin using
> Shiro file.
> Now my question is How we can set Zeppelin interpreter settings to be
> restricted to a particular AD admin user.
>
> Here is the sample of my shiro file .
>
> [main]
> activeDirectoryRealm = org.apache.zeppelin.realm.ActiveDirectoryGroupRealm
> activeDirectoryRealm.systemUsername = adminuser1
> activeDirectoryRealm.systemPassword = ^^
> #activeDirectoryRealm.hadoopSecurityCredentialPath =
> jceks://user/zeppelin/zeppelin.jceks
> activeDirectoryRealm.searchBase = "OU=AADDC 
> Users,DC=team2testdomain,DC=testabc
> ,DC=com"
> activeDirectoryRealm.url = ldaps://team2testdomain.testabc.com:636
> activeDirectoryRealm.principalSuffix = @TEAM2TESTDOMAIN.testabc.COM
> activeDirectoryRealm.groupRolesMap = "OU=AADDC
> Users,DC=team2testdomain,DC= testabc,DC=com":"admin"
> activeDirectoryRealm.authorizationCachingEnabled = true
> sessionManager = org.apache.shiro.web.session.mgt.DefaultWebSessionManager
> cacheManager = org.apache.shiro.cache.MemoryConstrainedCacheManager
> securityManager.cacheManager = $cacheManager
> securityManager.sessionManager = $sessionManager
> # 86,400,000 milliseconds = 24 hour
> securityManager.sessionManager.globalSessionTimeout = 8640
> shiro.loginUrl = /api/login
>
> [urls]
> # anon means the access is anonymous.
> # authcBasic means Basic Auth Security
> # To enfore security, comment the line below and uncomment the next one
> #/** = anon
> /** = authc
>
> Thanks
> Vartul
>


Re: Visualization class of Helium: Which method should I must override?

2017-04-12 Thread Park Hoon
Hi, Jun.

- every class which extending visualization should override `render` method.
- and should have `this.transformation` which is used to convert table data.

currently, 3 transformations are available.

-
https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/tabledata/columnselector.js
-
https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/tabledata/pivot.js
-
https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/tabledata/passthrough.js

and here are few examples.

- (Pivot)
https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/visualization/builtins/visualization-linechart.js
- (Columnselector) https://github.com/1ambda/zeppelin-highcharts-heatmap
- (Columnselector)
https://github.com/1ambda/zeppelin-highcharts-bubble/blob/master/index.js

since PassthroughTransformation is not that useful, you can ignore.

Thanks.


On Tue, Apr 11, 2017 at 2:37 AM, Jun Kim  wrote:

> Hi, dear devs!
>
> Currently, I'm developing a Helium visualization package.
>
> But, class Visualization
>  web/src/app/visualization/visualization.js>
> is
> not clear about abstract methods.
> It seems like I should override render, but some of them are optional (e.g.
> refresh, destroy etc.)
>
> So, If you let me know which method is mandatory and which one is optional,
> I would like to create an issue :)
> A mandatory method should throw an error if not implemented. (Like
> SpellBase
>  web/src/app/spell/spell-base.js>
> !)
>
> Thanks,
> --
> Taejun Kim
>
> Data Mining Lab.
> School of Electrical and Computer Engineering
> University of Seoul
>


[GitHub] zeppelin pull request #2196: fix bugs for timer saver

2017-04-12 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2196


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


removing variables from context

2017-04-12 Thread Иван Шаповалов
All,

Could anyone please point to some mechanisms to remove datasets /
dataframes from the current interpreter context?

Thanks in advance

-- 
Ivan Shapovalov
Kharkov, Ukraine


[GitHub] zeppelin issue #2237: ZEPPELIN-2377. Hive Support can not be enabled in spar...

2017-04-12 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2237
  
Thanks @felixcheung for review, will merge it into master 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.
---


[GitHub] zeppelin issue #2241: ZEPPELIN-2390. Improve returnType for z.checkbox

2017-04-12 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2241
  
Thanks for review, will merge to master 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.
---


[GitHub] zeppelin issue #2229: [Zeppelin 2367] Hive JDBC proxy user option should be ...

2017-04-12 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2229
  
@AhyoungRyu, @felixcheung ping


---
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 #2237: ZEPPELIN-2377. Hive Support can not be enabled in spar...

2017-04-12 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/2237
  
I don't disagree - we have other dependencies on private methods though?

Ok I'm fine with this approach. Thanks for the explanation.




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


Start the server in remote debugging mode

2017-04-12 Thread Minudika Malshan
Hi,

I am trying to start the zeppelin server in remote debugging mode.
I have already entered the entry *"export ZEPPELIN_MEM="-Xdebug -Xnoagent
-Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=8111"* to
*zeppelin-env.sh* and configured the InteliJ Idea for remote debugging.

But still there is a message mentioning that connection to the port 8111 is
refused.

Is it enough to run the server by ./zeppelin.sh or are there any more
parameters to be added to it?

Thanks and regards!

-- 
*Minudika Malshan*
Undergraduate
Department of Computer Science and Engineering
University of Moratuwa
Sri Lanka.


[jira] [Created] (ZEPPELIN-2398) comments in %pyspark mode do not highlight correctly as e.g. green

2017-04-12 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2398:


 Summary: comments in %pyspark mode do not highlight correctly as 
e.g. green
 Key: ZEPPELIN-2398
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2398
 Project: Zeppelin
  Issue Type: Bug
Reporter: Nassir


When in %pyspark mode within a cell comments in %pyspark mode do not highlight 
correctly as e.g. green



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2221: [ZEPPELIN-2216] general solution to precode. refactori...

2017-04-12 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue:

https://github.com/apache/zeppelin/pull/2221
  
@Leemoonsoo 
prevous discussion #2096 


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


shiro.ini : Zeppelin interpreter authorization: lock to one user

2017-04-12 Thread Vartul Gupta
Hi Team,

I am trying to setup authentication and authorization for Zeppelin using Shiro 
file.
Now my question is How we can set Zeppelin interpreter settings to be 
restricted to a particular AD admin user.

Here is the sample of my shiro file .

[main]
activeDirectoryRealm = org.apache.zeppelin.realm.ActiveDirectoryGroupRealm
activeDirectoryRealm.systemUsername = adminuser1
activeDirectoryRealm.systemPassword = ^^
#activeDirectoryRealm.hadoopSecurityCredentialPath = 
jceks://user/zeppelin/zeppelin.jceks
activeDirectoryRealm.searchBase = "OU=AADDC Users,DC=team2testdomain,DC=testabc 
,DC=com"
activeDirectoryRealm.url = ldaps://team2testdomain.testabc.com:636
activeDirectoryRealm.principalSuffix = @TEAM2TESTDOMAIN.testabc.COM
activeDirectoryRealm.groupRolesMap = "OU=AADDC Users,DC=team2testdomain,DC= 
testabc,DC=com":"admin"
activeDirectoryRealm.authorizationCachingEnabled = true
sessionManager = org.apache.shiro.web.session.mgt.DefaultWebSessionManager
cacheManager = org.apache.shiro.cache.MemoryConstrainedCacheManager
securityManager.cacheManager = $cacheManager
securityManager.sessionManager = $sessionManager
# 86,400,000 milliseconds = 24 hour
securityManager.sessionManager.globalSessionTimeout = 8640
shiro.loginUrl = /api/login

[urls]
# anon means the access is anonymous.
# authcBasic means Basic Auth Security
# To enfore security, comment the line below and uncomment the next one
#/** = anon
/** = authc

Thanks
Vartul


[GitHub] zeppelin pull request #2246: [ZEPPELIN-1828] Flaky Test: RemoteInterpreterTe...

2017-04-12 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request:

https://github.com/apache/zeppelin/pull/2246

[ZEPPELIN-1828] Flaky Test: RemoteInterpreterTest hanged in 
zeppelin-interpreter

### What is this PR for?
When `interpreter.close()` occurs in code it shuts down 
_RemoteInterpreterEventPoller_ and then tries to close thrift connection. While 
closing thrift client will wait until events queue is empty. If 
_RemoteInterpreterEventPoller_ has not processed all thrift events before the 
shutdown signal comes, thrift client will wait forever. As a result Travis will 
terminate the build after code is not responding for 10 minutes.

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

### Todos
We need to:
* [ ] - write a test that checks that event queue is empty after 
_RemoteInterpreterEventPoller_ is done,
* [ ] - clear unread events on _RemoteInterpreterEventPoller_ shutdown 
signal.

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

### How should this be tested?
This issue can be reprodused if you put `Thread.sleep(3000L);` to  
_RemoteInterpreterEventPoller.java_ 
```
...
} else if (event.getType() == RemoteInterpreterEventType.OUTPUT_UPDATE_ALL) 
{
Thread.sleep(3000L);
...
}
```
and run _RemoteInterpreterTest.java_.

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


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

$ git pull https://github.com/AlexanderShoshin/zeppelin ZEPPELIN-1828

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

https://github.com/apache/zeppelin/pull/2246.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 #2246


commit 66fe9c7c480f4dfdb5e7e00eff7198b99e2d3bac
Author: Alexander Shoshin 
Date:   2017-04-12T08:03:54Z

test that EventPoller clear unread events on shutdown

commit 5818a83d638fdf77f0b6f146f669ad523bb0e10c
Author: Alexander Shoshin 
Date:   2017-04-12T08:04:31Z

clear unread events on shutdown




---
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-2397) Add toJson and fromJson method for serializable objects

2017-04-12 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2397:


 Summary: Add toJson and fromJson method for serializable objects
 Key: ZEPPELIN-2397
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2397
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.1
Reporter: Jeff Zhang
Assignee: Jeff Zhang


Instead of do serialize/deserialize out of these objects. These object should 
just provide wrap serialize/deserialize in itself, provide toJson and fromJson 
method instead. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin pull request #2230: ZEPPELIN-2366 In zeppelin SystemUser fails to a...

2017-04-12 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2230


---
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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
Okay, Ci pass :)
@felixcheung @zjffdu 
Could you please check on this pr?
Thank you :)


---
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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
and tested environment.


| | pyspark intp | python intp |
|-||--|
|python3 with matplotlib | O| O |
|python2 with matplotlib | O | O |



---
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 #2207: [Zeppelin-802] Support for Zeppelin Context redefiniti...

2017-04-12 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2207
  
apply to namespace on python

and zeppelin context name change

before
```
z = _zc = zeppelin context
```
to
```
z = __zeppelin__ = zeppelin context
```

same
```
z.input("title", "my title")
__zeppelin__.input("title", "my title")
```


---
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 #2228: [ZEPPELIN-2365] button to create note into folder

2017-04-12 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2228
  
Tested and it's work well, 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.
---


[jira] [Created] (ZEPPELIN-2396) Ctrl+L; this shortcut key is not necessary.

2017-04-12 Thread Sora Lee (JIRA)
Sora Lee created ZEPPELIN-2396:
--

 Summary: Ctrl+L; this shortcut key is not necessary.
 Key: ZEPPELIN-2396
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2396
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Sora Lee
Assignee: Sora Lee
 Attachments: screenshot-1.png

We don't need {{Ctrl+L}} shortcut key it is not necessary.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)