[GitHub] zeppelin issue #2192: [ZEPPELIN-2320] fixing test + modifying map browsing

2017-04-06 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2192 thanks for the clarification @AhyoungRyu --- If your project is set 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 #2192: [ZEPPELIN-2320] fixing test + modifying map browsing

2017-04-06 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2192 @astroshim: why? does that mean I also need to do https://github.com/apache/zeppelin/pull/2099 based on branch-0.7 ? --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin issue #2192: [ZEPPELIN-2320] fixing test + modifying map browsing

2017-04-06 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2192 @astroshim Why? --- If your project is set 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 #2151: [ZEPPELIN-465] Be able to run paragraph and the follow...

2017-04-03 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2151 @AhyoungRyu sorry for the late answer, I have updated the doc accordingly to your advice, and changed the wording of the UI command. It is now "Run with successors", but if a nati

[GitHub] zeppelin issue #2186: [ZEPPELIN-2229] revert ndv3 version from 1.8.5 to 1.7....

2017-03-27 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2186 @soralee , @AhyoungRyu , @anthonycorbacho : proposal of workaround here, https://github.com/apache/zeppelin/pull/2194 . Feedbacks welcome! --- If your project is set up for it, you can reply

[GitHub] zeppelin pull request #2194: [ZEPPELIN-2229] Providing workaround for nvd3 v...

2017-03-27 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2194 [ZEPPELIN-2229] Providing workaround for nvd3 voronoi ### What is this PR for? After bumping nvd3 to 1.8.5, an error was spotted due to a bug in nvd3. This PR provides a workaround until

[GitHub] zeppelin pull request #2192: [ZEPPELIN-2320] fixing test + modifying map bro...

2017-03-27 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2192 [ZEPPELIN-2320] fixing test + modifying map browsing ### What is this PR for? fixing String comparison and improving map browsing ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #2186: [ZEPPELIN-2229] revert ndv3 version from 1.8.5 to 1.7....

2017-03-27 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2186 My 2 cts here: Yep the scatter chart bug in 1.8.5 is annoying, and unfortunately nvd3 does not seem to release very frequently ( 2-3 times a year or so). However I am not sure reverting

[GitHub] zeppelin pull request #2151: [ZEPPELIN-465] Be able to run paragraph and the...

2017-03-21 Thread Remilito
GitHub user Remilito reopened a pull request: https://github.com/apache/zeppelin/pull/2151 [ZEPPELIN-465] Be able to run paragraph and the following ones ### What is this PR for? Allow to run a paragraph **and** the ones following. This is useful for instance if you

[GitHub] zeppelin pull request #2151: [ZEPPELIN-465] Be able to run paragraph and the...

2017-03-21 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2151 --- If your project is set 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 #2151: [ZEPPELIN-465] Be able to run paragraph and the follow...

2017-03-21 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2151 @AhyoungRyu : updated (and now building and passing tests :sweat_smile: ). @jongyoul : I am not sure to fully understand what you mean. If you are talking "UI" IMO the n

[GitHub] zeppelin issue #2111: [ZEPPELIN-2230] clear bower_contents if bower.json cha...

2017-03-20 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2111 @jongyoul : https://github.com/apache/zeppelin/pull/2167 should help --- If 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 #2099: [ZEPPELIN-2106] providing paragraph config in create n...

2017-03-20 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2099 @Leemoonsoo : 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 have this feature enabled

[GitHub] zeppelin pull request #2167: [ZEPPELIN-2230] .travis.yml protecting against ...

2017-03-20 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2167 [ZEPPELIN-2230] .travis.yml protecting against ambiguous commit range ### What is this PR for? Following the revert of [](https://github.com/apache/zeppelin/pull/2111 ) I added some

[GitHub] zeppelin issue #2099: [ZEPPELIN-2106] providing paragraph config in create n...

2017-03-18 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2099 @Leemoonsoo thanks for the feedback, I think we should keep only B. I'll update the PR in a couple of days. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin pull request #2151: [ZEPPELIN-465] Be able to run paragraph and the...

2017-03-17 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2151 [ZEPPELIN-465] Be able to run paragraph and the following ones ### What is this PR for? Allow to run a paragraph **and** the ones following. This is useful for instance if you

[GitHub] zeppelin issue #2099: [ZEPPELIN-2106] providing paragraph config in create n...

2017-03-15 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2099 @Leemoonsoo : the most common usages would be to set visual paramaters, i.e. graph options, paragraph width and paragraph title show, so the idea was to provide a more direct way to configure

[GitHub] zeppelin pull request #2111: [ZEPPELIN-2230] clear bower_contents if bower.j...

2017-03-08 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2111 [ZEPPELIN-2230] clear bower_contents if bower.json changed ### What is this PR for? Before travis build: * check if any bower.json file is part of the commit range * check if any

