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

2017-03-07 Thread 1ambda
GitHub user 1ambda opened 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 #2097: [MINOR][ZEPPELIN-2154] Enable work in the web dev mode...

2017-03-06 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2097 Tested and works as expected. --- 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 #2077: [ZEPPELIN-2191] Helium package should be able t...

2017-03-06 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2077 --- 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-06 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 #2077: [ZEPPELIN-2191] Helium package should be able t...

2017-03-06 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2077 [ZEPPELIN-2191] Helium package should be able to import css ### What is this PR for? Helium package should be able to import css. For example, - Someone want to create spell

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

2017-03-06 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 #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

2017-03-05 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2042 I think bumping nvm3 version causes CI failure in profile 2, 3 and 6 - https://travis-ci.org/apache/zeppelin/builds/207845221 - https://travis-ci.org/apache/zeppelin/builds/208059749

[GitHub] zeppelin issue #2077: [ZEPPELIN-2191] Helium package should be able to impor...

2017-03-05 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2077 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 enabled

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

2017-03-05 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2095 Thanks @NohSeho. It would be nice if a description for reproducing this issue in 0.7.0 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't transfile im...

2017-03-04 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2071 @Leemoonsoo CI is green: https://travis-ci.org/1ambda/zeppelin/builds/207631806 --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't trans...

2017-03-04 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2071 [ZEPPELIN-2166] HeliumBundleFactoty can't transfile imported es6+ ### What is this PR for? Currently, we don't use any preset. This cause error messages like when a helium package

[GitHub] zeppelin pull request #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't trans...

2017-03-04 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2071 --- 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 #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't trans...

2017-03-04 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2071 [ZEPPELIN-2166] HeliumBundleFactoty can't transfile imported es6+ ### What is this PR for? Currently, we don't use any preset. This cause error messages like when a helium package

[GitHub] zeppelin pull request #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't trans...

2017-03-04 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2071 --- 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 #2060: [ZEPPELIN-2094] Decrease npm install retry time

2017-03-04 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2060 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 #2037: [ZEPPELIN-2109][2110] Sortable Helium pkgs & Introduce...

2017-03-02 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2037 Tested and works as described. - LGTM except https://github.com/apache/zeppelin/pull/2037#pullrequestreview-24893904 --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #2074: ZEPPELIN-2148: On creation of Bar graph zeppelin UI sh...

2017-03-02 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2074 @prabhjyotsingh sorry for late reply :sob: - Tested and works well as described. - `ng-if` is great implementation!, since they will re-draw - It would be nice if we apply

[GitHub] zeppelin issue #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't transfile im...

2017-02-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2071 @AhyoungRyu It's because invalid `.babelrc`. - https://github.com/1ambda/zeppelin-advanced-transformation/blob/master/.babelrc Thanks for detailed review! --- If your project

[GitHub] zeppelin issue #2077: [ZEPPELIN-2191] Helium package should be able to impor...

2017-02-28 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2077 @AhyoungRyu Thanks for cool review 😃 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] zeppelin issue #1887: [ZEPPELIN-1940] most of eslint rules are NOT applied a...

2017-02-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1887 @AhyoungRyu Thanks for caring this! - It would be easier to start from scratch i think. 😭 - I will work on after #1982 is handed! --- If your project is set up for it, you can

[GitHub] zeppelin issue #2074: ZEPPELIN-2148: On creation of Bar graph zeppelin UI sh...

2017-02-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2074 Sure. 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

[GitHub] zeppelin issue #2077: [ZEPPELIN-2191] Helium package should be able to impor...

2017-02-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2077 CI failed, but irrelevant. - build profile: https://travis-ci.org/1ambda/zeppelin/builds/206002571 - failed log1: https://s3.amazonaws.com/archive.travis-ci.org/jobs/206002573/log.txt

[GitHub] zeppelin pull request #2077: [ZEPPELIN-2191] Helium package should be able t...

2017-02-27 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2077 [ZEPPELIN-2191] Helium package should be able to import css ### What is this PR for? Helium package should be able to import css. For example, - Someone want to create spell

[GitHub] zeppelin issue #2037: [ZEPPELIN-2109][2110] Sortable Helium pkgs & Introduce...

2017-02-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2037 I will review and comment soon. Great Feature! --- 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 #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't transfile im...

