[GitHub] zeppelin pull request #2157: [HOTFIX] Fix CI

2017-03-18 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2157 [HOTFIX] Fix CI ### What is this PR for? CI is failing since https://github.com/apache/zeppelin/commit/970b8117a48a31a9375bf7f76142117fd9b3bd86 This PR tries to make CI green

[GitHub] zeppelin issue #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread guicaro
Github user guicaro commented on the issue: https://github.com/apache/zeppelin/pull/2154 @AhyoungRyu just glanced through comments in #2156 I also wondered why changes in check boxes did not trigger a run when placed next to a select form and with the `Run on selection change` glad

[GitHub] zeppelin pull request #2157: [HOTFIX] Fix CI

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

[jira] [Created] (ZEPPELIN-2284) Handle angularObjects in angularObjectRegistry on interpreter restart.

2017-03-18 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-2284: -- Summary: Handle angularObjects in angularObjectRegistry on interpreter restart. Key: ZEPPELIN-2284 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2284

[GitHub] zeppelin issue #2157: [HOTFIX] Fix CI

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2157 CI became green. I'm merging it to master and branch-0.7 as a hotfix. Created issue https://issues.apache.org/jira/browse/ZEPPELIN-2284 to make sure remove AngularObject on

[GitHub] zeppelin issue #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select forms be...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2156 @Leemoonsoo Thanks! CI has been passed after rebasing from `branch-0.7`. Will merge into `branch-0.7` as a hotfix if there are no more comments on this. --- If your project is set up for

[GitHub] zeppelin issue #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2154 @guicaro Just noticed that the behaviour that you mentioned was intended one. In Zeppelin, we support `editOnDoubleClick` \w Markdown interpreter. With this feature, you can hide/show the

[GitHub] zeppelin pull request #2157: [HOTFIX] Fix CI

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/2157 --- 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 #2157: [HOTFIX] Fix CI

2017-03-18 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/2157 [HOTFIX] Fix CI ### What is this PR for? CI is failing since https://github.com/apache/zeppelin/commit/970b8117a48a31a9375bf7f76142117fd9b3bd86 This PR tries to make CI green

[GitHub] zeppelin issue #2145: [ZEPPELIN-2271] encoding password of credentials

2017-03-18 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2145 I'm concerned that this would give people a false sense of security... --- 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 #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2154 @guicaro Seems sth screwed up in your git. It's okay I can try to reproduce it now :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #2148: [branch-0.7][ZEPPELIN-2060] Make dynamic select form t...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2148 It was merged yesterday but seems wan't closed automatically by asfgit. So I'm closing this manually now. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin pull request #2148: [branch-0.7][ZEPPELIN-2060] Make dynamic select...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu closed the pull request at: https://github.com/apache/zeppelin/pull/2148 --- 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 #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 #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2154 FYI, quick show for `edit on double click` with markdown interpreter ![edit on double

Re: [CI] second job exceeds travis.org max job time

2017-03-18 Thread moon soo Lee
Nowadays, the second and the third profile takes around 40~45min. Which is very close to limit of travis 50min. I've seen sometimes second or the third profile just fails because of it takes longer than usual (by slow network download, etc). I think we need to split second/third profile so the

[GitHub] zeppelin issue #2152: [MINOR] Display chart by default in basic tutorial not...

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2152 Merge to master and branch-0.7 if no more 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

[GitHub] zeppelin issue #2085: [ZEPPELIN-1999] get interpreter property with replaced...

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2085 Thanks @tinkoff-dwh for useful new feature. 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

[GitHub] zeppelin pull request #2154: [HOT FIX][MASTER] Fix multi dynamic select form...

2017-03-18 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/2154 [HOT FIX][MASTER] Fix multi dynamic select forms behaviour ### What is this PR for? After #2100 merged, we can control the behaviour of running select form using `Run on selection change`

[jira] [Created] (ZEPPELIN-2283) Support 'compatibility' field in helium.json

2017-03-18 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2283: --- Summary: Support 'compatibility' field in helium.json Key: ZEPPELIN-2283 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2283 Project: Zeppelin Issue

