[jira] [Created] (ZEPPELIN-692) Add setting to set s3 endpoint url ZEPPELIN_NOTEBOOK_S3_ENDPOINT

2016-02-22 Thread Alexander Pivovarov (JIRA)
Alexander Pivovarov created ZEPPELIN-692: Summary: Add setting to set s3 endpoint url ZEPPELIN_NOTEBOOK_S3_ENDPOINT Key: ZEPPELIN-692 URL: https://issues.apache.org/jira/browse/ZEPPELIN-692 Pr

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187583387 I've tried to add an icon before and it overflows the maximum width of the popup. We can also reduce the font-size so the long paragraph id will fit in bu

[GitHub] incubator-zeppelin pull request: Option to restart interpreter fro...

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-187570184 @felixcheung I know it can be confusing (or ambiguous if run all was used), and yes I agree if restart, it may clear local variables depending upon in

[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187568405 @felixcheung sure if I see it after https://github.com/apache/incubator-zeppelin/pull/727 gets merged, I'll let you know. --- If your project is set

[GitHub] incubator-zeppelin pull request: Add selenium test case for show a...

2016-02-22 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/692#issuecomment-187559272 @corneadoug Thank you for pointing it out. Changes done, please have a look. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-691] Enable user to cha...

2016-02-22 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/738#issuecomment-187519080 @Leemoonsoo @felixcheung Thanks! I've just made a Jira issue for this PR : ) --- If your project is set up for it, you can reply to this email and have yo

[jira] [Created] (ZEPPELIN-691) Enable user to change shell interpreter command timeout property

2016-02-22 Thread Ahyoung (JIRA)
Ahyoung created ZEPPELIN-691: Summary: Enable user to change shell interpreter command timeout property Key: ZEPPELIN-691 URL: https://issues.apache.org/jira/browse/ZEPPELIN-691 Project: Zeppelin

[GitHub] incubator-zeppelin pull request: Get travis to retry download of S...

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/727#issuecomment-187510088 Merging this then since it seems to help.. --- 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: Get travis to retry download of S...

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/727#issuecomment-187510015 more download errors: https://s3.amazonaws.com/archive.travis-ci.org/jobs/110955308/log.txt I suspect we should travis_retry build as well - t

[GitHub] incubator-zeppelin pull request: Enable user to change shell inter...

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/738#issuecomment-187509898 looks good --- If your project is set 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: test testAngularDisplay failure

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187507531 if you see `No output has been received in the last 10 minutes` please let me know --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187507463 LGTM. merging if no more comment --- If 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] incubator-zeppelin pull request: User in interpreter context

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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187505828 Is it feasible to have a copy button? or is it to crowded? http://fortawesome.github.io/Font-Awesome/icon/clipboard/ --- If your project is set up fo

[GitHub] incubator-zeppelin pull request: Get travis to retry download of S...

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

[GitHub] incubator-zeppelin pull request: Get travis to retry download of S...

2016-02-22 Thread felixcheung
GitHub user felixcheung reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/727 Get travis to retry download of Spark release ### What is this PR for? Refactor download and use travis' builtin mechanism to retry on failure - hopefully it will then hit a di

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-680] Cron job will run ...

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

[GitHub] incubator-zeppelin pull request: [MINOR] Update copyright year

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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-631 Allow user to config...

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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-680] Cron job will run ...

2016-02-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/730#issuecomment-187462805 Thanks @felixcheung for taking a look. I'm merging it. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin pull request: Option to restart interpreter fro...

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-187440655 I guess from the usability point of view, would it be confusing - because restarting interpreter at a particular paragraph, but all local variables would

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-680] Cron job will run ...

2016-02-22 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/730#issuecomment-187440281 looks good --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[jira] [Created] (ZEPPELIN-690) %dep interpreter could report "Already loaded" or "WARN" instead of "ERROR"

2016-02-22 Thread Jeff Steinmetz (JIRA)
Jeff Steinmetz created ZEPPELIN-690: --- Summary: %dep interpreter could report "Already loaded" or "WARN" instead of "ERROR" Key: ZEPPELIN-690 URL: https://issues.apache.org/jira/browse/ZEPPELIN-690

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187370473 Done, see updated 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 proje

[GitHub] incubator-zeppelin pull request: Add selenium test case for show a...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/692#issuecomment-187324633 When you use multiple times the same XPath, it would be better to have a reusable variable for it. This Xpath for the title `//div[contains(@class

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187320493 Ok, makes sense, thanks, I'll update the PR accordingly --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] incubator-zeppelin pull request: Add new selenium test case to tes...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/693#issuecomment-187318586 Bad rebase again, you are replacing `testRemoveButton()`. It could be better for you to do a batch of tests instead of a PR for each of them, because e

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187315999 The drop-down menu is closing on mouse-up event. Since we want to select the paragraph id to do a copy/paste, its better to remove that behavior on thi

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187310412 @corneadoug Good idea, but why do we need the `ng-click="$event.stopPropagation()"` ? --- If your project is set up for it, you can reply to this email a

