Github user vinayakshedgeri closed the pull request at:
https://github.com/apache/zeppelin/pull/3247
---
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3246
Ready to review. Failed tests might be related to #3243
---
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3247
@vinayakshedgeri I think you can solve this inconvenience by changing the
base of PR to `branch-0.8`
---
GitHub user vinayakshedgeri opened a pull request:
https://github.com/apache/zeppelin/pull/3247
[ZEPPELIN-3878] Unable to load zeppelin 0.8.0 note at IE 11
### What is this PR for?
Previous version of the zeppelin supports IE 11 when we updated to
zeppelin 0.8.0, Zeppelin web
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3243
Any update?
---
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/3246
ZEPPELIN-3892 Make SecurityUtils as a service
### What is this PR for?
Making SecurityService more configurable. Currently, SecurityUtils consists
of static methods, and which means it's
Github user liuxunorg commented on the issue:
https://github.com/apache/zeppelin/pull/3245
Hi @zjffdu , This bug fix contains test cases that are asynchronous to the
runParagraph API. Should it be helpful for the zeppelin, or should the PR be
merged first?
When you submit a new
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/3245
Hi @liuxunorg I would prefer to add another method
```
public boolean runParagraph(String noteId,
String paragraphId,
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/3239
@monsieurp I though about it again, and feel that providing 2 properties is
very confusing to users. I think it is better to give one property
(zeppelin.jdbc.maxRows) what is user care about.
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/3245
merging if no more comment
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
@zjffdu: Sorry for taking so long to respond. It's been a busy week. I've
rebased my PR as requested.
---
Github user conker84 commented on the issue:
https://github.com/apache/zeppelin/pull/3244
Can you give me a ping when it's ready?
---
Github user liuxunorg commented on the issue:
https://github.com/apache/zeppelin/pull/3245
Hi @felixcheung , I added a test case. Please help me review the code,
thank you!
---
13 matches
Mail list logo