[GitHub] incubator-zeppelin pull request #963: [gh-pages][ZEPPELIN-952] Refine websit...

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

https://github.com/apache/incubator-zeppelin/pull/963

[gh-pages][ZEPPELIN-952] Refine website style 

### What is this PR for?
Apply same style in website main page as #962

### What type of PR is it?
Documentation

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

### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/15803735/32142684-2aa2-11e6-9b9e-b4db1673f2fd.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/15803734/2f0771e4-2aa2-11e6-888a-d2d0852bb92a.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/incubator-zeppelin 
gh-page/ZEPPELIN-952

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

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


commit 5f7e885baea14b7c1a87120fb99f338059249414
Author: Mina Lee <mina...@nflabs.com>
Date:   2016-06-04T09:32:39Z

remove incubating from mailing list url

commit f862dac384aa360c5b073a6fdde1a714009ef779
Author: Mina Lee <mina...@nflabs.com>
Date:   2016-06-05T04:45:28Z

prettify document




---
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] incubator-zeppelin issue #963: [gh-pages][ZEPPELIN-952] Refine website style...

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

https://github.com/apache/incubator-zeppelin/pull/963
  
Merge 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] incubator-zeppelin issue #962: [ZEPPELIN-952] Refine website style

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

https://github.com/apache/incubator-zeppelin/pull/962
  
Merge 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 #1001: [Minor] Remove 'incubator' from virtual_machine.md

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

https://github.com/apache/zeppelin/pull/1001
  
Duplicated #992 


---
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 #994: [WIP][ZEPPELIN-986] Create publish release scrip...

2016-06-12 Thread minahlee
Github user minahlee closed the pull request at:

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


---
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-12 Thread minahlee
Github user minahlee 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 pull request #994: [WIP][ZEPPELIN-986] Create publish release scrip...

2016-06-12 Thread minahlee
GitHub user minahlee reopened a pull request:

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

[WIP][ZEPPELIN-986] Create publish release script

### What is this PR for?
This PR is to automate release publish to maven repository.
We used to use maven-deploy-plugin and maven-release-plugin for release but 
somehow it didn't work well with Zeppelin so 0.5.5 and 0.5.6 haven't been 
published to maven repository.

Publishing release to maven repository will eventually help zeppelin to 
reduce binary package size by leading users to use Dynamic interpreter 
loading(#908).
Originally below modules were skipped for maven release
 - all interpreters(except spark)
 - zeppelin-display
 - zeppelin-server 
 - zeppelin-distribution

on the other hand this pr will skip only:
 - zeppelin-server
 - zeppelin-distribution

### What type of PR is it?
Infra

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

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? Yes, 
https://cwiki.apache.org/confluence/display/ZEPPELIN/Preparing+Zeppelin+Release 
will be updated accordingly once this pr is merged.


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

$ git pull https://github.com/minahlee/zeppelin ZEPPELIN-986

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

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


commit e03e03ef0fd7e073c941871468627d4371d13888
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-10T20:35:42Z

Add maven publish release script

commit 488d98cc48f2ca335c4a11a1f413b76a59d42de8
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-10T20:35:51Z

Refactor create release script

commit eba94a72e83ebed9ad7809a25ce48cffc14f3fa0
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-10T21:45:16Z

remove deploy plugin from pom since custom script will be used instead for 
deploy




---
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-12 Thread minahlee
GitHub user minahlee reopened a pull request:

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

[MINOR] Make nav zeppelin version to point ZEPPELIN_VERSION in _config.yml

### What is this PR for?
One less line to change when releasing docs by making `ZEPPELIN_VERSION` to 
be retrieved from `_config.yml`.

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? Will be updated in 
https://cwiki.apache.org/confluence/display/ZEPPELIN/Preparing+Zeppelin+Release



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

$ git pull https://github.com/minahlee/zeppelin docs/navZeppelinVersion

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

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


commit dd547b2797f69d67809217e74796b9a702ecd5ee
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-11T01:19:45Z

Make nav zeppelin version to point ZEPPELIN_VERSION in _config.yml




---
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 #992: [ZEPPELIN-980] missing "incubation-" references

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

https://github.com/apache/zeppelin/pull/992
  
Thanks for catching it. 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 #983: [ZEPPELIN-980] Move git repository from incubator-zeppe...

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

https://github.com/apache/zeppelin/pull/983
  
Merging it as hotfix


---
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 #983: [ZEPPELIN-980] Move git repository from incubator-zeppe...

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

https://github.com/apache/zeppelin/pull/983
  
I think it's not changed only in github ui.

you can see that you can access to https://git.apache.org/zeppelin.git/ but 
not to https://git.apache.org/incubator-zeppelin.git/


---
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] incubator-zeppelin pull request #959: [ZEPPELIN-951] Update cron expression ...

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