[GitHub] zeppelin issue #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

2017-03-08 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2042 FYI: https://issues.apache.org/jira/browse/ZEPPELIN-2230, comments appreciated. --- 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 #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

2017-02-23 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2042 @AhyoungRyu : I have just checked, those "axis-boxes" seemed to have disappeared from nvd3 after 1.7.1. However the new tooltip provides both x and y values. WDYT? --- If your proj

[GitHub] zeppelin issue #2043: [ZEPPELIN-2114] adding reload endpoint

2017-02-22 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2043 @AhyoungRyu: let me know if need me to enrich the new notebook-repositories doc. --- 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 #2043: [ZEPPELIN-2114] adding reload endpoint

2017-02-20 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2043 [ZEPPELIN-2114] adding reload endpoint ### What is this PR for? Adding endpoint in NotebookRepoRestApi to trigger reload and broadcast of the note list. Sending a GET request to /api

[GitHub] zeppelin pull request #2023: [ZEPPELIN-2114]: adding reload endpoint + fixin...

2017-02-20 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2023 --- If your project is set 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 #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

2017-02-20 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2042 [ZEPPELIN-1588]: bumping nvd3 to 1.8.5 ### What is this PR for? * bump nvd3 to 1.8.5 (and remove depecrated functions) * display percentage in pie chart [solve ZEPPELIN-1891] NB

[GitHub] zeppelin pull request #2040: [ZEPPELIN-1588]: bumping nvd3

2017-02-20 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2040 --- If your project is set 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 #2040: [ZEPPELIN-1588]: bumping nvd3

2017-02-20 Thread Remilito
GitHub user Remilito reopened a pull request: https://github.com/apache/zeppelin/pull/2040 [ZEPPELIN-1588]: bumping nvd3 ### What is this PR for? * bump nvd3 to 1.8.5 (and remove depecrated functions) * display percentage in pie chart [solve [ZEPPELIN-1891]] NB

[GitHub] zeppelin pull request #2040: [ZEPPELIN-1588]: bumping nvd3

2017-02-20 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2040 --- If your project is set 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 #2040: [ZEPPELIN-1588]: bumping nvd3

2017-02-20 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2040 --- If your project is set 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 #2040: [ZEPPELIN-1588]: bumping nvd3

2017-02-20 Thread Remilito
GitHub user Remilito reopened a pull request: https://github.com/apache/zeppelin/pull/2040 [ZEPPELIN-1588]: bumping nvd3 ### What is this PR for? * bump nvd3 to 1.8.5 (and remove depecrated functions) * display percentage in pie chart [solve [ZEPPELIN-1891]] NB

[GitHub] zeppelin pull request #2040: [ZEPPELIN-1588][ZEPPELIN-1891]: bumping nvd3

2017-02-20 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2040 [ZEPPELIN-1588][ZEPPELIN-1891]: bumping nvd3 ### What is this PR for? * bump nvd3 to 1.8.5 (and remove depecrated functions) * display percentage in pie chart NB: visualization

[GitHub] zeppelin issue #2023: [ZEPPELIN-2114]: adding reload endpoint + fixing test

2017-02-17 Thread Remilito
Github user Remilito commented on the issue: https://github.com/apache/zeppelin/pull/2023 I don't think the failing build has anything to do with this PR. Please review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin pull request #2023: [ZEPPELIN-2114]: adding reload endpoint + fixin...

2017-02-16 Thread Remilito
GitHub user Remilito reopened a pull request: https://github.com/apache/zeppelin/pull/2023 [ZEPPELIN-2114]: adding reload endpoint + fixing test ### What is this PR for? Adding endpoint in NotebookRepoRestApi to trigger reload and broadcast of the note list. Sending a GET

[GitHub] zeppelin pull request #2023: [ZEPPELIN-2114]: adding reload endpoint + fixin...

2017-02-16 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2023 --- If your project is set 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 #2023: [ZEPPELIN-2114]: adding reload endpoint + fixin...

2017-02-15 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2023 [ZEPPELIN-2114]: adding reload endpoint + fixing test ### What is this PR for? Adding endpoint in NotebookRepoRestApi to trigger reload and broadcast of the note list. Sending a GET

[GitHub] zeppelin pull request #2018: [ZEPPELIN-2114]: Adding an endpoint in Notebook...

2017-02-15 Thread Remilito
Github user Remilito closed the pull request at: https://github.com/apache/zeppelin/pull/2018 --- If your project is set 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 #2018: [ZEPPELIN-2114]: Adding an endpoint in Notebook...

2017-02-14 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2018 [ZEPPELIN-2114]: Adding an endpoint in NotebookRepoRestApi to reload note list ### What is this PR for? Adding endpoint in NotebookRepoRestApi to trigger reload and broadcast of the note