[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

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

[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

[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

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

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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

  1   2   3   4   >