https://github.com/apache/incubator-zeppelin/pull/959

[ZEPPELIN-951] Update cron expression link and fix broken cron container 
style

### What is this PR for?
Update cron expression link and fix broken cron container style

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

### What is the Jira issue?
ZEPPELIN-951

### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/15796973/e19a492a-29bd-11e6-89dd-cd69310e077d.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/15796974/e73c4ac2-29bd-11e6-8f9a-0e95ab69d2a6.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/incubator-zeppelin ZEPPELIN-951

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

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


commit 82d8e9ad28c331e677b518d89be04a512709c2d4
Author: Mina Lee <mina...@nflabs.com>
Date:   2016-06-04T02:02:12Z

Update cron expression link and fix broken cron container style




---
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] incubator-zeppelin issue #959: [ZEPPELIN-951] Update cron expression link an...

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

https://github.com/apache/incubator-zeppelin/pull/959
  
Same test failure is already reported by @bzz in 
https://issues.apache.org/jira/browse/ZEPPELIN-942


---
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] incubator-zeppelin pull request: [ZEPPELIN-891] Update doap.rdf

2016-05-28 Thread minahlee
Github user minahlee commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/922#issuecomment-92198
  
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] incubator-zeppelin pull request: Zeppelin-531 ] Job management

2016-05-28 Thread minahlee
Github user minahlee commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-98108
  
This is what I get when I direct to Job menu, seems like it doesn't have 
valid information though.
```
 INFO [2016-05-28 01:51:45,603] ({main} Server.java[doStart]:379) - Started 
@6372ms
 INFO [2016-05-28 01:51:45,603] ({main} ZeppelinServer.java[main]:117) - 
Done, zeppelin server started
 INFO [2016-05-28 01:51:55,589] ({qtp1439541970-36} 
AbstractValidatingSessionManager.java[enableSessionValidation]:230) - Enabling 
session validation scheduler...
 WARN [2016-05-28 01:51:55,641] ({qtp1439541970-36} 
SecurityRestApi.java[ticket]:79) - 
{"status":"OK","message":"","body":{"principal":"anonymous","ticket":"anonymous","roles":"[]"}}
 INFO [2016-05-28 01:51:55,790] ({qtp1439541970-15} 
NotebookServer.java[onOpen]:95) - New connection from 0:0:0:0:0:0:0:1 : 54959
ERROR [2016-05-28 01:51:59,300] ({qtp1439541970-11} 
NotebookServer.java[onMessage]:212) - Can't handle message
java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.LinkedList.checkElementIndex(LinkedList.java:553)
at java.util.LinkedList.get(LinkedList.java:474)
at 
org.apache.zeppelin.socket.NotebookServer.generateNotebooksJobInfo(NotebookServer.java:434)
at 
org.apache.zeppelin.socket.NotebookServer.unicastNotebookJobInfo(NotebookServer.java:493)
at 
org.apache.zeppelin.socket.NotebookServer.onMessage(NotebookServer.java:141)
at 
org.apache.zeppelin.socket.NotebookSocket.onWebSocketText(NotebookSocket.java:56)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextMessage(JettyListenerEventDriver.java:128)
at 
org.eclipse.jetty.websocket.common.message.SimpleTextMessage.messageComplete(SimpleTextMessage.java:69)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.appendMessage(AbstractEventDriver.java:65)
at 
org.eclipse.jetty.websocket.common.events.JettyListenerEventDriver.onTextFrame(JettyListenerEventDriver.java:122)
at 
org.eclipse.jetty.websocket.common.events.AbstractEventDriver.incomingFrame(AbstractEventDriver.java:161)
at 
org.eclipse.jetty.websocket.common.WebSocketSession.incomingFrame(WebSocketSession.java:309)
at 
org.eclipse.jetty.websocket.common.extensions.ExtensionStack.incomingFrame(ExtensionStack.java:214)
at 
org.eclipse.jetty.websocket.common.Parser.notifyFrame(Parser.java:220)
at org.eclipse.jetty.websocket.common.Parser.parse(Parser.java:258)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.readParse(AbstractWebSocketConnection.java:632)
at 
org.eclipse.jetty.websocket.common.io.AbstractWebSocketConnection.onFillable(AbstractWebSocketConnection.java:480)
at 
org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
at 
org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
at java.lang.Thread.run(Thread.java:745)
ERROR [2016-05-28 01:52:00,301] ({qtp1439541970-18} 
NotebookServer.java[onMessage]:212) - Can't handle message
```


---
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 link page

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

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

[DOCS] Fix link to specific doc from link page

