Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2925
I am saying using group is not good, it just depends on user's setting.
BTW, it looks like this PR also works for your scenario. In this PR, I would
always choose the owner to run the cronjob. And
Github user r-kamath commented on the issue:
https://github.com/apache/zeppelin/pull/2932
LGTM
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2925
@mebelousov Your solution depends on shiro setting, we could not assume
that in code.
---
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2925
@zjffdu This is your solution ;) I have tested your branch.
Do we have uniform opinion that running document as the group is not good?
---
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2932
Merging this to master and branch-0.8 if no more discussion.
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2930
@mebelousov I don't think this will work for 0.8. In 0.9, I just put py4j
in the code base of zeppelin, that's why I can avoid downloading it in pom.
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2930
@mebelousov I don't think this will work for 0.8. In 0.9, I just put py4j
in the code base of zeppelin, that's why I can avoid downloading it in pom.
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2864
4 is flaky test, LGTM
---
Github user necosta commented on the issue:
https://github.com/apache/zeppelin/pull/2864
ping @zjffdu
https://travis-ci.org/nokia/zeppelin/builds/368034675 (stage 4 looks broken)
Thanks
---
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/2933
ZEPPELIN-3416: Bump up the version of xercesImpl to 2.11.0.SP5
### What is this PR for?
Update xercesImpl to 2.11.0.SP5 to get some fixes. This is a required
dependency for
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2933
@zjffdu @r-kamath Please help review this.
---
Prabhjyot Singh created ZEPPELIN-3416:
-
Summary: Update xercesImpl to 2.11.0.SP5 to get some fixes.
Key: ZEPPELIN-3416
URL: https://issues.apache.org/jira/browse/ZEPPELIN-3416
Project: Zeppelin
GitHub user Savalek opened a pull request:
https://github.com/apache/zeppelin/pull/2934
[ZEPPELIN-3417] fix dependency in zeppelin-web/package.json
### What is this PR for?
The address of the dependency in zeppelin-web/package.json changed from
"shahata/angular-viewport-watch"
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2931
LGTM
---
Github user r-kamath commented on the issue:
https://github.com/apache/zeppelin/pull/2931
will merge this if there are no more discussions
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2931
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2864
---
Aleksey Savelenko created ZEPPELIN-3417:
---
Summary: Change the dependency in zeppelin-web/package.json
Key: ZEPPELIN-3417
URL: https://issues.apache.org/jira/browse/ZEPPELIN-3417
Project:
Github user weand commented on the issue:
https://github.com/apache/zeppelin/pull/2925
@mebelousov I fully agree that running document as the group is not what a
user would expect.
That's why I'm proposing another approach again:
1) prior to #2914 the cronExecutionUser
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2925
Thanks @weand , I agree with you that your proposal seems much easier. I
have updated the PR, please help verify it, thanks
---
20 matches
Mail list logo
