[GitHub] incubator-zeppelin pull request #963: [gh-pages][ZEPPELIN-952] Refine websit...
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...
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
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
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...
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...
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...
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...
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
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...
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...
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 ...
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...
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
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
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
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...
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
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
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
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...
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
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 ...
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
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
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
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...
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
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...
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
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
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
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
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
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...
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...
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...
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
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
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
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
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 ...
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
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...
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...
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...
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...
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...
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
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
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
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...
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...
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
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
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
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...
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
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
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...
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
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...
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...
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
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...
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...
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...
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...
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
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
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
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
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
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
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
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
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...
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
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...
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 ...
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.
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
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
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
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...
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...
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
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...
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
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...
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
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
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...
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
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...
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
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...
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...
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
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...
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. ---