### What is this PR for?
Link to docs page leads to 
http://zeppelin.apache.org/releases/docs/0.5.6-incubating instead of 
http://zeppelin.apache.org/docs/0.5.6-incubating

### What type of PR is it?
Documentation

### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/15999615/cfabef62-30fb-11e6-92a4-488942f374b4.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/15999625/e12e6ef4-30fb-11e6-8c6c-8c07698d560b.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 fix/docLink

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

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


commit afab83f3c979f2584fc40b201320ec7252fffca8
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-13T07:11:40Z

fix link to specific doc from link page




---
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 #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 #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 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 #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 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 <mina...@apache.org>
Date:   2016-06-15T04:34:28Z

Move interpreter, credential, configuration to dropdown menu

commit 378d83183c6334775bf455e1e02ff087dbd24ff3
Author: Mina Lee <mina...@apache.org>
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 pull request #1016: Bump up version to 0.7.0-SNAPSHOT

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

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

Bump up version to 0.7.0-SNAPSHOT

### What is this PR for?
Bump up version to 0.7.0-SNAPSHOT


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

$ git pull https://github.com/minahlee/zeppelin 0.7.0-SNAPSHOT

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

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


commit f5332212e4a31a1a97b4519738b2448538c1f55b
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-15T07:32:24Z

Bump up version to 0.7.0-SNAPSHOT




---
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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
@echarles I think my description caused confusion. Spark will be included 
in release since most of Zeppelin user is based on Spark. The modules I wrote 
in description is skipped modules for publish and Spark always hasn't been 
skipped from release.


---
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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
@echarles 
- This script is written exactly because of the reason that @bzz explained 
above.

> calling it multiple times with different arguments, instead of single 
script was quite error prone.

- AFAIK, repo is under zeppelin-zengine module, not zeppelin-server. But if 
there is any other use case for zeppelin-server other than notebook repo I 
don't see the reason not to publish. If you come up with other use case, please 
let me know


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

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

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

[MINOR] Small modification in notebook actionbar

### What is this PR for?
Small ui improvements
 - Change keyboard shortcut icon
 - Modify commit container style

### What type of PR is it?
Improvement

### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/16021450/aebc14dc-3167-11e6-91d2-32fa8cd607af.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/16021451/b159c86a-3167-11e6-9311-b59cfffb3f6b.png;>

**Before**
https://cloud.githubusercontent.com/assets/8503346/16021480/d23661e2-3167-11e6-90f9-44340d5efe8f.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/16021482/d4cf92ac-3167-11e6-95a0-1eb28d0ed325.png;>

**Before**
https://cloud.githubusercontent.com/assets/8503346/16021502/eb7c7966-3167-11e6-9b86-79c890b4ab51.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/16021505/ee26bd66-3167-11e6-8c0b-73668b32e01c.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 minor/ui_improve

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

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


commit 64e7a92e701ab7f915e6d5313fe8f6aa61b9e6e3
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-13T20:03:55Z

Update keyboard shortcut icon
Small modification for version control action UI




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

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

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

[ZEPPELIN-997] Export data to csv

### What is this PR for?
Add csv download from front-end leveraging #714

### What type of PR is it?
Improvement

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

### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/16027024/acb80824-3187-11e6-8535-090b06e4807e.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/16027027/afc2b000-3187-11e6-81b8-f79ac3f1ebac.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-997

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

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


commit 2ad6f47b86159ffb0bea9ffb456d05f35f4885e6
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-13T23:53:13Z

Export data to csv




---
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] incubator-zeppelin pull request #966: [ZEPPELIN-958] Support syntax highligh...

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

https://github.com/apache/incubator-zeppelin/pull/966

[ZEPPELIN-958] Support syntax highlight for python and r interpreter

### What is this PR for?
Support syntax highlight for python and r interpreter

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

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


### Screenshots (if appropriate)
**Before**
https://cloud.githubusercontent.com/assets/8503346/15810667/c96e85c0-2b55-11e6-9539-d3cc9a0df35f.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/15810670/cd7960fe-2b55-11e6-8145-50517eaf2195.png;>

**Before**
https://cloud.githubusercontent.com/assets/8503346/15810554/c6e49ec6-2b54-11e6-8b8b-a327dd76d437.png;>

**After**
https://cloud.githubusercontent.com/assets/8503346/15810551/bf48cfc0-2b54-11e6-9ad1-bc16596074ee.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/incubator-zeppelin ZEPPELIN-958

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

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


commit 9821f94e94a6e15520a125966ee6f990945714f8
Author: Mina Lee <mina...@nflabs.com>
Date:   2016-06-06T02:33:00Z

Support syntax hightlight for python and r interpreter




---
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] incubator-zeppelin pull request: Remove incubating from doc

2016-05-31 Thread minahlee
Github user minahlee commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/941
  
