Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/711#issuecomment-184535856
@felixcheung FYI, I just deployed the latest documentation on website.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/654#issuecomment-184535519
Merged with #706, and
https://github.com/apache/incubator-zeppelin/pull/709. CI green.
Ready for review.
---
If your project is set up for it, yo
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/711#issuecomment-184534006
+1 for @minahlee
---
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 hayssams commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-184531939
@prasadwagle
I understand your concern regarding LDAP. However, you can still implement
your own Authorisation realm using shiro and this would be a fil
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/711#issuecomment-184530640
@felixcheung Could you also build the docs and deploy them after you merge
the PRs which includes docs changes? So the updated docs can be reflected to
webs
Zhong Wang created ZEPPELIN-676:
---
Summary: Dynamic forms cannot be commented in %sh
Key: ZEPPELIN-676
URL: https://issues.apache.org/jira/browse/ZEPPELIN-676
Project: Zeppelin
Issue Type: Bug
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 so, or if
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails.
This to reproduce it on travis and fix it.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-184527678
Thanks - couple more instances in the last day:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/109303040/log.txt
https://s3.amazonaws.com/arch
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 so, or if
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails.
This to reproduce it on travis and fix it.
Github user anthonycorbacho commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/715#issuecomment-184519619
+1
---
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
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/715#issuecomment-184516027
Looks Good!
---
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
Github user cjmatta commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/684#issuecomment-184515705
Oops, totally missed that, sorry about that.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/710#issuecomment-184513238
Thanks @felixcheung for pointing it out. I was neither able to reproduce
it on my local, nor was able to do it on local travis. Hence, raise this PR
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails.
This to reproduce it on travis and fix it.
Github user malur commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184510375
@jongyoul Are you talking about adding sections in jdbc.md for configuring
different Databases? Wouldn't the JDBC page be too lengthy, if we add all in
one pag
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/717#issuecomment-184509262
It looks fantastic!!
---
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 n
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184509145
@malur Thanks for the documentation, but how about improving JDBC document?
It looks good by itself, but basically, because drill is using JDBC
interpreter,
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/667#issuecomment-184506451
@vgmartinez could you rebase to latest on master to get tests to run again?
---
If your project is set up for it, you can reply to this email and have yo
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/711
---
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 so, or if the fea
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/706
---
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 so, or if the fea
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/715#issuecomment-184502091
LGTM
---
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 fe
Github user sourav-mazumder commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-184497319
Hi Prasad,
In your organization if writing to enterprise level LDAP is not possible
(though that is the right approach from Enterprise Archit
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/715#issuecomment-184484524
LGTM
---
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 fe
Github user prasadwagle commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-184474829
@AhyoungRyu - I have fixed the documentation issues you reported. Thanks!
@hayssams - Your design allows for fine-grained controls and is elegant.
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-184467011
@FRosner Yes it will ignore the last row if it's empty. I couldn't
reproduce the error with the condition you explained but pushed a fix to handle
the case
Michael Gummelt created ZEPPELIN-675:
Summary: select * from crime
Key: ZEPPELIN-675
URL: https://issues.apache.org/jira/browse/ZEPPELIN-675
Project: Zeppelin
Issue Type: Improvement
Github user khalidhuseynov commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-184439606
LGTM
---
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 thi
Github user hayssams commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/681#issuecomment-184425585
I just pushed the code that implement support for shiro authorization on
websocket messages at
https://github.com/hayssams/incubator-zeppelin/commit/df43fb
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-184409223
merging today if no more comments
---
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 felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/697#issuecomment-184409000
any more comment?
---
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 n
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/672
---
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 so, or if the fea
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184266001
@malur If you want to find this docs in Zeppelin documentation web page,
you have to add `Drill` below [this
line](https://github.com/apache/incubator-zep
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184262845
@malur
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/640
---
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 so, or if the fea
Github user FRosner commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-184227703
Understood @minahlee. I did not have an empty row except the last one (with
the + on the right) I think. But because there is a +, I was not able to remove
i
Github user malur commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/644#issuecomment-184131164
Cool.
Meanwhile I have added documentation around using Drill with JDBC
Interpreter via PR #718
---
If your project is set up for it, you can reply to thi
GitHub user malur opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/718
Added details on using Drill with JDBC Interpreter
## What is this PR for?
Documentation to configure Drill using JDBC Interpreter
## What type of PR is it?
Docum
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-184128878
Sounds great, I'm looking forward.
Right now I do not see anything specific that need to be taken into account.
---
If your project is set up for it, you ca
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/717
[DOCS][JDBC]Guide user to add jdbc driver jar file via GUI interpreter menu
### What is this PR for?
Update jdbc documentation to let user load dependencies via GUI interpreter
menu
Github user zhongneu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/713#issuecomment-184118084
I was a little bit worried about this approach may not be very robust: we
have several reserved characters: `${}|(),`. Because the users will put more
synta
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/713#issuecomment-184112715
Right, multi-selection using string substitution can be complex,
considering target language syntax.
How about just concat selections and let user take
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/707
---
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 so, or if the fea
44 matches
Mail list logo
