[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2176 @FRosner **How do I get the remote changes / discard my local changes** - Zeppelin will synchronize text automatically. That's the reason why it doesn't have button

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2177 @FRosner Anything would be greatly appreciated. For example trying this PR, suggestion for how to write test code for this, and so on. regarding to triggering auto-save, Zeppelin send

[GitHub] zeppelin issue #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2174 @AhyoungRyu pagination DOM should be located outside of `ng-repeat` for helium containers. https://cloud.githubusercontent.com/assets/4968473/24344271/9e910240-1305-11e7-86f1-51438b9a76c3

[GitHub] zeppelin pull request #2181: [ZEPPELIN-2304] Fix vis Icon sizes in helium pa...

2017-03-23 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2181 [ZEPPELIN-2304] Fix vis Icon sizes in helium page ### What is this PR for? Icon size of helium vis packages doesn't fit into the container button. I attached a image

[GitHub] zeppelin pull request #2179: [ZEPPELIN-2211 Too many WARN logs: "Couldn't ge...

2017-03-23 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2179 [ZEPPELIN-2211 Too many WARN logs: "Couldn't get interpreter editor setting' ### What is this PR for? every time we insert invalid backend magic (e.g `sparc`), `InterpreterSettingMa

[GitHub] zeppelin pull request #2178: [ZEPPELIN-2302] Add info level logs when instal...

2017-03-23 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2178 [ZEPPELIN-2302] Add info level logs when installing node modules and bundling helium pkgs ### What is this PR for? Add info level logs when installing node modules and bundling helium

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 @FRosner I see and agree with that we need more realtime notification on paragraph editing. LGTM --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2177: [ZEPPELIN-2301] DON'T overwrite editor text when note ...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2177 @FRosner I updated GIF. What #2176 and #2177 is trying to prevent local change from overwritten. Overwriting occurs in many cases. - #2176 is only about **when other browsers run

[GitHub] zeppelin issue #2176: [ZEPPELIN-2300] SHOULD NOT update paragraph text when ...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2176 @FRosner I updated GIF. - The basic motivation is, prevent overwriting my local changes when other browsers execute paragraphs. > From the user perspective, this is a criti

[GitHub] zeppelin pull request #2177: [ZEPPELIN-2301] DON'T overwrite editor text whe...

2017-03-22 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2177 --- 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 #2177: [ZEPPELIN-2301] DON'T overwrite editor text whe...

2017-03-22 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2177 [ZEPPELIN-2301] DON'T overwrite editor text when note is renamed, created, ... ### What is this PR for? DON'T overwrite local text change in editor when note is renamed, created

[GitHub] zeppelin pull request #2177: [ZEPPELIN-2301] DON'T overwrite editor text whe...

2017-03-22 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2177 [ZEPPELIN-2301] DON'T overwrite editor text when note is renamed, created, ... ### What is this PR for? DON'T overwrite local text change in editor when note is renamed, created, and so

[GitHub] zeppelin issue #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2174 @AhyoungRyu sure! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2119 I updated and now PR CI is green :) --- 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 #2098: [ZEPPELIN-2217][WIP] AdvancedTransformation for Visual...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 @Leemoonsoo, @AhyoungRyu implementation is finally finished (IMO 😭 ) - Please help review this. - Testing these 2 examples will be helpful (not published yet) [ultimate-line-chart

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 @FRosner Could you attach a GIF for ``` Enter code in a cell Remove the focus to rename the notebook The code gets reset ``` --- If your project is set up for it, you can

[GitHub] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2144 @AhyoungRyu I resolved your all comments. Thanks! --- 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 #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 @FRosner Sorry for late reply. 😭... ### I can't reproduce the original issue reported in 1492 in current master Usually, paragraph will be updated If interpreter updates

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217][WIP] AdvancedTransformation for Visual...

2017-03-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 Few things are left. Thus i'v just changed to WIP again. Sorry for confusing. - adding more `transformation.method` for columnar and drill-down charts: `array:key`, `array:group

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-18 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 updated Please check https://github.com/1ambda/zeppelin-ultimate-line-chart as well. Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2150 It might be due to flaky tests. If so, please just restart failed jobs. I'v just looked your travis build and it's ok - https://travis-ci.org/andreaTP/zeppelin/builds/212066501

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 I'v just updated the PR description. Please help review this @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

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2150 Hi @andreaTP. Thanks for contribution. - Zeppelin build system relies on travis. So please setup your travis account and travis project for Zeppelin. - https://zeppelin.apache.org

[GitHub] zeppelin pull request #2114: [ZEPPELIN-2234][BUG] Can't display the same cha...

2017-03-16 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2114 --- 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 #2114: [ZEPPELIN-2234][BUG] Can't display the same chart agai...

2017-03-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2114 @Leemoonsoo Thanks! --- 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 #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 CI passed. I will update PR description --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] zeppelin pull request #2098: [ZEPPELIN-2217] AdvancedTransformation for Visu...

2017-03-16 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2098 --- 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 #2098: [ZEPPELIN-2217] AdvancedTransformation for Visu...

2017-03-16 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2098 [ZEPPELIN-2217] AdvancedTransformation for Visualization ### What is this PR for? Even if we already have 2 useful transformation, they have limitations. - we can't configure

[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2141 @guicaro I see. There are many ways to trigger travis again. but what you actually need is pushing new commits so that travis get noticed. - Amend your last commit with `git commit

[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2116 Thanks @jongyoul! --- 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 #2116: [ZEPPELIN-2179] "clear output" paragraph doesn'...

2017-03-16 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2116 --- 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 #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable...

2017-03-16 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2144 [ZEPPELIN-2267] Improve Helium Enable / Disable Dialogs ### What is this PR for? Improved Helium Enable / Disable Dialogs to - include more detailed info for enable dialog

[GitHub] zeppelin pull request #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable...

2017-03-16 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2144 --- 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 #2143: [ZEPPELIN-2266] Parse helium description as HTM...

2017-03-16 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2143 [ZEPPELIN-2266] Parse helium description as HTML ### What is this PR for? Parse helium description as HTML for detailed, beautiful explanation ``` "descri

[GitHub] zeppelin pull request #2143: [ZEPPELIN-2266] Parse helium description as HTM...

2017-03-16 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2143 --- 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 #2143: [ZEPPELIN-2266] Parse helium description as HTML

2017-03-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2143 I think this PR is detached from my original branch. can't update at all. I will create new PR for this. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2143: [ZEPPELIN-2266] Parse helium description as HTML

2017-03-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2143 @AhyoungRyu great! - writing new section would be better to for Helium JSON format. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable...

2017-03-16 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2144 [ZEPPELIN-2267] Improve Helium Enable / Disable Dialogs ### What is this PR for? Improved Helium Enable / Disable Dialogs to - include more detailed info for enable dialog

[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 Thanks for fixing bug. I will review and give you feedback soon. --- 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

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2134 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 #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2105 CI 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 and wishes so

[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2116 Regarding to the same fix for master branch, I will open another PR. Thanks for reviewing this PR @Leemoonsoo @jongyoul! --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #2105: [ZEPPELIN-2215] Progress bar for Spell executio...

2017-03-15 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2105 --- 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 #2105: [ZEPPELIN-2215] Progress bar for Spell executio...

2017-03-15 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2105 [ZEPPELIN-2215] Progress bar for Spell execution ### What is this PR for? Add progress bar for spell execution. - Used `RUNNING` state to display progress bar - Decided

[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2141 Hi @randerzander. Thanks for contribution. - Zeppelin build system relies on travis. So please setup your travis account and travis project for Zeppelin. - https

[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2119 CI is green :) 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 project does not have this feature enabled and wishes so

[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2116 CI passed. @Leemoonsoo --- 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 #2119: [ZEPPELIN-2243] Use uppercase for shortcut desc...

2017-03-15 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2119 --- 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 #2119: [ZEPPELIN-2243] Use uppercase for shortcut desc...

2017-03-15 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2119 [ZEPPELIN-2243] Use uppercase for shortcut description ### What is this PR for? Use uppercase for keyboard shortcut description so that user can recognize shortcuts easily. it's

[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2137 FYI, @NohSeho --- 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 #2114: [ZEPPELIN-2234][BUG] Can't display the same chart agai...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2114 Rebased and fixed using the implementation discussed in #2110 - i didn't bring refactoring commits because 0.7.0-branch is not actively developed now. --- If your project is set up

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 I will test and give you feedback soon. --- 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 #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2105 I'v just rebased to resolve conflict. --- 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 #2126: [gh-pages][ZEPPELIN-2209, 2110] Show 'INTERPRETER' typ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2126 Looks nice. @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 this feature enabled

[GitHub] zeppelin issue #2115: [ZEPPELIN-2202] Disable personalized mode btn when not...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2115 @Leemoonsoo Thanks! --- 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 #2115: [ZEPPELIN-2202] Disable personalized mode btn w...

2017-03-15 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2115 --- 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 #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2116 I'v just rebased, let me comment after fixing CI. --- 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 #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2119 Rebased to resolve @felixcheung's the last 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 project does not have

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2134 Sorry for late reply. - regarding to design, it would be great if we use bootstrap-alert. (`warning` level would be ok) For example, ![image](https

[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2119 @soralee Thanks 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 #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2105 @soralee Thanks 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 #2125: [ZEPPELIN-2222] Add Network Visualization

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Pretty graph! I have one question. - It's looks like new display system works with backend interpreter. - And enabled when we click the `network` visualization icon

[GitHub] zeppelin issue #2126: [gh-pages][ZEPPELIN-2209, 2110] Show 'INTERPRETER' typ...

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2126 Pagination what @AhyoungRyu added is nice. - Tested and works as described. ![image](https://cloud.githubusercontent.com/assets/4968473/23844325/bf21f958-0804-11e7-8918

[GitHub] zeppelin issue #2127: [gh-pages] Initialize bootstrap tooltip

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2127 Tested and works as described. --- 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 #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2117 LGTM except one minor thing. --- 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 #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-11 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2117 I will test and give you feed back soon! --- 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 #2114: [ZEPPELIN-2234][BUG] Can't display the same chart agai...

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2114 @Leemoonsoo Thanks for caring. The implementation we talked in #2110 is not applied yet. I will modify and comment soon. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #2117: ZEPPELIN-2241: JDBC interpreter throws npe on connecti...

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2117 @randerzander Hi, you can restart the failed job again using the `restart` button. If you close / open this PR after all jobs pass, you can see this ![image](https

[GitHub] zeppelin issue #2118: [ZEPPELIN-2242] Toggle helium type buttons

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2118 @AhyoungRyu Thanks 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 #1582: [ZEPPELIN-1604] Add Neo4j interpreter and Network visu...

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1582 Then, clicking `restart` button in the failed job would work. --- 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 #2098: [ZEPPELIN-2217][WIP] AdvancedTransformation for Visual...

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 Some features, improvements which can be done in this PR - refine `transform()` return value - selectable dynamic parameter. `{ ..., name: "theme" paramType: "opt

[GitHub] zeppelin issue #2120: WIP: ZEPPELIN-1492

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2120 Hi @FRosner. Thanks for contribution. - Zeppelin build system relies on travis. So please setup your travis account and travis project for Zeppelin. - https://zeppelin.apache.org

[GitHub] zeppelin pull request #2119: [ZEPPELIN-2243] Use uppercase for shortcut desc...

2017-03-10 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2119 [ZEPPELIN-2243] Use uppercase for shortcut description ### What is this PR for? Use uppercase for keyboard shortcut description so that user can recognize shortcuts easily. it's

[GitHub] zeppelin pull request #2119: [ZEPPELIN-2243] Use uppercase for shortcut desc...

2017-03-10 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2119 --- 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 #2108: [ZEPPELIN-2202] Disable personalized mode btn when not...

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2108 This is the same fix for branch-0.7 - https://github.com/apache/zeppelin/pull/2115/files --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #2118: [ZEPPELIN-2242] Toggle helium type buttons

2017-03-10 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2118 [ZEPPELIN-2242] Toggle helium type buttons ### What is this PR for? Toggle helium type buttons so that user can distinguish which button is clicked easily. Previously, - it's

[GitHub] zeppelin pull request #2118: [ZEPPELIN-2242] Toggle helium type buttons

2017-03-10 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2118 --- 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 #2110: [ZEPPELIN-2234][WIP] Can't display the same chart agai...

2017-03-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2110 Thanks @Leemoonsoo @prabhjyotsingh! --- 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 #2110: [ZEPPELIN-2234][WIP] Can't display the same chart agai...

2017-03-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2110 I'v just updated code. @Leemoonsoo Could you review again? Thanks! --- 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

[GitHub] zeppelin pull request #2118: [ZEPPELIN-2242] Toggle helium type buttons

2017-03-09 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2118 [ZEPPELIN-2242] Toggle helium type buttons ### What is this PR for? Toggle helium type buttons so that user can distinguish which button is clicked easily. Previously, - it's

[GitHub] zeppelin pull request #2116: [ZEPPELIN-2179] "clear output" paragraph doesn'...

2017-03-09 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2116 [ZEPPELIN-2179] "clear output" paragraph doesn't work in personalized mode (branch-0.7) ### What is this PR for? `clear output` (`cmd` + `opt` + `L`) doesn't work in the pe

[GitHub] zeppelin issue #2110: [ZEPPELIN-2234][BUG] Can't display the same chart agai...

2017-03-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2110 @Leemoonsoo Thanks for review. Then I will revert 1bdab41 and #2092 while trying to use `ng-show` instead of. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin pull request #2109: [ZEPPELIN-2231] Helium bundle should be able to...

2017-03-09 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2109 [ZEPPELIN-2231] Helium bundle should be able to import JSON ### What is this PR for? Helium bundle should be able to import JSON to use some libraries which require JSON. For example

[GitHub] zeppelin pull request #2108: [ZEPPELIN-2202] Disable personalized mode btn w...

2017-03-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2108 --- 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 #2105: [ZEPPELIN-2215] Progress bar for Spell executio...

2017-03-09 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2105 [ZEPPELIN-2215] Progress bar for Spell execution ### What is this PR for? Add progress bar for spell execution. - Used `RUNNING` state to display progress bar - Decided

[GitHub] zeppelin pull request #2105: [ZEPPELIN-2215] Progress bar for Spell executio...

2017-03-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2105 --- 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 #2108: [ZEPPELIN-2202] Disable personalized mode btn w...

2017-03-09 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2108 [ZEPPELIN-2202] Disable personalized mode btn when note is running (master) ### What is this PR for? Disable the personalized mode button when a note is running

[GitHub] zeppelin issue #2113: [ZEPPELIN-2190] Support custom web development port

2017-03-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2113 I will test and give you feedback soon. --- 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 #1582: [ZEPPELIN-1604] Add Neo4j interpreter and Network visu...

2017-03-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1582 ``` Zeppelin start [ OK ] < HTTP/1.1 200 OK Exception encountered when invoking run on a nested suite - The code passed to eventually never returned norma

[GitHub] zeppelin pull request #2110: [ZEPPELIN-2234][BUG] Can't display the same cha...

2017-03-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2110 --- 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 #2110: [ZEPPELIN-2234][BUG] Can't display the same cha...

2017-03-09 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2110 [ZEPPELIN-2234][BUG] Can't display the same chart again (master) ### What is this PR for? Can't display the same chart again. I attached a screenshot. - This should

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-07 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 @AhyoungRyu Thanks! I didn't attach screenshots for examples which use `aggregator: false,`. I will prepare in few days. --- If your project is set up for it, you can reply

[GitHub] zeppelin pull request #2110: [ZEPPELIN-2234][BUG] Can't display the same cha...

2017-03-07 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2110 [ZEPPELIN-2234][BUG] Can't display the same chart again (master) ### What is this PR for? Can't display the same chart again. I attached a screenshot. - This should be backported

[GitHub] zeppelin issue #2060: [ZEPPELIN-2094] Decrease npm install retry time

2017-03-07 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2060 Hi @echarles. I can install local (not published) spell modules even in current master. Could you describe more about problem? --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2095 @FRosner https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/HeliumBundleFactory.java#L91-#L95 --- If your project is set up

[GitHub] zeppelin pull request #2109: [ZEPPELIN-2231] Helium bundle should be able to...

2017-03-07 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2109 [ZEPPELIN-2231] Helium bundle should be able to import JSON ### What is this PR for? Helium bundle should be able to import JSON to use some libraries which require JSON. For example

[GitHub] zeppelin issue #2108: [ZEPPELIN-2202] Disable personalized mode btn when not...

2017-03-07 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2108 @jongyoul Can you help review this? --- 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 #2108: [ZEPPELIN-2202] Disable personalized mode btn w...

2017-03-07 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2108 [ZEPPELIN-2202] Disable personalized mode btn when note is running (master) ### What is this PR for? Disable the personalized mode button when a note is running

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2095 Then, What @NohSeho fixed is not related with that part. It's the delay caused by maven-frontend-plugin while downloading nodejs, npm. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #2095: [ZEPPELIN-2094] Decrease npm install retry time (for b...

2017-03-07 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2095 Hi @FRosner Could you specify which log is the last line (the last printed) before hang? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

<    3   4   5   6   7   8   9   10   11   12   >