@AhyoungRyu Thanks for review I just handled them in bcb6b54


---
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] incubator-zeppelin pull request: Remove incubating from doc

2016-05-31 Thread minahlee
GitHub user minahlee opened a pull request:

https://github.com/apache/incubator-zeppelin/pull/941

Remove incubating from doc

### What is this PR for?
Remove incubating from doc

### What type of PR is it?
Documentation

### 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/incubator-zeppelin 
doc/removeIncubating

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

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


commit accec53f3f79a904d45768900640a4830713da60
Author: Mina Lee <mina...@nflabs.com>
Date:   2016-06-01T01:08:19Z

Remove incubating from doc




---
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] incubator-zeppelin issue #948: Remove doap.rdf from gh-page branch

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

https://github.com/apache/incubator-zeppelin/pull/948
  
Sorry I was confused, let's keep it in `gh-pages`


---
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] incubator-zeppelin issue #948: Remove doap.rdf from gh-page branch

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

https://github.com/apache/incubator-zeppelin/pull/948
  
We don't update our website with `gh-pages` but with `master` so it will be 
still accessible via http://zeppelin.incubator.apache.org/doap.rdf
Thanks for pointing out svn update, I will do it for sure


---
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] incubator-zeppelin pull request #948: Remove doap.rdf from gh-page branch

2016-06-02 Thread minahlee
Github user minahlee closed the pull request at:

https://github.com/apache/incubator-zeppelin/pull/948


---
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] incubator-zeppelin issue #952: [HOTFIX] remove incubating from website nav m...

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

https://github.com/apache/incubator-zeppelin/pull/952
  
Merging it as a hotfix


---
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] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-26 Thread minahlee
Github user minahlee commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221996644
  
@astroshim Did you remove tutorial notebook on purpose? If not, can you 
revert the last commit?


---
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] incubator-zeppelin issue #938: ZEPPELIN-930 ] fix width and not close li tag...

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

https://github.com/apache/incubator-zeppelin/pull/938
  
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] incubator-zeppelin issue #941: Remove incubating from doc

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

https://github.com/apache/incubator-zeppelin/pull/941
  
@prabhjyotsingh thanks for pointing it out. I removed doap.rdf file from 
doc since it already exists in 
https://github.com/apache/incubator-zeppelin/blob/gh-pages/doap.rdf


---
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] incubator-zeppelin issue #941: Remove incubating from doc

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

https://github.com/apache/incubator-zeppelin/pull/941
  
I updated road.pdf file in master branch and removed it from gh-page branch 
in #948 for better search and maintenance.


---
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] incubator-zeppelin pull request: Remove incubating from pom files

2016-05-31 Thread minahlee
GitHub user minahlee opened a pull request:

https://github.com/apache/incubator-zeppelin/pull/942

Remove incubating from pom files

### What is this PR for?
Remove incubating from pom files

### 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/incubator-zeppelin 
tlp/removeIncubating

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

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


commit e605b54621e67757a77cbef87aba16104a964aaf
Author: Mina Lee <mina...@nflabs.com>
Date:   2016-06-01T01:48:24Z

Remove incubating from pom files




---
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] incubator-zeppelin pull request: Zeppelin-531 ] Job management

2016-05-29 Thread minahlee
Github user minahlee commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-222403825
  
@cloverhearts Seems like big number of notebooks/paragraphs slows down 
loading job manage page. Any idea to solve this issue?


---
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 #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 #1043: [ZEPPELIN-905] Fix import notebook with error result

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

https://github.com/apache/zeppelin/pull/1043
  
Merging it into master and branch-0.6 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 #1059: [ZEPPELIN-954] Fix table cell selection problem on sec...

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

https://github.com/apache/zeppelin/pull/1059
  
\cc @corneadoug 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 #971: [ZEPPELIN-961] Longer names to the notebook make the na...

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

https://github.com/apache/zeppelin/pull/971
  
Thanks for the update. I see one more thing needs to be handled.
Could you make btn group not to cover input box in below screenshot?
https://cloud.githubusercontent.com/assets/8503346/16256123/871ba188-3804-11e6-882f-0858c595e89c.png;>



---
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 #1063: [ZEPPELIN-1045] Apply new mechanism to PythonIn...

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

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

[ZEPPELIN-1045] Apply new mechanism to PythonInterpreter

### What is this PR for?
This PR is applying new interpreter register mechanism to python 
interpreter.

### What type of PR is it?
Improvement

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

### 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-1045

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

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


commit 5013890ed5c2f14a04cf4d0b5bdc18062dbed9f4
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-22T04:51:20Z

Apply new mechanism to PythonInterpreter




---
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 #1062: [ZEPPELIN-1007] Toggle only one of the interpre...

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

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