[GitHub] incubator-zeppelin pull request: Add selenium test case for show a...

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/692#issuecomment-187305812 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 thi

[GitHub] incubator-zeppelin pull request: Add new selenium test case to tes...

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/693#issuecomment-187305700 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: Add checkbox as a type of dynamic...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/713#issuecomment-187293536 So, from what I saw in the front-end code, a checkbox is considered as a group of checkbox by default? I'm going look into it a bit more, in the mean t

[GitHub] incubator-zeppelin pull request: Add new selenium test case to tes...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/689#issuecomment-187291948 Question, why is that test deleting: `testCreateNewButton`? Is that a result of the rebase? --- If your project is set up for it, you can reply to this e

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187282897 How about presenting it as the title: ``` {{paragraph.id}} ``` https://cloud.githubusercontent.com/assets/710411/13226659/7

[GitHub] incubator-zeppelin pull request: Enable user to change shell inter...

2016-02-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/738#issuecomment-187272641 Looks good to me. CI failing is irrelevant. If you don't mind, could you create corresponding JIRA issue for this? So later anyone easily see which ver

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187269321 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

[jira] [Created] (ZEPPELIN-689) Add AngularJS z object and mirror ZeppelinContext method for binding variable

2016-02-22 Thread DOAN DuyHai (JIRA)
DOAN DuyHai created ZEPPELIN-689: Summary: Add AngularJS z object and mirror ZeppelinContext method for binding variable Key: ZEPPELIN-689 URL: https://issues.apache.org/jira/browse/ZEPPELIN-689 Proje

[GitHub] incubator-zeppelin pull request: Option to restart interpreter fro...

2016-02-22 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-187250818 We can keep it like that for now --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

[jira] [Created] (ZEPPELIN-688) Giving an option to hide result numbering in Spark interpreter

2016-02-22 Thread Ahyoung (JIRA)
Ahyoung created ZEPPELIN-688: Summary: Giving an option to hide result numbering in Spark interpreter Key: ZEPPELIN-688 URL: https://issues.apache.org/jira/browse/ZEPPELIN-688 Project: Zeppelin

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-687] Display paragraph ...

2016-02-22 Thread doanduyhai
GitHub user doanduyhai opened a pull request: https://github.com/apache/incubator-zeppelin/pull/739 [ZEPPELIN-687] Display paragraph id in config dropdown menu ### What is this PR for? Display paragraph id in the config drop down menu This is a sub-task of epic **[ZEPPEL

[GitHub] incubator-zeppelin pull request: Add selenium test case to test hi...

2016-02-22 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/690#issuecomment-187132962 @prabhjyotsingh used handleException. please have a look. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin pull request: Add new selenium test case to tes...

2016-02-22 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/689#issuecomment-187133115 @prabhjyotsingh used handleException, please have a look --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] incubator-zeppelin pull request: Add selenium test case for show a...

2016-02-22 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/692#issuecomment-187133518 @felixcheung @prabhjyotsingh Used handle exception please have a look --- If your project is set up for it, you can reply to this email and have

[jira] [Created] (ZEPPELIN-687) Display ParagraphId in config drop down menu

2016-02-22 Thread DOAN DuyHai (JIRA)
DOAN DuyHai created ZEPPELIN-687: Summary: Display ParagraphId in config drop down menu Key: ZEPPELIN-687 URL: https://issues.apache.org/jira/browse/ZEPPELIN-687 Project: Zeppelin Issue Type:

[GitHub] incubator-zeppelin pull request: Enable user to change shell inter...

2016-02-22 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/incubator-zeppelin/pull/738 Enable user to change shell interpreter command timeout property ### What is this PR for? Although Shell Interpreter has already have [command timeout property](https://github.com

[GitHub] incubator-zeppelin pull request: Option to restart interpreter fro...

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-187107089 @corneadoug Yes, I did have a thought about of having `$scope.paragraph.interpreterRestarting` at Notebook level, but then it more suited here.

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-647] - Native Windows s...

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/734#issuecomment-187097842 I tried it on Windows 7, works well. Except for these two test failure in cassandra, when I do `SPARK_VER="1.6.0" HADOOP_VER="2.3" PROFILE

[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure

2016-02-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187075747 Have tried it multiple times, I think we can merge this. Only reason it failed in last few build was `No output has been received in the last