Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/372#issuecomment-153979630
Looks good, useful feature.
---
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
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/390#issuecomment-153982076
@corneadoug having coverage is a good idea. We should look at the issue one
more time.
---
If your project is set up for it, you can reply to this email
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/438#issuecomment-157002527
Looks good to me.
---
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
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/53#issuecomment-167797523
@jongyoul @elbamos @hayssams this change is an important one, and we should
try to get this in. My two cents on how we could divide the whole pull request
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/53#issuecomment-167866687
@hayssams please do that, it will be great. I can help with Junits, Docs
and separation, should you need it, will commit into your branches, which can
then
GitHub user rconline opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/625
Tested with local LDAP configurations.
### What is this PR for?
@hayssams @Leemoonsoo Just tested
https://github.com/apache/incubator-zeppelin/pull/586 with a local LDAP, works
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/625#issuecomment-170909250
@hayssams thanks for pointing out. Made the change.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/519#issuecomment-181870734
Looks good. Can we get some attention from committers. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/705#issuecomment-181871112
This is a requisite first step for getting to multi-user scenarios. I
believe this should get discussed a little bit in terms of approach.
---
If your
Github user rconline commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/705#issuecomment-183897757
@felixcheung @prabhjyotsingh The purpose of this discussion should be to
get to an approach that allows us zeppelin to do a couple of things - pass user
10 matches
Mail list logo
