Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2251
@tinkoff-dwh Since conflicts are about lint errors, you can simply rebase by
1. keeping your change and run `rebase --continue`
2. then, execute `eslint src --fix` in `zeppelin-web
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2265
@Leemoonsoo 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 user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2265
[ZEPPELIN-2427] Hide navbar and noteAction bar when scrolling down
### What is this PR for?
Hide navbar and noteAction bar when scrolling down to
- **widen view area
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/2252
[ZEPPELIN-1940] lint rule set is NOT applied at all.
### What is this PR for?
eslint ruleset is NOT applied at all due to invalid `.eslintrc`.
This PR includes
- fixes
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/2252
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
rebase to resolve conflict with #2245
---
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 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2264
[ZEPPELIN-1711] Create `Dockerfiles` for released bin
### What is this PR for?
Created `Dockerfile` for released bin
- based on ubuntu for desktop usage
- JDK 8
- Python
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
rebased to resolve conflict with #2244
---
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 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1689
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2260
@soralee 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
enabled
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2261
@tinkoff-dwh 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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
rebased to resolve conflict with #2133
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2253
@jongyoul ping
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1761
Let me create new PR which including only 1 OS (ubuntu) based on this PR.
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2225
Submitted #2261
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2259
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 user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2261
[ZEPPELIN-2420] Slow notebook listing in navbar
### What is this PR for?
Notebook listing is slow as you can see in the screenshot below.
The problem was,
- angular
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2260
[ZEPPELIN-2408] Should use $uibTooltip instead of $tooltip (master)
### What is this PR for?
Since we are using angular-bootstrap 2.5+, we should use `$uibTooltip`
instead of `$tooltip
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2256
Old version zeppelins will not render converted note.json. On the other
hand, if we keep 'result', they can render last version of old format. Is it ok?
---
If your project is set up for it, you
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2251
One suggestion. What about calling them as 'widget' or something instead of
'type'? cause we might add actual type like 'number', 'string',... as
@leemoonsu mentioned.
---
If your project is set
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2256
Do we need test for 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
enabled
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
rebased to resolve conflict with #2203, #2248
---
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 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2256
[ZEPPELIN-2315] (bug) new note.json is overwritten by old note.json
(master, branch-0.7)
### What is this PR for?
- `note.json` is converted every time
- as a result, changes
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2256
@jongyoul 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 user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/2255
---
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 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/2255
[HOTFIX] Invalid method signature in GroovyInterpreter
### What is this PR for?
- https://github.com/apache/zeppelin/pull/2203 was merged
- but https://github.com/apache/zeppelin
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2255
[HOTFIX] Invalid method signature in GroovyInterpreter
### What is this PR for?
- https://github.com/apache/zeppelin/pull/2203 was merged
- but https://github.com/apache/zeppelin/pull
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
rebased to resolve conflict with #2203, #2248
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2253
@jongyoul 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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2248
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 user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2253
[ZEPPELIN-2179] Clear output DOES NOT work in personalized mode (master)
### What is this PR for?
`clear output` (`cmd` + `opt` + `L`) doesn't work in the personalized mode
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
@AhyoungRyu Thanks for detailed review!
Those things are about regex and mixed boolean condition (without parens),
so I am sure i can keep the original intention. ð..
---
If your
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2248
Works as described. LGTM except 2 comments I left
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
@AhyoungRyu Could 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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2252
@AhyoungRyu Could 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 user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2252
[ZEPPELIN-1940] lint rule set is NOT applied at all.
### What is this PR for?
eslint ruleset is NOT applied at all due to invalid `.eslintrc`.
This PR includes
- fixes
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2250
Using paste mode would be better. We have so many if-else code to
manipulate code which is already covered by paste mode.
If we evaluate using paste mode, evaluation orders
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2125
@conker84 you should rebase based on master, not other branch. Currently,
your change mixed with others.
### What is this PR for?
Icon size of helium vis packages doesn't fit into the container button. I
attached a image
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2225
[ZEPPELIN-2357] fix: tooltip doesnt work with btngroup
### What is this PR for?
tooltip provided by [angular-ui:
bootstrap](https://github.com/angular-ui/bootstrap) doesn't work
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2224
@AhyoungRyu Could 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 user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2224
[ZEPPELIN-2304] Fix vis Icon sizes in helium page (branch-0.7)
### What is this PR for?
Icon size of helium vis packages doesn't fit into the container button. I
attached a image
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2125
It's just my opinion. Thus, it's ok to use d3 if you think it's reasonable
ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2181
@AhyoungRyu I saw your comment now. Let me create a PR for the branch 0.7
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2098
@AhyoungRyu I forgot as well to modify `index.md` ð.
I'v just updated
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2098
@AhyoungRyu resolved all comments. Thanks for detailed 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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2210
Sorry for bothering. Fixed that issue and tested like
- remove `conf/heli.json` and `local-repo/helium-bundle`
- build and restart Zeppelin
- install any bundle (**test 1 to see
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2125
Really sorry for late reply. I missed notification for a while.
- Spell might use JSON output of neo4j interpreter utilizing
[sigma.js](http://sigmajs.org/),
[vis.js](http://visjs.org
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2219
@Leemoonsoo I failed to reproduce it in branch-0.7, so it's ok to just
close this PR if it's save. But code looks like buggy.
---
If your project is set up for it, you can reply to this email
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2219
[ZEPPELIN-2348] Line chart setting is not rendered (branch-0.7)
### What is this PR for?
Line chart setting is not rendered.
### What type of PR is it?
[Bug Fix
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2218
[ZEPPELIN-2348] Line chart setting is not rendered (master)
### What is this PR for?
Line chart setting is not rendered.
### What type of PR is it?
[Bug Fix | Improvement
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2098
@AhyoungRyu Updated docs as well.
---
If your project is set up
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 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 user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/2098
[ZEPPELIN-2217] AdvancedTransformation for Visualization
### What is this PR for?
`AdvancedTransformation` has more detailed options while providing existing
features
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/2098
[ZEPPELIN-2217] AdvancedTransformation for Visualization
### What is this PR for?
`AdvancedTransformation` has more detailed options while providing existing
features
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2098
@AhyoungRyu
1. Hide other buttons when panel is minified
2
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2210
Added summary to the 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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2177
Let me implement locking and realtime notification soon. I will close this
PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
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 user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/2176
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2176
Let me implement locking and realtime notification soon. I will close this
PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2205
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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2199
@markito I mean, this PR not the JIRA ticket :)
---
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 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2210
[ZEPPELIN-2088] Evaluate helium bundle one by one
### What is this PR for?
Evaluate helium bundle one by one. Because
- Bundling them in one file is not good idea because
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2176
@FRosner Thanks for detailed reply.
We can't get the ultimate feature at once.
- locking
- realtime notification
- ...
IMO, This is just the first step. Then other
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2205
Let me rebase and comment 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 feature
enabled
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2199
Great! Then, Please close issue and re-open :) then jenkins will be
triggered again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2205
Thanks for review! @AhyoungRyu, @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 not have this feature
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2199
I am not sure but the failed test is due to one of flaky tests. Please
restart it.
```
Results :
Tests in error
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2202
Great!
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2176
@FRosner If auto-save works properly,
- User B's local change will update server state
- And User A's paragraph text will be updated because he/she doesn't have
local change
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2205
[ZEPPELIN-2330] Helium.html doesn't render spell information only in
production build
### What is this PR for?
Helium.html doesn't render spell information only in production build
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@AhyoungRyu Tested and works as described. 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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2098
@AhyoungRyu
Thanks for detailed review.
1. It would be nice to have some popup or feedback when conf is updated.
2. Let me hide them when panel is minimized (e.g the chart
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2177
@AhyoungRyu Could you help 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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2176
@AhyoungRyu Could you help 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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2098
Let me add docs and unit tests. I guess it will take few days. Then i will
comment. Thanks @Leemoonsoo
---
If your project is set up for it, you can reply to this email and have your
reply
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@AhyoungRyu Yes, it would be great to hide unnecessary things
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2174
@AhyoungRyu more precisely,
1. If we are in page 1, we don't need `<<`
