Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@1ambda It's wired. This is what I can see when I build my branch.
 for now, people only can use their local
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2186
@soralee And as you can see in the above gif image that i took, the tooltip
in scatter chart is shown like "(age, value)". But before #2042 it looks like
below.
![im
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2163
@Leemoonsoo Right it should be passed without refreshing. So I removed that
part and added `waitForParagraph(1, "FINISHED");` before every selections for
the stability. Now CI is
GitHub user AhyoungRyu reopened a pull request:
https://github.com/apache/zeppelin/pull/2174
[ZEPELIN-2209] Add pagination to Helium menu
### What is this PR for?
Like I did in #2126, added pagination to Helium menu in Zeppelin.
10 pkgs will be shown per 1 page
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/2174
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2186
@soralee Thanks for the explanation.
I tested this branch to make sure it works well or not, but when I select
"Line chart" below error ms
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1976
@del680202 Also tested on both latest branch-0.7 and rc2 but couldn't
reproduce. Here is my steps.
1. Activate shiro -> login
2. Run a paragraph with python intp(globall
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2180
CI failure is not related with this change since it's based on `gh-pages`
branch.
Merge into `gh-pages` if there are no more comments.
---
If your project is set up for it, yo
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1976
@del680202 Thanks for your time and effort to test it out.
>But I seem when I edit&save interpreter config then the zombie process
will be occurred
I checked it with
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2134
@tinkoff-dwh Thanks! Looks good :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2186
@soralee Yeah I can see the error when we have large data.
I tested
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2134
> I can add the close button
@tinkoff-dwh Yeah that's better. Having close button will be better for
other cases as well.
---
If your project is set up for it, you can
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2186
@Remilito Will be appreciated if you can take a look 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2186
@soralee Thanks for digging it up!
Seems https://github.com/novus/nvd3/pull/1934 fixed this issue and merged
in Jan. Hopefully they have a plan to release another minor version soon
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1872
@epahomov Could you close this PR please?
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1907
Hi @cuspymd, what do you think about @astroshim 's 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 pr
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2134
Tested and basically agree with @Leemoonsoo. But one thing I just noticed
is this new message can be seen in report mode.
![screen shot 2017-03-24 at 12 20 56
pm](https
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2181
Merge into master and branch-0.7 if there are no more comments on 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2184
Merge into master if there are 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 have
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2185
@Leemoonsoo Thanks!
Merge into master and branch-0.7 if there are no more comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/2185
---
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 user AhyoungRyu reopened a pull request:
https://github.com/apache/zeppelin/pull/2185
[MINOR][ZEPPELIN-2308] Fix vis icon button group
### What is this PR for?
Since "Save" button which shows up when there is change in the package
bundle ordering is includ
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/2185
[MINOR][ZEPPELIN-2308] Fix vis icon button group
### What is this PR for?
Since "Save" button which shows up when there is change in the package
bundle ordering is included in s
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/2184
[MINOR][ZEPPELIN-2306] Improve alignment of components in Helium header
### What is this PR for?
If there are 2 more enabled `VISUALIZATION` pkgs, the ordering control
section will be
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2178
Looks better :)
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2181
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, or if
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2180
@sravan-s Thanks for the suggestion. Addressed in
34dfec8a41a0e998fcc8c030860a6c796d11df86 :)
https://cloud.githubusercontent.com/assets/10060731/24241202/5cdc7386-0ff7-11e7-91ed
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@1ambda I fixed it in the latest commit
bc7643c46dc6690b5b1f1a91249bcf4ee0e93a43. You can simply check whether it was
fixed or not by comparing `INTERPRETER` type of packages are well located
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/2180
[gh-pages][ZEPPELIN-2303] Fix lately published package ordering
### What is this PR for?
I added pagination under Helium package list in
https://zeppelin.apache.org/helium_packages.html
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@1ambda Just noticed that `orderBy: publishedDate` doesn't work well
properly now. Let me updated again and will let you know when it's resolved.
(Actually same problem i
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1976
@del680202
>When I enable authentication with shiro or switch interpreter mode to Per
user & isolated, Restart does not work and it is possible to make zombie
process.
Thi
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2119
Thanks @1ambda
Merge into master and branch-0.7 if there are 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2144
Merge into master if no more discussions.
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2141
Merge into master and branch-0.7 if there are no more comments on 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2098
@1ambda Looks awesome! Let me test it and i'll get back to you 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
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/1339
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1339
I'm closing this PR since there'll be better solution for this (e.g.
[ZEPPELIN-1993](https://issues.apache.org/jira/browse/ZEPPELIN-1993)) :)
---
If your project is set up for i
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2144
Cool! Tested again and it works well as expected. 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@1ambda Will be really appreciated if you can 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
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/2174
[ZEPELIN-2209] Add pagination to Helium menu
### What is this PR for?
Like I did in #2126, added pagination to Helium menu in Zeppelin.
10 pkgs will be shown per 1 page
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2172
@Leemoonsoo Thanks!
CI failure is irrelevant with `gh-pages` change. Merge into gh-pages if
there are no more comments.
---
If your project is set up for it, you can reply to this email
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2163
All CI tests have passed now. Ready for review 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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2161
I tested two scenarios that you described and both(turn on & off Shiro) are
working well.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2163
I rebased but failed again in Selenium test. All test cases that I added
passed in my local but it's failed in here. Seem I need to do something more to
avoid this..
---
If your proje
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2151
@Remilito Thanks for the useful feature.
I think it'll be better to display only essential buttons in the paragraph.
How about placing it under paragraph control dropdown menu inste
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2141
@guicaro There are some unrelated commits in here after you rebased. Can
you try to fix it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2150
@andreaTP There are some conflict files after #2093 merged. Can you resolve
them?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2085
@tinkoff-dwh Saw your latest update. As you did,
```
## Usage
For example in database exist user `user1` with some password and exists
user with same name into
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2155
Tested both with & without this patch. It works well 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
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/2156
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2154
@guicaro Really appreciated your time & effort in the weekend. Sadly i have
no idea what could be the reason for your problem with limited information. But
i think we need only to focu
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2154
@guicaro BTW you don't need to test #2156(base branch: `branch-0.7`)
anymore since I updated this PR(base branch: `master`) as like #2156.
---
If your project is set up for it, you can
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2154
@guicaro Not sure how your local git repo status is now, but if you think
you can't make it back to normal status, I would recommend you to remove local
Zeppelin dir and reclone it. Whe
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2154
After rebasing from master, I restarted 2nd build job several times but it
keeps failed because of timeout.
https://api.travis-ci.org/jobs/212613496/log.txt?deansi=true
```
The job
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 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
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2154
FYI, quick show for `edit on double click` with markdown interpreter
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
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 checkin
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
---
If your project is set up
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2119
@1ambda Sadly one file has some conflicts after #2100 merged..
Could you resolve this? I'll merge this right after then.
---
If your project is set up for it, you can reply to this
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/2148
[branch-0.7][ZEPPELIN-2060] Make dynamic select form turn on or off using
checkbox
### What is this PR for?
This PR is same with #2100 with base branch `branch-0.7
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
@Leemoonsoo Thanks for reveiw.
Merge into branch-0.7 and master if there are no more comments
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2072
I'm closing this PR since
https://issues.apache.org/jira/browse/ZEPPELIN-2060 will be handled in #2100.
---
If your project is set up for it, you can reply to this email and have your
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/2072
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
CI is green now. Ready to review 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 not have this
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/2100
---
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 user AhyoungRyu reopened a pull request:
https://github.com/apache/zeppelin/pull/2100
[ZEPPELIN-2060] Make dynamic select form turn on or off using checkbox
### What is this PR for?
I added "Auto Run" checkbox for select dynamic form to make user turn on /
off
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2134
@tinkoff-dwh Then you can make this CI status green by reopening-closing
this PR :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
CI failed when it runs `yarn run test` and it seems relevant with this
change. Let me update again.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2143
@1ambda Looks good to me ð
It's not in the scope of this PR, but as ppl are getting have more interest
in Helium, i think we need a guide which explains how ppl can add h
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2105
Merge into master if there are no more comment on 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
@Leemoonsoo I made `runOnSelectionChange` can be persisted under
`note.json`'s `config` field like below.
https://cloud.githubusercontent.com/assets/10060731/23981188/fe8a5fbe-0a47
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2119
Merge into branch-0.7 and master if there are 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
@Leemoonsoo Didn't realize that. Thanks for letting me know. Let me update
again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2141
Cool!
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2126
Thanks again @1ambda.
Will merge into gh-pages if there are 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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
CI is also green now :D
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
@Leemoonsoo I added two test cases as you said in this commit:
66c6c21d1bb435f5ade327a8f68cf3e8897e1fb3
- Check "Run on selection change" menu appears or not, when select form i
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2100
@Leemoonsoo @felixcheung Sorry for my late update. To provide more
intuitive label, I changed it from `Auto run` -> `Run on selection change` and
added tooltip msg to let user know they st
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2126
@1ambda @felixcheung Thanks!
I updated the overview sentences as well.
- before
---
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 user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2101
Merge into master and branch-0.7 if there are 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
101 - 200 of 1030 matches
Mail list logo