2017-02-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2071 CI failed, but irrelevant. - profile: https://travis-ci.org/1ambda/zeppelin/builds/205664772 - failed log1: https://s3.amazonaws.com/archive.travis-ci.org/jobs/205664774/log.txt

[GitHub] zeppelin issue #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1982 I'v just rebased to resolve conflict ## Newly added commits - Remove `local` values in helium package jsons: https://github.com/apache/zeppelin/pull/1982/commits

[GitHub] zeppelin issue #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

2017-02-26 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2054 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 #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't trans...

2017-02-26 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2071 [ZEPPELIN-2166] HeliumBundleFactoty can't transfile imported es6+ ### What is this PR for? Currently, we don't use any preset. This cause error messages like when a helium package

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

2017-02-24 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2060 Thanks for improvement! I will review and comment 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

[GitHub] zeppelin issue #2053: [ZEPPELIN-1964] Layout info is lost after refresh

2017-02-21 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2053 I will review and give you feedback soon. Thanks for fixing bug @astroshim! --- 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 #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

2017-02-21 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2054 I will review and comment soon. Thanks for fixing bug @soralee! --- 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 #2015: [ZEPPELIN-2103] Unnecessary read to Helium registry

2017-02-21 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2015 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 have this feature enabled and wishes so

[GitHub] zeppelin pull request #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-21 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1982 [ZEPPELIN-2069] Helium Package Configuration ### What is this PR for? Supporting helium package configurations. I attached screenshots. Implementation details

[GitHub] zeppelin pull request #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-21 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1982 --- 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 #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-20 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1982 just rebased to apply new CI system. ### Newly added commits - Close config panel after saving: https://github.com/apache/zeppelin/pull/1982/commits

[GitHub] zeppelin issue #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-18 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1982 @Leemoonsoo cause I added config before to test :( https://github.com/1ambda/zeppelin-echo-spell/blob/master/package.json#L13-#L18 --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-18 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1982 @Leemoonsoo Thanks for review. - Then we need to close the configuration panel after saving. I will update it. - You have to use echo spell provided by zeppelin (`examples

[GitHub] zeppelin issue #2016: [ZEPPELIN-2105] yarn installed by frontend-maven-plugi...

2017-02-18 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2016 @prabhjyotsingh - tested and it works well. - I'v changed implementation of this PR according to your suggestion. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #2016: [ZEPPELIN-2105] yarn installed by frontend-maven-plugi...

2017-02-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2016 @prabhjyotsingh Sure. 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

[GitHub] zeppelin issue #1902: [ZEPPELIN-1821] Add HTTP client to elasticsearch inter...

2017-02-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1902 @jaspinderdineout This was merged into master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] zeppelin issue #1995: [ZEPPELIN-2086] Move zeppelin-web related test files i...

2017-02-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1995 Thanks @AhyoungRyu. This definitely help to write and check test easily. --- 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 #2015: [ZEPPELIN-2103] Unnecessary read to Helium registry

2017-02-16 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2015 I will review this 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 enabled

[GitHub] zeppelin issue #2016: [ZEPPELIN-2105] yarn installed by frontend-maven-plugi...

2017-02-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2016 Hope to merge into 0.7.0 branch as well to help 0.7.1+ users --- 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 #1995: [ZEPPELIN-2086] Move zeppelin-web related test files i...

2017-02-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1995 @AhyoungRyu Please 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 issue #1982: [WIP][ZEPPELIN-2069] Helium Package Configuration

2017-02-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1982 But current implementation is really inefficient in terms of API calls. I will refactor it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1982: [ZEPPELIN-2069] Helium Package Configuration

2017-02-15 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1982 @Leemoonsoo I'v just updated this PR. - Using `artifact` as config - https://github.com/apache/zeppelin/pull/1982/commits/6e91b78880777ace0e780c746e376a8e0c88b520

[GitHub] zeppelin pull request #2016: [ZEPPELIN-2105] yarn installed by frontend-mave...

2017-02-14 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2016 [ZEPPELIN-2105] yarn installed by frontend-maven-plugin causes build failure (BUG) ### What is this PR for? yarn installed by frontend-maven-plugin causes build failure **when

[GitHub] zeppelin pull request #2016: [ZEPPELIN-2105] yarn installed by frontend-mave...

