Github user FireArrow commented on the issue:
https://github.com/apache/zeppelin/pull/2544
Any updates on this?
---
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
If there are no comments anymore, I will merge to master on this pr.
---
Github user dwhsys commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Can someone review please?
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2568
@malayhm What about this icon?
- http://fontawesome.io/icon/bars/
Github user malayhm commented on the issue:
https://github.com/apache/zeppelin/pull/2568
@1ambda This is generally used for hamburger menu in mobile mode.
What do you think about a custom icon like this, this is used on Facebook?
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2449
Ready to review (red for other tests )
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2578
LGTM, just one concern about `PythonCondaInterpreterTest.java`. It is using
mock which may not found potential bugs. I think we can do unit without mock.
But anyway this could be done in a followin
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
@zjffdu Thank you a lot :)
---
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2576
This is a hot-fix change.
Therefore, they merge quickly.
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2578
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2568
@malayhm it's nice if we can remove `More` text and there is no license
problem ð
---
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/2580
---
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/2580
[ZEPPELIN-2924] Fix broken javascript source map
### What is this PR for?
Fixed broken javascript source map. The coverage loader should be used only
for test env.
**that's rea
Github user malayhm commented on the issue:
https://github.com/apache/zeppelin/pull/2568
@1ambda It seems this icon is available in icon finder with free commercial
use:
https://www.iconfinder.com/icons/476329/continued_detail_details_ellipses_more_icon#size=16
What do yo
Github user malayhm commented on the issue:
https://github.com/apache/zeppelin/pull/2568
@1ambda Please see the
https://user-images.githubusercontent.com/1881135/30287842-2e72d740-9744-11e7-8468-f9a86766487b.png";>
new screenshot with the new icon:
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2568
@Madhuka Happy to see the icon.
I just searched the license but couldn't find the license. It only says
"free for commercial usage". Could you check it can be used even in opensource?
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2581
Fix/jdbc connection release 0.7
### What is this PR for?
This PR fixes JDBC connection release problem.
for example whenever i run not executable command like following
