Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/2621
LGTM
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2585
Hi, Thanks for rebasing and testing. But this recent PR passed the CI.
If you don't mind please re-run few times until successful and check the
failure reasons?
If we can verify they
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2620
not yet, it only affects 0.8.0, so won't cause any compatibility break.
---
Github user yu74n commented on the issue:
https://github.com/apache/zeppelin/pull/2585
I rebased and ran test again, but this test is still failed.
https://travis-ci.org/yu74n/zeppelin/jobs/286508544
I think probably my PR doesn't affect the test.
I ran test several
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2613
@Leemoonsoo parent project must be pom type, but zeppelin-interpreter is
jar. So I have to introduce module `interpreter-parent`. Interpreter dev guide
is updated in the new commit.
---
GitHub user yanboliang reopened a pull request:
https://github.com/apache/zeppelin/pull/2621
[Minor] Remove hardcoded key in zeppelinhub.
### What is this PR for?
Hardcoded seed/key may cause security issue, so remove potential risk.
### What type of PR is it?
Github user yanboliang closed the pull request at:
https://github.com/apache/zeppelin/pull/2621
---
Github user yanboliang commented on the issue:
https://github.com/apache/zeppelin/pull/2621
@prabhjyotsingh Please help review.
---
GitHub user yanboliang opened a pull request:
https://github.com/apache/zeppelin/pull/2621
[Minor] Remove hardcoded key in zeppelinhub.
### What is this PR for?
Hardcoded key may cause security issue, so remove the potential risk.
### What type of PR is it?
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2620
did we release this?
---
Github user necosta closed the pull request at:
https://github.com/apache/zeppelin/pull/2596
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2617
Thanks @gef756 for the fix. LGTM
---
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2613
I see. Could you also explain little bit about why you decided to make
`interpreter-parent` module instead of just add copy plugin to existing common
parents `zeppelin-interpreter` ?
Github user PetroKostyuk commented on the issue:
https://github.com/apache/zeppelin/pull/1783
As it seems, this pull request is still open, so this feature is not in
added to application. I no longer work on zeppelin project, so I don't know
whether this feature will get published in
Github user aurely06 commented on the issue:
https://github.com/apache/zeppelin/pull/1783
Hi,
Any update about this request ? The link to paragraph in context is
available or not yet ?
Thank you !
---
Github user necosta commented on the issue:
https://github.com/apache/zeppelin/pull/2596
Thanks @felixcheung and apologies for the delay on this.
Rebased on top of latest master and updated documentation.
Can you have another look?
Thanks
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2620
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2585
The 3/4 jobs failed. Please rebase this PR based on master. And Could you
resolve them?
---
=AKIAJRYRXRSVGNKPKO5A/20171011/us-east-1/s3/aws4_request=host=15e71aa2c61cf1ee2f241e91b3d4ab31c47293888f81b865603c6d3171142ad4
```
[INFO] **
[INFO] *Failures*
[INFO
Github user yu74n commented on the issue:
https://github.com/apache/zeppelin/pull/2585
@1ambda Sure
---
GitHub user yu74n reopened a pull request:
https://github.com/apache/zeppelin/pull/2585
[ZEPPELIN-2647] Bypass auth logic when a user logins as admin role
### What is this PR for?
For administrator, make new admin role that assigned user can see all
notebooks.
### What
Github user yu74n closed the pull request at:
https://github.com/apache/zeppelin/pull/2585
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2573
rebased since it's quite old
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2616
Nowadays many browser support password auto-completion. (e.g Chrome, ...)
So +1 for this feature.
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2585
I wanted to merge now. But cound't find the build history.
- https://builds.apache.org/job/zeppelin-pull-request/2960/
@yu74n if you don't mind close and reopen this issue to
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2580
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2619
@namanmishra91 I don't see the current behavior is retained in this PR.
Personally I don't think the current behavior needs to be retained. Even we
want to keep the current behavior, it should be
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2617
I tried to search other typo but couldn't find (for viable text, not for
css class things). LGTM.
Github user namanmishra91 commented on the issue:
https://github.com/apache/zeppelin/pull/2619
@zjffdu Thanks for the feedback. Directly modifying the
`persistAndExecuteSingleParagraph` method will lead to the current behaviour
getting lost completely. The reason I chose to not
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2618
notebook storage is in module zeppelin-zengine. Just found it has jar
conflicts with the native azure and s3 storage. Will keep it pending for now.
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2619
@namanmishra91 I don't mean your PR doesn't do it correctly. I mean you
could do it in an easier approach with less code changes. Just modifying
`persistAndExecuteSingleParagraph` to make it run
Github user dmvk commented on the issue:
https://github.com/apache/zeppelin/pull/2614
I'll try to implement new storage using official minio sdk
https://github.com/minio/minio-java. It seems this patch is still not enough
for s3 client to work properly with minio.
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2478
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2478
merged, thanks!
---
Github user namanmishra91 commented on the issue:
https://github.com/apache/zeppelin/pull/2619
@zjffdu
> What we need to do it to run it blocking way.
The code does exactly that. It might look complicated overall as a lot of
code is related to handling and passing
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2619
@namanmishra91 Thanks for the contribution. But the implementation is a
little complicated to me. I think the easiest implementation is just run
paragraphs sequentially in backend. Look at the
36 matches
Mail list logo