[ZEPPELIN-1007] Toggle only one of the interpreter/permission settings

### What is this PR for?
Prevent Interpreter binding and Note Permissions dialog opens at the same 
time.

### What type of PR is it?
Improvement

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

### Screenshots (if appropriate)
Before
![jun-21-2016 
17-55-31](https://cloud.githubusercontent.com/assets/8503346/16251286/90685022-37d9-11e6-972c-aedb4d48d983.gif)

After
![jun-21-2016 
17-54-45](https://cloud.githubusercontent.com/assets/8503346/16251288/9594f9b0-37d9-11e6-9d84-8d96da221980.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/minahlee/zeppelin ZEPPELIN-1007

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

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


commit 01f9c0557babbd8e7aeb5693d9b1ef59bfd52924
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-22T00:52:17Z

Toggle only one of the interpreter/permission settings




---
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 #1063: [ZEPPELIN-1045] Apply new mechanism to PythonInterpret...

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

https://github.com/apache/zeppelin/pull/1063
  
\cc @bzz @jongyoul 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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
Merge 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 #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 #1006: [MINOR] Small ui modification in notebook actionbar

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

https://github.com/apache/zeppelin/pull/1006
  
Selenium test passes after rebasing 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 #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

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

https://github.com/apache/zeppelin/pull/1013
  
Updated selenium test and CI passes


---
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 #1058: BugFix of Automatically adds %.* of previous paragraph...

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

https://github.com/apache/zeppelin/pull/1058
  
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 #1061: [DOC] Fix broken code block in jdbc document

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

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

[DOC] Fix broken code block in jdbc document

### What is this PR for?
Fix broken code block in jdbc document

### What type of PR is it?
Documentation

### Screenshots (if appropriate)
Before
https://cloud.githubusercontent.com/assets/8503346/16247613/65aa78a2-37bf-11e6-9370-7d0bcba4cbf5.png;>

After
https://cloud.githubusercontent.com/assets/8503346/16247606/61ef7ad2-37bf-11e6-90a4-10234b18611e.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 doc/fixJdbc

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

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






---
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 #458: [WIP] Fix notebook clone bug

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

https://github.com/apache/zeppelin/pull/458
  
Duplicated with #933


---
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 #458: [WIP] Fix notebook clone bug

2016-06-17 Thread minahlee
Github user minahlee closed the pull request at:

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


---
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 #1043: [ZEPPELIN-905] Fix import notebook with error r...

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

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

[ZEPPELIN-905] Fix import notebook with error result

### What is this PR for?
This PR is fixing import/clone notebook with error result. This PR adds 
test based on #933.

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

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

### How should this be tested?
When you try to import or clone notebook with error result, it should work 
without error

### 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-905

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

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


commit a4188be5ec162defecefec115532c7e5fca8f0db
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-05-31T03:05:30Z

[ZEPPELIN-905] fix failed notebook import bug

commit 1e45a9e2d6f50dbe3c8d7eefa89618aa7496411d
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-05-31T05:14:12Z

[ZEPPELIN-905] add test

commit c13293fe7076803709aee4b7e6d0f8b1ad9d883e
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-05-31T10:23:55Z

fix test, better implementation

commit 803e08abb93ad44ef90608d4e3997518eeca2424
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-06-01T01:20:45Z

revert implementation

commit 32949bc26b31450d0f3e9df83109760eb647d43f
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-06-01T02:41:22Z

trigger CI build

commit d4f6699804db8f332380b17fd594162fc70d14a1
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-06-03T03:48:25Z

log exception

commit 7bf5d01de085332c009ef31ce18b5eae353ba4b5
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-06-03T10:02:43Z

log info -> warn, add message

commit e7af919e82b54d858f933810d9a23c6dffb06e96
Author: Kevin Kim <sangwookim...@gmail.com>
Date:   2016-06-03T10:31:19Z

stylish code

commit 69b8c02f6623686c3d23e6692f397468846eae8c
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-18T07:38:45Z

Add test for clone notebook with String type result




---
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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
@bzz Thanks for the review. I removed mistakenly pushed file.
I included geode, scalding interpreter in maven publish script so that 
geode, scalding users can take advantage of #1042. 
To be honest, I don't know why these two interpreters were not included in 
last release. Can someone explain?


---
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 #1067: ZEPPELIN-1048: Pandas support for python interpreter

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

https://github.com/apache/zeppelin/pull/1067
  
I left few notes on documentation and `help() ` function. Looks good 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 pull request #1070: [HOTFIX] Fix compile error caused by merging #1...

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

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

[HOTFIX] Fix compile error caused by merging #1043

### What is this PR for?
Hotfix for compile error of master, branch-0.6 after merging #1043

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

$ git pull https://github.com/minahlee/zeppelin hotfix/compile_error

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

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


commit b0d5a6bf1598d2b0d09d1968f4f6dc4af366bf09
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-23T05:31:09Z

Fix compile error after merging #1043




---
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 #1070: [HOTFIX] Fix compile error caused by merging #1043

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

https://github.com/apache/zeppelin/pull/1070
  
Thank you for taking a look. Merging it as hot fix


---
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-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
@khalidhuseynov Thanks for the feedback.
I totally agree that showing `Connected` message for non-logged user can be 
confusing.
I made several changes based on your opinion.
- show user name as `anonymous` when shiro is configured as anonymous 
mode.(suggested by @bzz in [previous 
comment](https://github.com/apache/zeppelin/pull/1013#issuecomment-227034217))
- move websocket connected/disconnected text to tooltip.
- merge two dropdown menu to one. I am also skeptical about having one more 
dropdown menu only for logout action. I believe with caret icon, users will 
easily know where to click if they wants to do some action like change 
settings, logout, etc.

**anonymous**
https://cloud.githubusercontent.com/assets/8503346/16289950/82191274-38ae-11e6-9209-48d058137173.png;>
https://cloud.githubusercontent.com/assets/8503346/16289959/904a542a-38ae-11e6-8220-d2bc89feea27.png;>

**authc**
https://cloud.githubusercontent.com/assets/8503346/16289943/7aa104fc-38ae-11e6-8d26-6cb7efed09bf.png;>
https://cloud.githubusercontent.com/assets/8503346/16289947/7eb50a0c-38ae-11e6-9aa0-fb99916b91bc.png;>



---
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 #1042: [ZEPPELIN-1046] bin/install-interpreter.sh for netinst...

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

https://github.com/apache/zeppelin/pull/1042
  
@Leemoonsoo Could you rebase and resolve conflicts?


---
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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
@Leemoonsoo Thanks for pointing it out! I reverted commit of adding geode, 
scalding


---
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-21 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
I am ok with either approach, seems like @corneadoug has strong opinion on 
having two separate menu so I will leave it as it is. I will need to fix 
selenium test to make it to be merged though.


---
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 #1053: [ZEPPELIN-1039] updated CreateNewButton Testcase in Pa...

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

https://github.com/apache/zeppelin/pull/1053
  
Merging it as hotfix


---
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 #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

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

https://github.com/apache/zeppelin/pull/1048
  
Merging it as hotfix


---
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-16 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
Thanks for the feedback guys! I've just pushed some commits and updated 
description with new screenshot.


---
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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
@bzz Sorry for late response, I am working on it and going to push commits 
by today. Thanks for heads up :)


---
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 #1020: [HOTFIX][DOC] Fix image path

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

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

[HOTFIX][DOC] Fix image path

### What is this PR for?
Use relative path instead of absolute path so doc can find image under 
proper version folder

### What type of PR is it?
Documentation

### Screenshots (if appropriate)
Current 0.6.0-SNAPSHOT main page looks like this:
https://cloud.githubusercontent.com/assets/8503346/16102042/9ca90772-3320-11e6-8628-5850003a64a9.png;>


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

$ git pull https://github.com/minahlee/zeppelin hotfix/doc_image

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

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


commit 9574e0a75e99b7333b445eb306e23a5a03539b9f
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-16T00:41:35Z

Fix image path




---
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 #994: [ZEPPELIN-986] Create publish release script

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

https://github.com/apache/zeppelin/pull/994
  
@echarles If there are such needs, nothing stops us from publishing it. I 
will make change accordingly. Thank you for sharing your situation.


---
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 #1016: Bump up version to 0.7.0-SNAPSHOT

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

https://github.com/apache/zeppelin/pull/1016
  
@Leemoonsoo Thanks for the explanation!
@felixcheung as @Leemoonsoo said, branch-0.6 will be bumped up to 
0.6.1-SNAPSHOT once I create v0.6.0-rc1 tag. We have documented this versioning 
policy in 
[wiki](https://cwiki.apache.org/confluence/display/ZEPPELIN/Preparing+Zeppelin+Release#PreparingZeppelinRelease-Createreleasebranch)
 few months ago. Bug fixes will keep being merged both in master and branch-0.6 
and I hope we deliver those maintenance release in one or two month so many 
users can take advantage of 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 #1074: [HOTFIX] Fix python test case

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

https://github.com/apache/zeppelin/pull/1074
  
Closes since it is duplicated with #1073 


---
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 #1074: [HOTFIX] Fix python test case

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

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

[HOTFIX] Fix python test case

### What is this PR for?
Fix python test case caused by #1067 

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

### How should this be tested?
run
```
mvn -pl 'zeppelin-interpreter,python' 
-Dtest=org.apache.zeppelin.python.PythonInterpreterTest -Denforcer.skip=true 
test -DfailIfNoTests=false
```
and see if build success


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

$ git pull https://github.com/minahlee/zeppelin hotfix/pythonTest

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

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


commit 5042bc260f3183739747f745e7d4bc8af179a047
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-23T17:44:01Z

Fix python test case




---
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 #1075: [HOTFXI] Fix python test case

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

https://github.com/apache/zeppelin/pull/1075
  
\cc @bzz 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 #1074: [HOTFIX] Fix python test case

2016-06-23 Thread minahlee
Github user minahlee closed the pull request at:

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


---
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-23 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
Thanks for the feedback all! 
@Leemoonsoo After your explanation, it makes sense to display `anonymous`
@khalidhuseynov I added `WebSocket` because logged user can misunderstand 
that he is connected/disconnected to session.
Integration test passes, I think I will merge this both to master, 
branch-0.6 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 #1066: [ZEPPELIN-1047] Add filter to handle upper case

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

https://github.com/apache/zeppelin/pull/1066
  
Merge if there is no more dicussion


---
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 #971: [ZEPPELIN-961] Longer names to the notebook make the na...

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

https://github.com/apache/zeppelin/pull/971
  
LGTM merging it into master and branch-0.6


---
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 #1071: [ZEPPELIN-1052] Application does not logout user when ...

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

https://github.com/apache/zeppelin/pull/1071
  
@prabhjyotsingh Thanks for the fix!
Seems like this PR is bringing CI error in master, branch-0.6:
```
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 81.986 sec 
<<< FAILURE! - in org.apache.zeppelin.integration.AuthenticationIT
testGroupPermission(org.apache.zeppelin.integration.AuthenticationIT)  Time 
elapsed: 42.459 sec  <<< ERROR!
org.openqa.selenium.TimeoutException: Timed out after 30 seconds waiting 
for org.apache.zeppelin.AbstractZeppelinIT$1@4c156b4b
Build info: version: '2.48.2', revision: 
'41bccdd10cf2c0560f637404c2d96164b67d9d67', time: '2015-10-09 13:08:06'
System info: host: 'testing-worker-linux-docker-8fa71dca-3379-linux-8', ip: 
'172.17.9.181', os.name: 'Linux', os.arch: 'amd64', os.version: 
'3.13.0-40-generic', java.version: '1.7.0_76'
Driver info: driver.version: unknown
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
at 
org.openqa.selenium.remote.ErrorHandler.createThrowable(ErrorHandler.java:206)
at 
org.openqa.selenium.remote.ErrorHandler.throwIfResponseFailed(ErrorHandler.java:158)
at 
org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:647)
at 
org.openqa.selenium.remote.RemoteWebDriver.findElement(RemoteWebDriver.java:353)
at 
org.openqa.selenium.remote.RemoteWebDriver.findElementByXPath(RemoteWebDriver.java:490)
at org.openqa.selenium.By$ByXPath.findElement(By.java:361)
at 
org.openqa.selenium.remote.RemoteWebDriver.findElement(RemoteWebDriver.java:345)
at 
org.apache.zeppelin.AbstractZeppelinIT$1.apply(AbstractZeppelinIT.java:116)
at 
org.apache.zeppelin.AbstractZeppelinIT$1.apply(AbstractZeppelinIT.java:114)
at org.openqa.selenium.support.ui.FluentWait.until(FluentWait.java:238)
at 
org.apache.zeppelin.AbstractZeppelinIT.pollingWait(AbstractZeppelinIT.java:114)
at 
org.apache.zeppelin.integration.AuthenticationIT.authenticationUser(AuthenticationIT.java:109)
at 
org.apache.zeppelin.integration.AuthenticationIT.testGroupPermission(AuthenticationIT.java:177)
Caused by: org.openqa.selenium.NoSuchElementException: Unable to locate 
element: {"method":"xpath","selector":"//div[contains(@class, 
'navbar-collapse')]//li//button[contains(.,'Login')]"}
```
I was able to reproduce error that logout doesn't work by following:
1. start zeppelin with default shiro.ini
2. create `Untitled Note 1` and set permissions to (admin, admin, admin)
3. go back to zeppelin main page (http://localhost:8080)
4. click `Untitled Note 1`
5. login as admin
6. click logout button -> doesn't work
Could you take a look please?


---
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 #1037: fix typo the description of interpreter menu.

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

https://github.com/apache/zeppelin/pull/1037
  
Nice catch! Can we make make space to be consistent between `create / edit 
/ remove` and `bind/unbind`?


---
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 #1043: [ZEPPELIN-905] Fix import notebook with error result

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

https://github.com/apache/zeppelin/pull/1043
  
@swkimme @Leemoonsoo 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 issue #1033: Python: fix for 'run all' paragraphs

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

https://github.com/apache/zeppelin/pull/1033
  
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 #1006: [MINOR] Small ui modification in notebook actionbar

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

https://github.com/apache/zeppelin/pull/1006
  
Rebased master and updated screenshot in description


---
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 #1042: [ZEPPELIN-1046] bin/install-interpreter.sh for netinst...

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

https://github.com/apache/zeppelin/pull/1042
  
Thanks for the effort @Leemoonsoo, @AhyoungRyu I've tested this branch and 
it works like a charm! I left few more comments, once they are addressed it's 
good to merge :)


---
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 #1042: [ZEPPELIN-1046] bin/install-interpreter.sh for netinst...

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

https://github.com/apache/zeppelin/pull/1042
  
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 pull request #1075: [HOTFXI] Fix python test case

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

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

[HOTFXI] Fix python test case

### What is this PR for?
Update  `testPy4jIsNotInstalled `, `testPy4jIsInstalled` test
 - `z.show` -> `def show` to check `show` function is defined
 - check if `bootstrap_input.py` excuted by checking `z = 
Py4jZeppelinContext` instead of `z = PyZeppelinContext`

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

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

$ git pull https://github.com/minahlee/zeppelin adjustPythonTest

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

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


commit f66e9dc3fabebd98441d7da8a70265fbe377261d
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-23T17:44:01Z

Fix python test case




---
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 #1031: [ZEPPELIN-1018] Apply auto "Table of Contents" generat...

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

https://github.com/apache/zeppelin/pull/1031
  
It looks great to me. Thanks for improving document


---
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 #1098: [HOTFIX] update release script

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

https://github.com/apache/zeppelin/pull/1098
  
Merging it to master and branch-0.6 so I can create 0.6.0-rc1


---
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 #1062: [ZEPPELIN-1007] Toggle only one of the interpreter/per...

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

https://github.com/apache/zeppelin/pull/1062
  
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 #1040: [ZEPPELIN-1031] Update Elasticsearch to release 2.3.3

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

https://github.com/apache/zeppelin/pull/1040
  
Merge 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 #1039: [ZEPPELIN-1032] Update Apache Flink to 1.0.3 release

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

https://github.com/apache/zeppelin/pull/1039
  
Merge 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 pull request #1089: [DOC] Update doc image in Explore Zeppelin UI p...

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

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

[DOC] Update doc image in Explore Zeppelin UI page

### What is this PR for?
There were small changes in Zeppelin UI after #1006, #1013, #1081 so update 
screenshot of documents accordingly.

### What type of PR is it?
Documentation

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

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

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


commit a17f31ff59b3c1c2520c24fb4d9ee77b45cedc11
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-26T18:54:08Z

Update doc image in Explore Zeppelin UI page




---
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 #1084: [HOTFIX] Fix flaky AuthenticationIT

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

https://github.com/apache/zeppelin/pull/1084
  
Awesome!!! let's merge 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 #1059: [ZEPPELIN-954] Fix table cell selection problem on sec...

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

https://github.com/apache/zeppelin/pull/1059
  
Merge it 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 pull request #1098: [HOTFIX] update release script

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

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

[HOTFIX] update release script

### What is this PR for?
- Add sparkr, r profile to netinst release build
- Exit release_publish.sh in case creating staging repository fails

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

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


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

$ git pull https://github.com/minahlee/zeppelin fix/release_script

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

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


commit 2fee0826751e7db41687aa411dba14fd6aa99932
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-27T22:14:03Z

Add sparkr, r profile to netinst binary package build

commit bf25d5b84d5d66722f8d2102ab0e1ae7fa83b797
Author: Mina Lee <mina...@apache.org>
Date:   2016-06-27T22:15:04Z

remove quote to make sed works properly




---
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-17 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
I also don't feel the necessity of showing full user name since logged user 
would know his own username. Let me remove username under dropdown then.


---
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-16 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
@corneadoug Having separate menu for the anonymous user totally makes sense.
Then the next question would be where do we want to place it?
Next to Notebook? - will give less confusion who is used to current layout, 
but there won't be big problem once they recognize it
On the right side of nav bar? - will take less space since it will use gear 
icon instead of `Settings` text.

I have no special preference on setting menu location.
@corneadoug, @bzz, @Leemoonsoo what is your opinion 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 #1022: [HOTFIX][DOC] Fix image path again

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

https://github.com/apache/zeppelin/pull/1022
  
@corneadoug yes, the doc for 0.6.0 will be published based on branch-0.6


---
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-16 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1013
  
One question, if `Confidential` is per user menu, should it be shown when 
only user logged in?


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


  1   2   3   4   >