Re: [CI] second job exceeds travis.org max job time

2017-03-18 Thread Jongyoul Lee
I found same issue. In my case, Zeppelin server won't run. I'm not sure why it happens. On Sat, Mar 18, 2017 at 2:17 AM, Remy Gayet wrote: > nevermind. One of my test is hanging because of a bug somewhere in my code. > > On Fri, Mar 17, 2017 at 12:15 PM, Remy Gayet

[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 #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select f...

2017-03-18 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/2156 [HOTFOX][Branch-0.7] Fix multi dynamic select forms behaviour ### What is this PR for? This PR is same with #2154 with base branch `branch-0.7`. Please refer to #2154's

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

2017-03-18 Thread asfgit
Github user asfgit 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 is

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

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2141 @guicaro I feel so sorry to you since it was my bad. Just created hotfix patch #2154 to fix below two issues that you reported. - Select form is not shown up when multiple dynamic forms

[GitHub] zeppelin pull request #2131: ZEPPELIN-2256. poll job progress is called twic...

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

[GitHub] zeppelin issue #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2154 @guicaro Thank you for the quick checking! I made some change in #2156(it's base branch is `branch-0.7` while this PR was created on `master`) for the checkbox. The every checking the

[GitHub] zeppelin issue #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2154 2nd and 3rd build jobs keep failed even after restarting them several times. And the reasons are as below ``` 1. Restart Zeppelin 2. Create interpreter setting in 'Interpreter'

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

2017-03-18 Thread guicaro
Github user guicaro commented on the issue: https://github.com/apache/zeppelin/pull/2141 @AhyoungRyu no worries, will be glad to review your work, fix `testMultipleDynamicFormsSameType` and fox my commits, will definetly ping you about this last issue :) Thanks --- If your project

[GitHub] zeppelin issue #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select forms be...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2156 @Leemoonsoo You mean the `checkbox` dynamic form? The `Run on selection change` is only for `select form`. Do you think this can make user confused? --- If your project is set up

[GitHub] zeppelin pull request #2155: [HOTFIX] Dynamic form in python interpreter

2017-03-18 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2155 [HOTFIX] Dynamic form in python interpreter ### What is this PR for? https://github.com/apache/zeppelin/pull/2106 rewrote python interpreter. But dynamic form feature is not rewritten

[GitHub] zeppelin issue #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select forms be...

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2156 When "Run on selection change" is checked, check box change does not run paragraph. @AhyoungRyu can you check? --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select forms be...

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2156 Yes, i think user will expect that same behavior `dropdown` and `checkbox`. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select forms be...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2156 @Leemoonsoo Yeah it makes sense. So I updated again as you said. - When two types of dynamic form in the paragraph

[GitHub] zeppelin issue #2154: [HOT FIX][MASTER] Fix multi dynamic select forms behav...

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2154 The error is not related to this change. I'm working on the CI error. --- 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 #2156: [HOTFOX][Branch-0.7] Fix multi dynamic select forms be...

2017-03-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2156 Tested and 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

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

2017-03-18 Thread guicaro
Github user guicaro commented on the issue: https://github.com/apache/zeppelin/pull/2141 @AhyoungRyu I'm currently looking at why testMultipleDynamicFormsSameType is failing. It now fails after I rebased from master. It's the test case that includes two select forms in one single

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

2017-03-18 Thread guicaro
Github user guicaro commented on the issue: https://github.com/apache/zeppelin/pull/2141 Quick question, what is the behavior when we have more than 2 select forms? Shouldn't we also have a **"Run on selection"** check box? I believe this was one of the pain points for some (see

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

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2141 @guicaro Thanks for letting me know. Let me 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 project does

[GitHub] zeppelin issue #2148: [branch-0.7][ZEPPELIN-2060] Make dynamic select form t...

2017-03-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2148 CI failed in two cases with below reason which is irrelevant with this change. ``` No output has been received in the last 10m0s, this potentially indicates a stalled build or