Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/693#issuecomment-185041982
CI green, tested, 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 projec
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/692#issuecomment-185041682
@ravicodder can you please rebase again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/690#issuecomment-185040743
Tested, CI green, 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 projec
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/691#issuecomment-185040790
Tested, CI green, 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 projec
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/689#issuecomment-185040701
Tested, CI green, 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 projec
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/654#issuecomment-185040374
Thanks @felixcheung for the review. Made appropriate changes.
---
If your project is set up for it, you can reply to this email and have your
reply ap
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 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 corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/720#issuecomment-185025224
Its not backward compatible for some of my Notebooks.
We should ask some AngularDisplay power user to test it against theirs, so
that they can share ho
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/723#issuecomment-184991556
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 hav
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/715#issuecomment-184983357
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
Github user btiernay commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-184983264
Curious if this PR supports form variables in R paragraphs. I tried with no
luck. Would be a nice addition if not.
---
If your project is set up for it, yo
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/649#issuecomment-184975147
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 felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-184974923
Could you please rebase again.
@Leemoonsoo @corneadoug comments?
---
If your project is set up for it, you can reply to this email and have your
repl
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/670#issuecomment-184973918
@tzolov how would you like to proceed?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/693#issuecomment-184973712
@prabhjyotsingh could you review this as well?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/692#issuecomment-184973699
@prabhjyotsingh could you review this as well?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zhongneu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/190#issuecomment-184970473
What is the status of this pr 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 p
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/716#issuecomment-184968866
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 felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/720#issuecomment-184968515
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 f
Hi Jeff,
You can find the list of PPMC here [1], although it doesn't include the
affiliations.
We do not currently have commiters only status in the project.
Informations about how to become a Committer is inside the Contributing
document of the project [2]
For the selection, we follow the usual
Zhong Wang created ZEPPELIN-679:
---
Summary: Printing the full exception stack for SQL queries is scary
Key: ZEPPELIN-679
URL: https://issues.apache.org/jira/browse/ZEPPELIN-679
Project: Zeppelin
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/710#issuecomment-184963957
another download error
https://s3.amazonaws.com/archive.travis-ci.org/jobs/109754384/log.txt:
```
SPARK_HOME is
/home/travis/build/apache/incubat
GitHub user felixcheung opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/723
update README for doc
### What is this PR for?
Updating doc for the Zeppelin docs
### What type of PR is it?
Documentation
### Todos
* [x] - Update doc
Mainly for clarity (as I may have missed such a summary) -
Is there a summary that exists (or perhaps we could reply here) that outlines:
What's the process within the zeppelin project to add additional Committers
and/or PMC members?
Who selects the people for these roles, and is there diversity
Zhong Wang created ZEPPELIN-678:
---
Summary: Table header doesn't scroll for wide table
Key: ZEPPELIN-678
URL: https://issues.apache.org/jira/browse/ZEPPELIN-678
Project: Zeppelin
Issue Type: Bug
GitHub user jeffsteinmetz opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/722
allows zeppelin to be run and managed as a service. ZEPPELIN-641
### What is this PR for?
allows zeppelin to be run and managed as a service, does not start in
background vi n
Github user jeffsteinmetz closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/721
---
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 jeffsteinmetz opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/721
allow startup scripts to manage zeppelin as a service. Jira Ticket
ZEPPELIN-641
You can merge this pull request into a Git repository by running:
$ git pull https://github.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/711#issuecomment-184904562
@felixcheung Permission granted. Sorry i forgot adding your id to cwiki.
---
If your project is set up for it, you can reply to this email and have your
r
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/711#issuecomment-184895862
thanks, I was not aware of that process. Could you give me permission to
add to https://cwiki.apache.org/confluence/display/ZEPPELIN/Committer+Guide
---
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/668#issuecomment-184891761
@vgmartinez could this be brought up to date with #667 ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/667#issuecomment-184891527
looks good, merging 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.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/716#issuecomment-184883698
Changes 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
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/720
Bringing compiledScope for backward compatibility
### What is this PR for?
compiledScope was not exposed API for users.
However, some users were using compiledScope in their no
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-184818151
Found 2 issues:
- AbstractFunctionalSuite.scala : we were never closing the webDriver
- ZeppelinIT.java : this X-Path (//div[contains(@id,\"_t
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-184805345
@bzz All of the issues you identified are git-rebase artifacts. None of
them are intended as part of the PR. Please keep in mind the PR was originally
in A
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-184798609
Concerning `vendor-repo`, it's already merged by #701, thus there's no need
to be added separately.
---
If your project is set up for it, you can reply to
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/6#issuecomment-184788071
@hyonaldo @ankurmitujjain Thanks for explaining it, but, personally, #714
looks better approach. How about you?
---
If your project is set up for it, you can
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 jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/684#issuecomment-184785853
@cjmatta No problem. Thanks for fixing it. And I will try it.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Jeff Steinmetz created ZEPPELIN-677:
---
Summary: Add ability to hide legend in graphs
Key: ZEPPELIN-677
URL: https://issues.apache.org/jira/browse/ZEPPELIN-677
Project: Zeppelin
Issue Type: B
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/667#issuecomment-184766822
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 feat
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/705#issuecomment-184762793
Thank you for the suggestion @Leemoonsoo. Have added AuthenticationInfo
class, and made relevant changes.
---
If your project is set up for it, you c
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/718#issuecomment-184756035
@malur yes adding drill to the supported DB in the JDBC documentation.
However most of the content of your document is already inside the JDBC, so
I do
Github user cjmatta commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/684#issuecomment-184729455
The latest web server code is failing the CI tests. The JDBC project
compiled without error though.
---
If your project is set up for it, you can reply to t
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-184692116
I have reviewed and commented inline.
Generally it looks great, but I have the feeling that we could dramatically
speed merging it up by reducing the sco
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-184609241
Last commit makes the build fail for profiles 1.6 (one at zeppelin-display,
the other after having built all modules saying after `No output has been
receiv
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/716#issuecomment-184604587
@FRosner Sure take your time and thank you for keeping your eyes on it :)
---
If your project is set up for it, you can reply to this email and have your
re
Github user FRosner commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/716#issuecomment-184603352
Hi @minahlee I am not in the position to build zeppelin right now. I can
test it over the weekend, if this is fine for you.
---
If your project is set up fo
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-184592116
Rebase done, but not fully tested... Work ongoing...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user vgmartinez commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/667#issuecomment-184589886
hi @felixcheung @jongyoul...I have added more test and fix the other
---
If your project is set up for it, you can reply to this email and have your
reply
53 matches
Mail list logo
