[GitHub] zeppelin issue #993: [ZEPPELIN-987] Enable user to secure interpreter settin...

2016-06-14 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/993
  
@AhyoungRyu  is this still a work in progress ? 
Is there a way in which we can go away with seeing all of these three munu 
i.e. Interpreter setting, Credentials and Configurations if user is not 
authorize  to ?

Otherwise 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 issue #1014: Remove Incubator and fix typos in README

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1014
  
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 issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
I think this kind of menu should be under a setting icon instead.
And keep the admin dropdown element only for logout if there is auth



---
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 #979: Alert message when trying to delete last paragraph.

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
Merging if there's no more 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 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 #1013: [ZEPPELIN-1002] Move configuration menus under ...

2016-06-14 Thread minahlee
GitHub user minahlee opened a pull request:

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

[ZEPPELIN-1002] Move configuration menus under dropdown

### What is this PR for?
- Move configuration menus under dropdown menu
- Change dropdown menu style
- Change `Login` button style

### What type of PR is it?
Improvement

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

### Screenshots (if appropriate)
**Before** (as anonymous)
https://cloud.githubusercontent.com/assets/8503346/16068223/19a73576-3279-11e6-81bd-04b4277ccc50.png;>

**After** (as anonymous)
https://cloud.githubusercontent.com/assets/8503346/16068229/2a1a448e-3279-11e6-9e45-f4c63cfe798c.png;>

**Before** (as auth user)
https://cloud.githubusercontent.com/assets/8503346/16068316/ead6d480-3279-11e6-82c2-6e6ede37538a.png;>

**After** (as auth user)
https://cloud.githubusercontent.com/assets/8503346/16068319/ee0600c2-3279-11e6-8dc0-b8c94a18d4ab.png;>

**Before** (on mouse over)
https://cloud.githubusercontent.com/assets/8503346/16068244/4c95f79c-3279-11e6-976c-69ba8bb63f26.png;>

**After** (on mouse over)
https://cloud.githubusercontent.com/assets/8503346/16068245/4f267266-3279-11e6-8483-b5715fc9ec3e.png;>

**Before** (login button style)
https://cloud.githubusercontent.com/assets/8503346/16068291/ba3d57a4-3279-11e6-81b8-0492c1265fd4.png;>

**After** (login button style)
https://cloud.githubusercontent.com/assets/8503346/16068270/9bb2406a-3279-11e6-9f9f-25126410dc47.png;>

### 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/minahlee/zeppelin ZEPPELIN-1002

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

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


commit 4d3f5948e4fd7d0055dbbd8ae2af99854fb77af6
Author: Mina Lee 
Date:   2016-06-15T04:34:28Z

Move interpreter, credential, configuration to dropdown menu

commit 378d83183c6334775bf455e1e02ff087dbd24ff3
Author: Mina Lee 
Date:   2016-06-15T04:34:46Z

Fix integration test caused by interprter menu location change




---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-14 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/976
  
LGTM. 
@minahlee The selenium test of travis has a problem so @corneadoug is 
fixing 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 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 #1000: typo

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1000
  
Last commit needs to be removed by using git interactive rebase, as it's 
been taken care of under #1009 

It would also be great to have more descriptive PR title like `Fix typo in 
bin/zeppelin.sh`


---
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: Problem on importing Notebook

2016-06-14 Thread Hyung Sung Shim
Hi.
It's a bug and is fixing in [1] PR.

[1]
https://github.com/apache/zeppelin/pull/933

2016-06-15 1:39 GMT+09:00 Andrea Santurbano :

> I have cloned Zeppelin from master branch right now and the import
> mechanism doesn't work.
> I have tried to import the Zeppelin Tutorial notebook after having download
> it and but it seems not work.
> I miss something?
> Is this a bug?
>


[GitHub] zeppelin issue #1007: Update and refactor NotebookRepo versioning API

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1007
  
Thank you for addressing feedback promptly.

Well, in my oppinion, if by "encourage more versioned implementation" you 
mean having this code duplicated 5 times around the code base
```java
  @Override
  public Note get(String noteId, Revision rev) throws IOException {
// Auto-generated method stub
return null;
  }
```
it looks like not very elegant solution. 

