[GitHub] zeppelin issue #1055: [ZEPPELIN-1028] Fix exported notebook importing error

2016-06-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1055 @sixmen Yeah thanks for clarifying 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

[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 #1055: [ZEPPELIN-1028] Fix exported notebook importing error

2016-06-22 Thread sixmen
Github user sixmen commented on the issue: https://github.com/apache/zeppelin/pull/1055 It works well in my test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

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

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1058 @astroshim No worries. Thanks for the quick response. LGTM and merge into master branch if there're no more discussions. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1059: [ZEPPELIN-954] Fix table cell selection problem on sec...

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1059 Tested, No more handsontable duplicates. Was there no other way to just update the data instead of destroying the table and creating a new one? --- If your project is set up for it, you

[GitHub] zeppelin issue #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1066 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

[GitHub] zeppelin pull request #994: [ZEPPELIN-986] Create publish release script

2016-06-22 Thread asfgit
Github user asfgit 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

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1013 I agree with @khalidhuseynov that having Connected/Disconnected being replace by the username isn't great. However I still think that showing a username cf: 'anonymous' for an app

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1013 @khalidhuseynov Right. I totally agree with you. @minahlee It's more intuitive than before 👍 --- If your project is set up for it, you can reply to this email and have your reply

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

2016-06-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1063 Looks awesome to me, thank you! I think it's ready to merge, if there is no further discussion --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1067: ZEPPELIN-1048: Pandas support for python interpreter

2016-06-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1067 Thank you guys for the prompt reviews! @Leemoonsoo agree on API change and I'm glad to have it merged in 0.6, updated JIRA @minahlee review addressed in 3b1ad36 If there is no

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-22 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1013 I've just went through the conversation here, and i think that having `userName` instead of `connected` status in authenticated mode with additional `logout` menu is quite misleading.

[GitHub] zeppelin issue #1067: ZEPPELIN-1048: Pandas support for python interpreter

2016-06-22 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1067 I left few notes on documentation and `help() ` function. Looks good to me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1058 @Leemoonsoo Sorry for late response. I fixed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[jira] [Created] (ZEPPELIN-1050) Cannot scroll down if there are many notebooks

2016-06-22 Thread Ahyoung (JIRA)
Ahyoung created ZEPPELIN-1050: - Summary: Cannot scroll down if there are many notebooks Key: ZEPPELIN-1050 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1050 Project: Zeppelin Issue Type:

[GitHub] zeppelin issue #1042: [ZEPPELIN-1046] bin/install-interpreter.sh for netinst...

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1042 It's ready to review and merge. Tried updated create_release.sh script. `zeppelin-0.6.0-bin-all.tgz` is about 498M while `zeppelin-0.6.0-netinst.tgz` will be about 208MB. --- If your

[GitHub] zeppelin pull request #1069: [ZEPPELIN-281] Windows7 build2

2016-06-22 Thread fvaleri
GitHub user fvaleri opened a pull request: https://github.com/apache/zeppelin/pull/1069 [ZEPPELIN-281] Windows7 build2 ### What is this PR for? For next release I would like to do simple additions to my proxy setup on main README: - Add Maven proxy configuration example.

[GitHub] zeppelin issue #908: ZEPPELIN-598 ] Dynamic loading for Interpreter and API

2016-06-22 Thread ggdupont
Github user ggdupont commented on the issue: https://github.com/apache/zeppelin/pull/908 No luck, it can't get to load my jar from my local repo (nexus on docker). My request : { "artifact": "org.apache.zeppelin:zeppelin-neo4j:0.6.0-SNAPSHOT", "className":

[GitHub] zeppelin issue #1055: [ZEPPELIN-1028] Fix exported notebook importing error

2016-06-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1055 @corneadoug Thank you for sharing your opinion and I agree with you. So I made it to support below two date format for backward compatibility. - `-MM-dd'T'HH:mm:ssZ` : exported after

[GitHub] zeppelin pull request #1056: [TEST] Selenium Tests 01

2016-06-22 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1056 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #1056: [TEST] Selenium Tests 01

2016-06-22 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1056 pass more retry ci test. --- If your project is set up for 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 #1056: [TEST] Selenium Tests 01

2016-06-22 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1056 [TEST] Selenium Tests 01 only test - selenium merge - https://github.com/apache/zeppelin/pull/1048 , https://github.com/apache/zeppelin/pull/1053 Please, DO NOT

[GitHub] zeppelin issue #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1058 @astroshim Now there're commits not belongs to this contributions, 41a7302, fd715c8, ab5ce62 Could you remove these commits? --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1058 The selenium test has passed!! --- If your project is set up for it, 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 #1067: ZEPPELIN-1048: Pandas support for python interpreter

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1067 I saw https://issues.apache.org/jira/browse/ZEPPELIN-1048 marked 0.7.0 as a fix version. Not a strong opinion, and i don't think it should be blocker for 0.6 release. But If we're

[GitHub] zeppelin pull request #1068: [TEST] CI -BUILD TEST - TASK 1

2016-06-22 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1068 [TEST] CI -BUILD TEST - TASK 1 TEST PR ![cap 2016-06-23 02-21-46-486](https://cloud.githubusercontent.com/assets/10525473/16276424/46c26e56-38e9-11e6-945e-45d8a361543e.png) You

[GitHub] zeppelin issue #1057: [TEST] Selenium Tests 02

2016-06-22 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1057 Test done. Closed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] zeppelin pull request #1057: [TEST] Selenium Tests 02

2016-06-22 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1057 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #1067: ZEPPELIN-1048: Pandas support for python interpreter

2016-06-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1067 It is ready to be merged now. TODOs will be addressed in different PR, after #1063 is merged --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin pull request #1067: ZEPPELIN-1048: Pandas support for python interp...

2016-06-22 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/zeppelin/pull/1067 ZEPPELIN-1048: Pandas support for python interpreter ### What is this PR for? ### What type of PR is it? Feature ### Todos * [x] fix NPE in logs on empty paragraph

[GitHub] zeppelin issue #795: [ZEPPELIN-183] Interpreter process per note

2016-06-22 Thread JHowix
Github user JHowix commented on the issue: https://github.com/apache/zeppelin/pull/795 Done, thank you for the quick response! https://issues.apache.org/jira/browse/ZEPPELIN-1049 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Created] (ZEPPELIN-1049) Stop Interpretters at Notebook level when Running in Isolated and Scoped Mode

2016-06-22 Thread John J. Howard (JIRA)
John J. Howard created ZEPPELIN-1049: Summary: Stop Interpretters at Notebook level when Running in Isolated and Scoped Mode Key: ZEPPELIN-1049 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1049

[GitHub] zeppelin issue #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1058 @Leemoonsoo I rebased. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[jira] [Created] (ZEPPELIN-1048) Python: pandas support using Table display system

2016-06-22 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-1048: Summary: Python: pandas support using Table display system Key: ZEPPELIN-1048 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1048 Project:

[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 #795: [ZEPPELIN-183] Interpreter process per note

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/795 @JHowix It does not include functionality to restart interpreter at the notebook level. I think it really make sense restart interpreter at the notebook level in case of 'scoped' or

[GitHub] zeppelin issue #1059: [ZEPPELIN-954] Fix table cell selection problem on sec...

2016-06-22 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1059 \cc @corneadoug for review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[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 #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-22 Thread suvam97
Github user suvam97 commented on the issue: https://github.com/apache/zeppelin/pull/1066 @corneadoug : sorry for late response ! Changes updated. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] zeppelin pull request #1065: FIX merge mistake on Branch 0.6

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/1065 --- If your project is set up for it, you can reply to this email and 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-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1013 CI failure is irrelevant to this change. Tested and it looks good to me. +1 for merge it into master and branch-0.6 --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1065: FIX merge mistake on Branch 0.6

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1065 Thanks @bzz @minahlee taking a look. i'm merging it into branch-0.6 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

[GitHub] zeppelin issue #1064: BugFix-blocking of blank values insertion on the Crede...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1064 @bzz I fixed as your comments. 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

[GitHub] zeppelin issue #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1066 @suvam97 Sorry, I was actually testing your branch :) Works, great, could you then delete the `notebookFilter` since it is not being used? --- If your project is set up for it, you can

[GitHub] zeppelin issue #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1066 @suvam97 Maybe you should try building from master and cleaning your cache. Current filtering is working perfectly for me

[GitHub] zeppelin pull request #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-22 Thread suvam97
GitHub user suvam97 opened a pull request: https://github.com/apache/zeppelin/pull/1066 [ZEPPELIN-1047] Add filter to handle upper case ### What is this PR for? The notebook filter in navbar does not filter the results of note if note name is given in upper case .

[GitHub] zeppelin issue #1063: [ZEPPELIN-1045] Apply new mechanism to PythonInterpret...

2016-06-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1063 BTW while we are here, how do you think shall we also set `zeppelin.python.maxResult` to 1000? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Created] (ZEPPELIN-1047) Notebook filter in navbar matching is case senstive.

2016-06-22 Thread Suvam Mohanty (JIRA)
Suvam Mohanty created ZEPPELIN-1047: --- Summary: Notebook filter in navbar matching is case senstive. Key: ZEPPELIN-1047 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1047 Project: Zeppelin

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1013 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

[GitHub] zeppelin issue #1063: [ZEPPELIN-1045] Apply new mechanism to PythonInterpret...

2016-06-22 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1063 Looks great to me, thank you @minahlee ! Will 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

[GitHub] zeppelin issue #971: [ZEPPELIN-961] Longer names to the notebook make the na...

2016-06-22 Thread sagarkulkarni3592
Github user sagarkulkarni3592 commented on the issue: https://github.com/apache/zeppelin/pull/971 Also, observed this. When the notebook name is shorter, buttons appear far from the name or box as shown. ![screen shot 2016-06-22 at 12 48 00

[GitHub] zeppelin issue #971: [ZEPPELIN-961] Longer names to the notebook make the na...

2016-06-22 Thread sagarkulkarni3592
Github user sagarkulkarni3592 commented on the issue: https://github.com/apache/zeppelin/pull/971 @minahlee Done the changes necessary to make the overlap go away. ![screen shot 2016-06-22 at 12 45 34

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1050 Tested, 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 #1065: FIX merge mistake on Branch 0.6

2016-06-22 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1065 FIX merge mistake on Branch 0.6 ### What is this PR for? There were some mistake merging https://github.com/apache/zeppelin/pull/998 into branch-0.6 ### What type of PR is it?

[GitHub] zeppelin pull request #1046: [TEST] Experimental with Apache's travis accoun...

2016-06-22 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/1046 --- If your project is set up for it, you can reply to this email and 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 #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?