Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2354
LGTM. Thanks @sohaibiftikhar for the contribution.
Merge to master if no further discussions here.
---
If your project is set up for it, you can reply to this email and have your
Github user sohaibiftikhar commented on the issue:
https://github.com/apache/zeppelin/pull/2354
CI is green. Earlier errors seem to have been erratic and resolved
themselves after estart:
https://travis-ci.org/sohaibiftikhar/zeppelin/builds/238705120
---
If your project is set up
Github user sohaibiftikhar commented on the issue:
https://github.com/apache/zeppelin/pull/2354
CI looks good now it was initially now working before as for some reason:
`curl -sSL
"http://keyserver.ubuntu.com/pks/lookup?op=get=0x51716619E084DAB9; |
sudo -E apt-key add -`
was
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/2354
@sohaibiftikhar i guess we can rebase it from master now
---
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
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/2354
Hi @FRosner, I think best way would be to help review and test #932 first.
i was going to do it as well, but didn't get much time recently. thanks for
help, and i'll check it this week
Github user FRosner commented on the issue:
https://github.com/apache/zeppelin/pull/2354
Hi @khalidhuseynov!
Thanks for looking into it. Is there a way to support you?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/2354
thanks for clarification, then i may look into #932 as well, and after
finalising that we can rebase this one
---
If your project is set up for it, you can reply to this email and have
Github user sohaibiftikhar commented on the issue:
https://github.com/apache/zeppelin/pull/2354
@khalidhuseynov At front it seems that the issues are unrelated however for
reasons I do not know #2182 is triggered when using the LdapRealm for me. #932
fixes that issue and hence for
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/2354
@sohaibiftikhar thanks for adding option for authentication with role
check, I was wondering if these changes are related to the issue in #932, from
first look seems a bit unrelated and i
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2354
\cc @khalidhuseynov
---
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
10 matches
Mail list logo