It looks more meaningful and elegant to having 2 separate interfaces 
(types) documented, 1 for non-versioned backends and 1 for versioned ones to 
allow authors to choose from.

Otherwise, if we extend the suggested logic - why do not just to type 
everything as `Object` and keep only 1 interface for everything? It kind of 
defeats the benefits of having a type system.

It's not a strong opinion though, and if after this arguments, you still 
feel that having single one is better - please let me know.

Other that this issues - it looks great to me.


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-14 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
@corneadoug I really appreciate for your effort. Thank you. I reset my 
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 issue #1002: ZEPPELIN-992 Move some tests from InterpreterFactoryTe...

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1002
  
Looks great to me and CI passes 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 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 #1003: [DOCS] Fix link to specific doc from release pa...

2016-06-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1009: Prevent NullPointerException if "gatewayServer" does n...

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1009
  
Thank you for the prompt responce!

CI passes now and fix solves the issue and looks great to me.

Merging if there is no further 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 #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1012
  
Thank you @jongyoul, 👍  for having tests!
Improving this further by generalizing to other interpreters after a 
release sounds like a very good plan.

Looks great to me.


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
Tested Selenium Tests locally from original commit:

https://github.com/apache/zeppelin/pull/979/commits/6f181efa09f4d616ac66eec5641df954883af432

And it's working well.

I will create a flacky test issue on JIRA,
@astroshim Could you rewrite history back to the commit I just Mentioned? 
After that 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 issue #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-14 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/976
  
@minahlee it's not my issue.
I think the problem seems to arise from a sudden UI test master branch.
Let's solve the problem from a different PR.
One will stop working on this feature.


---
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 #1006: [MINOR] Small ui modification in notebook actionbar

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1006
  
Merging if there is no more 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 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-1008) Improve Interpreter binding dialog

2016-06-14 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1008:
--

 Summary: Improve Interpreter binding dialog
 Key: ZEPPELIN-1008
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1008
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: Lee moon soo
Priority: Minor


Currently, "interpreter binding" dialog lists available interpreter settings.
User can enable / disable them and drag to the top, to make it default. 
(screenshot attached)

I think we can think better UX for interpreter binding dialog by considering,

- Expose interpreter restart button for convenience
- Instead of drag and drop to make default, we can just think about radio box 
or something more intuitive. Maybe with ZEPPELIN-707 default interpreter can 
fade away



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (ZEPPELIN-1007) Prevent Interpreter binding and Note Permissions dialog opens at the same time

2016-06-14 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1007:
--

 Summary: Prevent Interpreter binding and Note Permissions dialog 
opens at the same time
 Key: ZEPPELIN-1007
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1007
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: Lee moon soo


"Interpreter binding" and "Note Permission" dialog opens at the same time.
it's bit weird see two dialogs open one after another.





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #999: Fix navigation link of documentation for storage using ...

2016-06-14 Thread yoonjs2
Github user yoonjs2 commented on the issue:

https://github.com/apache/zeppelin/pull/999
  
@minahlee No problem :)


---
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 #1006: [MINOR] Small ui modification in notebook actionbar

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1006
  
I made one more change in table ascending, descending icon. Please 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 #918: ZEPPELIN-885 Add option to send email in scheduler exec...

2016-06-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/918
  
@vgmartinez Thanks for update.

I tried and found 'Cancel' button is not actually discarding chances in 
dialog. Check following screenshot

![scheduler](https://cloud.githubusercontent.com/assets/1540981/16062253/ce3390a8-3246-11e6-9784-07b99fc5149e.gif)



---
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 #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-14 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1012
  
Nice! Can this be generalized to not just jdbc?


---
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 #981: ZEPPELIN-925 Part 2. Merge Hive Interpreter into...

2016-06-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-1006) Scalding documentation update

2016-06-14 Thread Prasad Wagle (JIRA)
Prasad Wagle created ZEPPELIN-1006:
--

 Summary: Scalding documentation update
 Key: ZEPPELIN-1006
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1006
 Project: Zeppelin
  Issue Type: Bug
  Components: documentation
Reporter: Prasad Wagle
Assignee: Prasad Wagle
Priority: Trivial






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (ZEPPELIN-1005) Reorder paragraph with Drag & Drop