2017-02-14 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2016 --- 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 #2016: [ZEPPELIN-2105] yarn installed by frontend-maven-plugi...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2016 @felixcheung Yes, build time increase 40 secs ~ 1 min approximately per build profile. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1998: [ZEPPELIN-2070][DOCS] Create Docs for Spell

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1998 CI failed, but irrelevant. --- 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 #2016: [ZEPPELIN-2105] yarn installed by frontend-mave...

2017-02-13 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2016 [ZEPPELIN-2105] yarn installed by frontend-maven-plugin causes build failure (BUG) ### What is this PR for? yarn installed by frontend-maven-plugin causes build failure **when

[GitHub] zeppelin pull request #2016: [ZEPPELIN-2105] yarn installed by frontend-mave...

2017-02-13 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2016 --- 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 #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra - I think we don't need to worry about **max call stack size** since it occurs in unusual env like windows, *linux on vm*. - Zeppelin is not the thing like JDK

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 So, what you want to solve is **building zeppelin in vms which can't have enough symlinks**? right? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra I think the log you'v just posted is quite different. It's not for build. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 Regrading to **max call stack size** what @DmytroShkvyra mentioned in above comment (for other people) - https://github.com/npm/npm/issues/9953 - https://github.com/npm/npm/issues

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra Should we support build env such as windows on vm? --- 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 #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra Really sorry if you feel like that. But IMO - you didn't try almost all possible alternatives - you didn't try to understand my comments at all. I repeatedly

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra Please read and try #2016 ### Details - yarn installed by frontend-maven-plugin can't run commands described in scripts tag (package.json) - but npm is able

[GitHub] zeppelin pull request #2016: [ZEPPELIN-2105] yarn installed by frontend-mave...

2017-02-13 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2016 [ZEPPELIN-2105] yarn installed by frontend-maven-plugin causes build failure (BUG) ### What is this PR for? yarn installed by frontend-maven-plugin causes build failure **when

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra That's cool. You are trying to modify something without understanding. Anyway, We have 2 problems. 1. local development - you must install yarn if you want to use

[GitHub] zeppelin issue #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs published...

2017-02-13 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2006 @Leemoonsoo Someday - what about black? - we might use labels such as `local` and `online` for helium packages. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1991: [ZEPPELIN-2081] Prettify input, textarea in interprete...

2017-02-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1991 @felixcheung Fixed. (`font-size: 12`) ![image](https://cloud.githubusercontent.com/assets/4968473/22870210/cafa98b6-f1e8-11e6-9b7f-d2e2b5323ac3.png) --- If your project is set up

[GitHub] zeppelin issue #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 @DmytroShkvyra If you want to use node modules in terminal, you have to install them globally. (e.g `$ yarn install`) But in case of grunt (and karma so on..), we don't need

[GitHub] zeppelin pull request #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-12 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1995 [ZEPPELIN-2086] Move zeppelin-web related test files into src/ for efficiency ### What is this PR for? Since we are using a separated test directory, - it's uncomfortable

[GitHub] zeppelin issue #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs published...

2017-02-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2006 Or are there other ways to figure out local package? instead of `return pkg.artifact && !pkg.artifact.includes('@');` --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-12 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1991 [ZEPPELIN-2081] Prettify input, textarea in interpreter setting page ### What is this PR for? Fix small details in interpreter setting page about inputboxes, textareas - Set

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-12 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1991 --- 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 #1998: [ZEPPELIN-2070][DOCS] Create Docs for Spell

2017-02-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1998 @AhyoungRyu Thanks for review. I'v just addressed your comments. --- 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 #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs published...

2017-02-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2006 @AhyoungRyu Thanks for review. I'v changed the condition for checking local package. --- 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 #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-11 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1995 --- 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 #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-11 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1995 [ZEPPELIN-2086] Move zeppelin-web related test files into src/ for efficiency ### What is this PR for? Since we are using a separated test directory, - it's uncomfortable

[GitHub] zeppelin pull request #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs pu...

2017-02-11 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2006 --- 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 #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs pu...

2017-02-11 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2006 [ZEPPELIN-2099] Add NPM link for helium pkgs published in npm ### What is this PR for? Add NPM link for helium pkgs published in npm while coloring grey for local packages

[GitHub] zeppelin pull request #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs pu...

2017-02-11 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2006 [ZEPPELIN-2099] Add NPM link for helium pkgs published in npm ### What is this PR for? Add NPM link for helium pkgs published in npm while coloring grey for local packages

[GitHub] zeppelin pull request #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs pu...

2017-02-11 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2006 --- 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 #2006: [ZEPPELIN-2099] Add NPM link for helium pkgs pu...

2017-02-11 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2006 [ZEPPELIN-2099] Add NPM link for helium pkgs published in npm ### What is this PR for? Add NPM link for helium pkgs published in npm while coloring grey for local packages to distinguish

[GitHub] zeppelin pull request #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-11 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1995 [ZEPPELIN-2086] Move zeppelin-web related test files into src/ for efficiency ### What is this PR for? Since we are using a separated test directory, - it's uncomfortable

[GitHub] zeppelin pull request #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-11 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1995 --- 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 #1998: [ZEPPELIN-2070][DOCS] Create Docs for Spell

2017-02-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1998 Resolved @soralee's comments. --- 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 #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-10 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1991 --- 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 #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-10 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1991 [ZEPPELIN-2081] Prettify input, textarea in interpreter setting page ### What is this PR for? Fix small details in interpreter setting page about inputboxes, textareas - Set

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-10 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1991 --- 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 #1991: [ZEPPELIN-2081] Prettify input, textarea in interprete...

2017-02-10 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1991 @AhyoungRyu thanks for review. CI failed but irrelavent --- 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 pull request #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1995 --- 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 #1997: [MINOR][ZEPPELIN-2087] Make each Zeppelin conf propert...

2017-02-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1997 Tested and works as expected. --- 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 #2000: [gh-pages] update contribution/webapplication.html

2017-02-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2000 Looks nice :) --- 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 #2002: [ZEPPELIN-1928]: Get a rid of preinstalled external de...