2016-06-14 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1005:
--

 Summary: Reorder paragraph with Drag & Drop
 Key: ZEPPELIN-1005
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1005
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: Lee moon soo


Check Drag & Drop section of 
https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+UX+Roadmap

Allow users to easily reorder notebook paragraphs, by introducing a drag and 
drop feature



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (ZEPPELIN-1004) Improve UX

2016-06-14 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1004:
--

 Summary: Improve UX
 Key: ZEPPELIN-1004
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1004
 Project: Zeppelin
  Issue Type: Improvement
  Components: GUI
Reporter: Lee moon soo


This is umbrella issue that taking care of various UX improvements suggested in 
 https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+UX+Roadmap 

and discussed at 
http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Zeppelin-UX-Design-Roadmap-tp2801.html



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #999: Fix navigation link of documentation for storage using ...

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/999
  
I just tried #1004 and it scroll to expected position. Since #1004 is using 
a lot of anchor, would you mind to close this pr so we can reduce burden of 
changing every anchor that is used in #1004?


---
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 #1006: [MINOR] Small ui modification in notebook actionbar

2016-06-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1006
  
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 #1007: Update and refactor NotebookRepo versioning API

2016-06-14 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at:

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


---
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 #986: [Zeppelin 946] Permissions not honoring group

2016-06-14 Thread rja1
Github user rja1 commented on the issue:

https://github.com/apache/zeppelin/pull/986
  
Thanks once again @prabhjyotsingh.  I really appreciate your work. The 
activeDirectoryRealm.principalSuffix works now.  I do have some concerns about 
the number of ldap calls made and the amount of data pulled back.

 It looks like the app:
1) makes ldap bindRequest as n...@mydomain.com and fails.
2) makes ldap bindRequest as usern...@mydomain.com and succeeds.
3) makes a bindRequest as activedirectoryrealm.systemusern...@mydomain.com 
and succeeds
4) does a searchRequest for the wholeSubTree

Step 4 pulls back about 5mb of data, which is a lot.  Could add quite a lot 
of load to AD, if lots of users are simultaneously using the UI...  I can limit 
the result set by more fully qualifying the activeDirectoryRealm.searchBase, 
however, then it seems to miss the group data.  Seems like there should really 
just be only be a couple lightweight calls.
1) bind the username
2) pull back the group memberships for username (if step one was a success).

Not sure if there's a more concise way to make these queries in java.  I 
can do it via command line the following way: ldapsearch -xLLL -h ldapServer -b 
"dc=company,dc=com" -D "CN=LDAP Bind,OU=Special,Accounts,DC=company,DC=com" -W 
uid=randerson.  This returns everything about the uid: randerson, including all 
group memberships.  The total size of the data is 60k... 

In addition, my groups / roles are still not mapped to my username, 
regardless of if the app searches the whole tree or not.  I'm not sure why.  
Perhaps I've missed something along the way.  

Here's my shiro.ini:

[main]
activeDirectoryRealm = org.apache.zeppelin.server.ActiveDirectoryGroupRealm
activeDirectoryRealm.systemUsername = username
activeDirectoryRealm.systemPassword = password
activeDirectoryRealm.searchBase = dc=company,dc=com
activeDirectoryRealm.url = ldap://server:389
activeDirectoryRealm.groupRolesMap = "cn=g.acl.ops.bigdata,ou=unix 
groups,ou=groups,ou=accounts,cn=users,dc=company,dc=com":"admin"
activeDirectoryRealm.authorizationCachingEnabled=false
activeDirectoryRealm.principalSuffix=@DOMAIN.COM
shiro.loginUrl = /api/login
sessionManager = org.apache.shiro.web.session.mgt.DefaultWebSessionManager
securityManager.sessionManager = $sessionManager
securityManager.sessionManager.globalSessionTimeout = 8640
shiro.loginUrl = /api/login

[roles]
admin = *

[urls]
/api/version = anon
/** = authc


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


Zeppelin Meetup - Hortonworks BLR

2016-06-14 Thread rohit choudhary
Hi All,

We have some confirmations from community members and companies from
Bangalore. Please do RSVP, if anyone from the community would like to join
us for the meetup. Here are the details:

*Venue :*

Hortonworks India, #92, Z Square, Jyoti Nivas College Road, Koramangla.
https://goo.gl/maps/Anb3Kp1tcuy

*Timing: *3:00pm onwards.
Please let me know should you need any assistance in getting to the Venue.

Thanks,
Rohit.


On Thu, Jun 2, 2016 at 3:03 PM, rohit choudhary  wrote:

> Hi All,
>
> We're planning to have a Zeppelin meetup on Thursday, 16th, at Hortonworks
> Bangalore. It will get great if interested folks could send me confirmation
> of their interest in participating. Please let me know if there are any
> questions. Exatct logistics are being worked out, will get back with more
> details soon.
>
> Thanks,
> Rohit
>


[jira] [Created] (ZEPPELIN-1002) Move configuration menus under dropdown

2016-06-14 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1002:
--

 Summary: Move configuration menus under dropdown
 Key: ZEPPELIN-1002
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1002
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Mina Lee


There is no more space for new menu in nav bar.
Since configuration menus are not used as much as working menu(like notebook, 
job), I suggest to move interpreter, credential, configuration menu under 
dropdown on the rightside



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1005
  
@jongyoul Right, it's not dynamically configurable. Interpreter need to be 
restarted to reconfigure. 

I think it's possibly related to ZEPPELIN-999 and it's long term plan. 
Let's say a user want to use jdbc interpreter for hive and mysql. Currently, 

A. user can create single interpreter setting and create two configuration 
for both hive and mysql connection. And select connection via %jdbc(hive) or 
%jdbc(mysql).
B. or user can create two interpreter setting and each interpreter setting 
have configuration for hive and mysql connection respectively. In this case, 
select jdbc connection limited and annoying, because a notebook can not use two 
or more same type of interpreter settings at the same time. So user have to 
constantly bind/unbind interpreter setting to switch hive <-> mysql.

If our long term plan is generalize ZEPPELIN-999 and allow user use 
interpreter setting alias for interpreter selection, like %hive, %mysql, then 
all disadvantages of approach **B** will be eliminated.

Managing only single jdbc connection in single interpreter setting, with 
generalized ZEPPELIN-999 will give advantages, like leverage [interpreter 
authorization](https://issues.apache.org/jira/browse/ZEPPELIN-945). And in this 
case, i was thinking simple returning appropriate scheduler from getScheduler() 
by configuration would be enough.


---
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 #1004: [ZEPPELIN-996] Improve first page and dropdown menu in...

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1004
  
https://cloud.githubusercontent.com/assets/8503346/16052985/ad3fa0ec-321b-11e6-9f15-e9725cee700f.png;>

This is what I see from my screen, and cannot scroll down lower. Can you 
adjust size of the dropdown menu?

And also, can we rollback the change of display system menu order to text 
-> html -> table? if there are a lot of menu then alphabetical order makes 
sense, but in this case I think better to display in order of basic to advanced

Other than above comments, this PR looks awesome!


---
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 #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1005
  
@Leemoonsoo I've missed something. I've known it was not configurable 
dynamically. Let me check.


---
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 #981: ZEPPELIN-925 Part 2. Merge Hive Interpreter into JDBC

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/981
  
LGTM, could you rebase? I merged #1011 without knowing that you took care 
of hive license 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 pull request #997: [MINOR] Make nav zeppelin version to point ZEPPE...

2016-06-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1004: [ZEPPELIN-996] Improve first page and dropdown menu in...

2016-06-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1004
  
@bzz Thanks for your prudent review! I just addressed them in 
[6a2f40b](https://github.com/apache/zeppelin/pull/1004/commits/6a2f40b73e65986bdd61b611c76243f7a8bd7916).
 


---
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 #1011: Update licenses

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1011
  
Merging it into master


---
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 #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1005
  
I was thinking just creating multiple interpreter settings if user want to 
use different scheduler. Then just returning appropriate scheduler from 
getScheduler() by configuration would be enough. Isn't it?


---
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 #1011: Update licenses

2016-06-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1011
  
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 #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-14 Thread jongyoul
GitHub user jongyoul opened a pull request:

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

ZEPPELIN-999 Support alias for JDBC properties

### What is this PR for?
In case of using JdbcInterpreter, you should use %jdbc(prefix) if you set 
multiple configurations. This PR makes you use %prefix only. 

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

### Todos
* [x] - Change %prefix to %jdbc(prefix) during running paragraph

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

### How should this be tested?

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/3612566/16049304/25db79f6-3292-11e6-876a-287bbbc50f50.png;>
https://cloud.githubusercontent.com/assets/3612566/16049313/31c2097e-3292-11e6-8c91-13d71360f25f.png;>

### 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/jongyoul/zeppelin ZEPPELIN-999

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

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


commit 4ca7d81285a51fd7641695af53d3e5e39c49e1d1
Author: Jongyoul Lee 
Date:   2016-06-14T11:16:23Z

Added logic to change from %property to %jdbc(property)

commit 37c48105db8df2ab6977b036b2280491d83cfefc
Author: Jongyoul Lee 
Date:   2016-06-14T15:20:28Z

- Fixed some exception to returning null
- Added effective text to interpret it actually
- Made ZeppelinConfiguration transient

commit 6d0293fbfed282d7590d72326b0f9fadba0a670e
Author: Jongyoul Lee 
Date:   2016-06-14T15:38:52Z

- Added some test cases




---
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 #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-14 Thread jongyoul
Github user jongyoul closed the pull request at:

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


---
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 #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-14 Thread jongyoul
GitHub user jongyoul reopened a pull request:

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

ZEPPELIN-934 Merge Phoenix interpreter into JDBC interpreter

### What is this PR for?
Removing Phoenix interpreter and adding example for using it through JDBC

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

### Todos
* [x] - Merge it into JDBC

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

### How should this be tested?

### 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/jongyoul/zeppelin ZEPPELIN-934

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

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


commit 7b094cc6f961afe138083e960906e071b7e418d8
Author: Jongyoul Lee 
Date:   2016-06-09T15:48:29Z

- Removed Phoenix Interpreter
- Updated example for using Apache Phoenix




---
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 #928: [ZEPPELIN-116][WIP] Add Mahout Support for Spark Interp...

2016-06-14 Thread rawkintrevo
Github user rawkintrevo commented on the issue:

https://github.com/apache/zeppelin/pull/928
  
If someone could help me out I'd appreciate it...

First of all, this works fine as expected in the notebooks (either way). 

In MahoutSparkInterpreter.java line 89, there is a jar I can load. 

If I don't load it, on testing only, I get the following error:
```
No Import : org.apache.mahout:mahout-spark-shell_2.10:0.12.1
:35: error: type mismatch;
 found   : org.apache.spark.SparkContext
 required: org.apache.mahout.sparkbindings.SparkDistributedContext
   implicit val sdc: SparkDistributedContext = sc
   ^
```

If I do load it, again in testing only, I get the following errors 
(truncated):
```
With org.apache.mahout:mahout-spark-shell_2.10:0.12.1:

== Enclosing template or block ==

Import( // val : ImportType(org.apache.mahout.sparkbindings)
  "org"."apache"."mahout"."sparkbindings" // final package sparkbindings in 
package mahout, tree.tpe=org.apache.mahout.sparkbindings.type
  List(
ImportSelector(_,1227,null,-1)
  )
)

uncaught exception during compilation: scala.reflect.internal.FatalError
:4: error: illegal inheritance;
 self-type line2125238280$23.$read does not conform to scala.Serializable's 
selftype Serializable
class $read extends Serializable {
^
:4: error: Serializable does not have a constructor
class $read extends Serializable {
^
:6: error: illegal inheritance;
 self-type $read.this.$iwC does not conform to scala.Serializable's 
selftype Serializable
  class $iwC extends Serializable {
 ^
:6: error: Serializable does not have a constructor
  class $iwC extends Serializable {
 ^
:11: error: illegal inheritance;
 self-type $iwC does not conform to scala.Serializable's selftype 
Serializable
class $iwC extends Serializable {
   ^
...
```

Since this works in the actual notebooks, I'm not sure what I'm doing wrong 
in setting up the testing env.  Any advice would be appreciated.

Thanks,

tg



---
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 #978: [ZEPPELIN-869] Search notebook to handle empty result.

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/978
  
Merging if there is no more 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 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 #1011: Update licenses

2016-06-14 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1011
  
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 #953: ZEPPELIN-947: Change docs link to point the same...

2016-06-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

2016-06-14 Thread cloverhearts
GitHub user cloverhearts reopened a pull request:

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

ZEPPLIN-976 ] HotFix - zeppelin server does not restart when incorrect 
credentials data.

### What is this PR 
zeppelin server does not restart when incorrect credentials data.

reproduce.
1. Click to zeppelin home for web ui.
2. Click to zeppelin Credentials.
3. 'Entity' information without writing, username and password only written 
to storage.
4. Click to zeppelin home for web ui.
5. Click to zeppelin Credentials.
and zeppelin restart.
# but, Zeppelin does not work.

It creates the wrong json file. 'conf / credentials.json' according to the 
null.

### What type of PR is it?
Hot Fix

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

### How should this be tested?

1. Click to zeppelin home for web ui.
2. Click to zeppelin Credentials.
3. 'Entity' information without writing, username and password only written 
to storage.
4. Click to zeppelin home for web ui.
5. Click to zeppelin Credentials.
and zeppelin restart.

zeppelin does work!

### Reproduced Screenshots

![zeppelin-server-error2](https://cloud.githubusercontent.com/assets/10525473/15889828/b92590d8-2da7-11e6-9b51-0a82c3bb9f1f.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/cloverhearts/zeppelin 
hotfix/CredentialsJsonBug

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

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


commit 1e5cd727c499ea1b700c1d9df81dd7aa69fa4fc9
Author: CloverHearts 
Date:   2016-06-08T09:12:39Z

Credentials Json serialize backend bug.

commit 4b9aba3455a5f441b17c41d6335a0222c1fb59a6
Author: CloverHearts 
Date:   2016-06-08T11:20:36Z

changed status code for CredentialsRestapi

commit e9a1e93289ec8802ec4fa1e96afb16e986d07629
Author: CloverHearts 
Date:   2016-06-14T05:08:47Z

Merge branch 'master' into hotfix/CredentialsJsonBug

commit 5079495feb1a3786127e9f1c484aabcfc3476817
Author: CloverHearts 
Date:   2016-06-14T05:50:30Z

add test code for credentials backends about invalid 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 issue #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-14 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/976
  
retry ci


---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

2016-06-14 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


---
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 #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1005
  
Looks great, thank you for prompt update!

@jongyoul is there a reason to make it configurable from another PR instead 
of this one? Just curious  


---
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 #1007: Update and refactor NotebookRepo versioning API

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1007
  
Great improvement! 

Could you please explain the rationale behind removing 
`NotebookRepoVersioned` and making a lot of boilerplate methods returning 
`null` in all other notebook storages that do not support versioning?


---
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 #1009: Prevent NullPointerException if "gatewayServer" does n...

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1009
  
@OutOfBedlam thank you for catching this and contributing the improvement!

Before merging, could you please check that your code follows [project code 
style 
guide](https://github.com/apache/zeppelin/blob/master/CONTRIBUTING.md#code-convention)
 esp regardin [optional braces in 
Java](https://google.github.io/styleguide/javaguide.html#s4.1.1-braces-always-used)?
 

CI failure seems un-related

```
Running org.apache.zeppelin.rest.ZeppelinSparkClusterTest
...
01:44:57,921  INFO 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterProcess:143 - Run 
interpreter process [..//bin/interpreter.sh, -d, ../interpreter/spark, -p, 
45995, -l, ..//local-repo/2BMVKYY9A]
..//bin/interpreter.sh: line 159: 
/home/travis/build/apache/zeppelin/run/zeppelin-interpreter-spark-travis-testing-worker-linux-docker-b1d10849-3384-linux-9.pid:
 No such file or directory
...
1:46:16,709  INFO org.apache.zeppelin.socket.NotebookServer:1116 - Job 
20160614-014616_147367 is finished
Spark version detected 1.6.1
01:46:16,728  INFO org.apache.zeppelin.notebook.Paragraph:241 - run 
paragraph 20160614-014616_1069054480 using r 
org.apache.zeppelin.interpreter.LazyOpenInterpreter@3af54c62


No output has been received in the last 10 minutes, this potentially 
indicates a stalled build or something wrong with the build itself.

The build has been terminated

```


---
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 #1011: Update licenses

2016-06-14 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1011
  
Thanks for catching it, looks great to me


---
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 #1008: [ZEPPELIN-997] Export data to csv

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1008
  
Unrelated CI failure (timeout)
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 issue #998: Authenticated user aware notebook storage layer

2016-06-14 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/998
  
somehow selenium tests keep error with 
```
Tests in error: 
  
ParagraphActionsIT.testRemoveButton:145->AbstractZeppelinIT.waitForParagraph:84->AbstractZeppelinIT.pollingWait:110
 » Timeout
  
ParagraphActionsIT.testMoveUpAndDown:182->AbstractZeppelinIT.waitForParagraph:84->AbstractZeppelinIT.pollingWait:110
 » Timeout
  
ParagraphActionsIT.testCreateNewButton:77->AbstractZeppelinIT.waitForParagraph:84->AbstractZeppelinIT.pollingWait:110
 » Timeout
```
which is not related,  will be reopening. 
ready 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 pull request #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

2016-06-14 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

2016-06-14 Thread cloverhearts
GitHub user cloverhearts reopened a pull request:

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

ZEPPLIN-976 ] HotFix - zeppelin server does not restart when incorrect 
credentials data.

### What is this PR 
zeppelin server does not restart when incorrect credentials data.

reproduce.
1. Click to zeppelin home for web ui.
2. Click to zeppelin Credentials.
3. 'Entity' information without writing, username and password only written 
to storage.
4. Click to zeppelin home for web ui.
5. Click to zeppelin Credentials.
and zeppelin restart.
# but, Zeppelin does not work.

It creates the wrong json file. 'conf / credentials.json' according to the 
null.

### What type of PR is it?
Hot Fix

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

### How should this be tested?

1. Click to zeppelin home for web ui.
2. Click to zeppelin Credentials.
3. 'Entity' information without writing, username and password only written 
to storage.
4. Click to zeppelin home for web ui.
5. Click to zeppelin Credentials.
and zeppelin restart.

zeppelin does work!

### Reproduced Screenshots

![zeppelin-server-error2](https://cloud.githubusercontent.com/assets/10525473/15889828/b92590d8-2da7-11e6-9b51-0a82c3bb9f1f.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/cloverhearts/zeppelin 
hotfix/CredentialsJsonBug

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

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


commit 1e5cd727c499ea1b700c1d9df81dd7aa69fa4fc9
Author: CloverHearts 
Date:   2016-06-08T09:12:39Z

Credentials Json serialize backend bug.

commit 4b9aba3455a5f441b17c41d6335a0222c1fb59a6
Author: CloverHearts 
Date:   2016-06-08T11:20:36Z

changed status code for CredentialsRestapi

commit e9a1e93289ec8802ec4fa1e96afb16e986d07629
Author: CloverHearts 
Date:   2016-06-14T05:08:47Z

Merge branch 'master' into hotfix/CredentialsJsonBug

commit 5079495feb1a3786127e9f1c484aabcfc3476817
Author: CloverHearts 
Date:   2016-06-14T05:50:30Z

add test code for credentials backends about invalid 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 issue #986: [Zeppelin 946] Permissions not honoring group

2016-06-14 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/986
  
@rja1 have made some changes, hope 
this(188ac170c7a0921044f301ae703bf580a71b2cdf) solves for 
"activeDirectoryRealm.principalSuffix isn't honoured"


---
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 #980: [ZEPPELIN-871] [WIP] spark 2.0 interpreter on scala 2.1...

2016-06-14 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/980
  
- If you ship in the same folder scala libs from different versions, it 
will at runtime - You don't have to rebuilt, but you have to arrange classpath 
is built with scala libs of the same version
- Gui, cli... Makes sense. Is t is what @felixcheung was refereing to?


---
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 #980: [ZEPPELIN-871] [WIP] spark 2.0 interpreter on scala 2.1...

2016-06-14 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/980
  
@echarles 

> I was just saying that a single binary will not fit all various users 
need (multiple combination of interpeters and versions).

If single SparkInterpreter binary is compatible with various spark versions 
and scala versions, and single FlinkInterpreter binary is compatible with 
various flink versions and scala versions, user can choose any combination 
without rebuild. isn't it?

> #629 is merged. What remains to get ZEPPELIN-598 operational?

After #908 is merged, we'll need proper commandline or GUI to access this 
feature.
And publish all interpreters to maven repository and remove interpreter 
dependencies from binary package.



---
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 #1003: [DOCS] Fix link to specific doc from release page

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1003
  
Merging if there is no more 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 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 #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1005
  
@Leemoonsoo I also agree to makes this configurable. I'll follow up this 
with another PR. I also think we need to replace using getScheduler to another 
way.


---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-14 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/976
  
add test code about invalid request for credentials api


---
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 #980: [ZEPPELIN-871] [WIP] spark 2.0 interpreter on scala 2.1...

2016-06-14 Thread echarles
Github user echarles commented on the issue:

https://github.com/apache/zeppelin/pull/980
  
@Leemoonsoo 
- I was just saying that a single binary will not fit all various users 
need (multiple combination of interpeters and versions).
- #629 is merged. What remains to get ZEPPELIN-598 operational?

@felixcheung Is there already a jira for the proposal you are thinkg to? I 
feel we have nearly everything open, but maybe we see an umbrella?



---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

2016-06-14 Thread cloverhearts
GitHub user cloverhearts reopened a pull request:

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

ZEPPLIN-976 ] HotFix - zeppelin server does not restart when incorrect 
credentials data.

### What is this PR 
zeppelin server does not restart when incorrect credentials data.

reproduce.
1. Click to zeppelin home for web ui.
2. Click to zeppelin Credentials.
3. 'Entity' information without writing, username and password only written 
to storage.
4. Click to zeppelin home for web ui.
5. Click to zeppelin Credentials.
and zeppelin restart.
# but, Zeppelin does not work.

It creates the wrong json file. 'conf / credentials.json' according to the 
null.

### What type of PR is it?
Hot Fix

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

### How should this be tested?

1. Click to zeppelin home for web ui.
2. Click to zeppelin Credentials.
3. 'Entity' information without writing, username and password only written 
to storage.
4. Click to zeppelin home for web ui.
5. Click to zeppelin Credentials.
and zeppelin restart.

zeppelin does work!

### Reproduced Screenshots

![zeppelin-server-error2](https://cloud.githubusercontent.com/assets/10525473/15889828/b92590d8-2da7-11e6-9b51-0a82c3bb9f1f.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/cloverhearts/zeppelin 
hotfix/CredentialsJsonBug

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

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


commit 1e5cd727c499ea1b700c1d9df81dd7aa69fa4fc9
Author: CloverHearts 
Date:   2016-06-08T09:12:39Z

Credentials Json serialize backend bug.

commit 4b9aba3455a5f441b17c41d6335a0222c1fb59a6
Author: CloverHearts 
Date:   2016-06-08T11:20:36Z

changed status code for CredentialsRestapi

commit e9a1e93289ec8802ec4fa1e96afb16e986d07629
Author: CloverHearts 
Date:   2016-06-14T05:08:47Z

Merge branch 'master' into hotfix/CredentialsJsonBug

commit 5079495feb1a3786127e9f1c484aabcfc3476817
Author: CloverHearts 
Date:   2016-06-14T05:50:30Z

add test code for credentials backends about invalid 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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

2016-06-14 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


---
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 #997: [MINOR] Make nav zeppelin version to point ZEPPELIN_VER...

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/997
  
Merging if there is 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 #1008: [ZEPPELIN-997] Export data to csv

2016-06-14 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1008
  
@corneadoug I just changed, please check and let me know your opinion.


---
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 #1004: [ZEPPELIN-996] Improve first page and dropdown menu in...

2016-06-14 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1004
  
@corneadoug Yeah you're right and it makes sense indeed. I addressed it :)


---
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 #1004: [ZEPPELIN-996] Improve first page and dropdown menu in...

2016-06-14 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1004
  
Oh indeed,
Since they are more generalistic and also a but hidden down there, it could 
be a better read flow to have them right below the overview


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