2017-02-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2002 FYI - https://github.com/eirslett/frontend-maven-plugin/issues/562 --- 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 #2000: [gh-pages] update contribution/webapplication.html

2017-02-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2000 Hi @soralee :) I would like to suggest using `'ngInject'l` instead of `$inject = ['$location'];` as like https://github.com/apache/zeppelin/pull/1994 > http://zeppelin.apache.

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1991 --- 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 #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-09 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1991 [ZEPPELIN-2081] Prettify input, textarea in interpreter setting page ### What is this PR for? Fix small details in interpreter setting page about inputboxes, textareas - Set

[GitHub] zeppelin pull request #1995: [ZEPPELIN-2086] Move zeppelin-web related test ...

2017-02-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1995 --- 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 #1994: [MINOR] Replace angular inject with ngInject

2017-02-09 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1994 --- 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 #1994: [MINOR] Replace angular inject with ngInject

2017-02-09 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1994 [MINOR] Replace angular inject with ngInject ### What is this PR for? Replaced angular inject with ngInject to reduce duplication and keep simple. There is no reason to use angular

[GitHub] zeppelin pull request #1998: [ZEPPELIN-2070][DOCS] Create Docs for Spell

2017-02-08 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1998 [ZEPPELIN-2070][DOCS] Create Docs for Spell ### What is this PR for? Docs for Spell which was added by #1940 ### What type of PR is it? [Documentation] ### Todos

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-08 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1991 [ZEPPELIN-2081] Prettify input, textarea in interpreter setting page ### What is this PR for? Fix small details in interpreter setting page about inputboxes, textareas - Set

[GitHub] zeppelin pull request #1994: [MINOR] Replace angular inject with ngInject

2017-02-08 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/1994 [MINOR] Replace angular inject with ngInject ### What is this PR for? Replaced angular inject with ngInject to reduce duplication and keep simple. There is no reason to use angular

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-08 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1991 [ZEPPELIN-2081] Prettify input, textarea in interpreter setting page ### What is this PR for? Fix small details in interpreter setting page about inputboxes, textareas - Set

[GitHub] zeppelin pull request #1991: [ZEPPELIN-2081] Prettify input, textarea in int...

2017-02-08 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1991 --- 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 #1989: [ZEPPELIN-2073]Replace AngularObjectRegistry by Resour...

2017-02-08 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1989 Hi @qinchaofeng. I think you pushed different branch. Could you check again? --- 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

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