Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1504
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 feature
enabled and wishes so, or if the
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1476
Try the commandline from Travis?
https://github.com/apache/zeppelin/blob/master/.travis.yml#L75
---
If your project is set up for it, you can reply to this email and have your
reply appea
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1504
[HOTFIX] Fix testWidth selnium test
### What is this PR for?
Last profile of CI test fails after #1461. This PR fixes it.
### What type of PR is it?
Test fix
### Todos
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1426
Also, can you update the doc as well.
Rest 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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1462
Livy interpreter is not a trivial interpreter although it use rest api. The
current unit test is not sufficient to guarantee the quality of livy
interpreter. I have found some issues there, but bef
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1502
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 feature
enabled and wishes so, or if the fe
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1462
As I have stated, I think these tests are good to have but worry about the
added complicity for a source release having these jars in the "source",
especially since these jars are not from an
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1503
@gss2002 I have to admit there's lots of issues in livy interpreter. Your
changes seems a little big for this fix. Would use `ConcurrentHashMap` directly
work for you ? rather than declare `LivyPyS
Github user gss2002 commented on the issue:
https://github.com/apache/zeppelin/pull/1503
Attempted to use the work around (protected static Map
userSessionMap = new HashMap<>(); ) provided by @zjffdu it did not work. So I
went with the more involved re-write and it has solved our prob
GitHub user gss2002 opened a pull request:
https://github.com/apache/zeppelin/pull/1503
NPE LivySparkSQLInterpreter thrown with %livy.sql interpreter function
### What is this PR for?
The LivySparkSQLInterpreter class does not correctly process the
userSessionMap throwing back a
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/1502
[gh-pages] Separate "Supported Interpreters" table from Zeppelin download
page
### What is this PR for?
Currently "Available Interpreters" table is located under [Zeppelin
download
page]
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1476
Thanks @felixcheung , Do you know how to run SELENIUM test ? I follow the
instruction on READM.MD, but fails
```
Zeppelin comes with a set of end-to-end acceptance tests driving headless
s
+1
On Tue, Oct 11, 2016 at 10:24 AM, Jeff Zhang wrote:
> +1, teste with spark 2.0 with basic notes, ZeppelinContext and UDF works
> correctly.
>
> On Tue, Oct 11, 2016 at 8:41 AM, Anthony Corbacho <
> corbacho.anth...@gmail.com> wrote:
>
> > +1
> >
> > I tested with my notebooks works as expe
+1, teste with spark 2.0 with basic notes, ZeppelinContext and UDF works
correctly.
On Tue, Oct 11, 2016 at 8:41 AM, Anthony Corbacho <
corbacho.anth...@gmail.com> wrote:
> +1
>
> I tested with my notebooks works as expected.
> Great job!
>
> On Mon, Oct 10, 2016 at 9:34 PM, Mina Lee wrote:
>
>
+1
I tested with my notebooks works as expected.
Great job!
On Mon, Oct 10, 2016 at 9:34 PM, Mina Lee wrote:
> Hi folks,
>
> I propose the following RC to be released for the Apache Zeppelin
> 0.6.2 release.
>
> The commit id is 9a44590841d5d83da1a297d507f6e2e67784ba1f which is
> corresponds to
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@corneadoug, @Leemoonsoo I re-based and added testcase. please review.
---
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 pr
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1433
@Leemoonsoo ping
---
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,
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/1501
[ZEPPELIN-1532] Remove map visualization from source tree, because of
incompatible license
### What is this PR for?
According to https://developers.arcgis.com/terms/ and and
http://www.es
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1210
Visualizations have incompatible license can be packaged as a [Helium
application](https://github.com/apache/zeppelin/blob/master/zeppelin-interpreter/src/main/java/org/apache/zeppelin/helium/Ap
Lee moon soo created ZEPPELIN-1532:
--
Summary: Remove Map visualization from source tree
Key: ZEPPELIN-1532
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1532
Project: Zeppelin
Issue Ty
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1476
Do you intent to include the TODO in this PR, or as a follow up?
Otherwise I think this is good to go - you can remove the WIP in the title
and I'll merge if there is no more comment.
--
Github user Peilin-Yang commented on the issue:
https://github.com/apache/zeppelin/pull/1500
@corneadoug It was really messy to rebase so I created this new PR. The
previous one was closed.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
GitHub user Peilin-Yang opened a pull request:
https://github.com/apache/zeppelin/pull/1500
[ZEPPELIN-1371]add text/numeric conversion support to table display
### What is this PR for?
We people in Twitter have great demand of adding a flexible drop down menu
for the columns in
Github user Peilin-Yang closed the pull request at:
https://github.com/apache/zeppelin/pull/1363
---
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 feature
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1460
ping
---
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 khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1459
ping
---
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
Hi folks,
I propose the following RC to be released for the Apache Zeppelin
0.6.2 release.
The commit id is 9a44590841d5d83da1a297d507f6e2e67784ba1f which is
corresponds to the tag v0.6.2-rc1:
https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=9a44590841d5d83da1a297d507f6e2e67784ba
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1462
Yes, that part I agree, it cannot be tested without jar, `org.mockito.Mock`
will always mock.
@Leemoonsoo @jongyoul @felixcheung any thought about checking in jar(s) ?
---
If yo
lichenglin created ZEPPELIN-1531:
Summary: Allow Multi sql in one paragraph
Key: ZEPPELIN-1531
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1531
Project: Zeppelin
Issue Type: Bug
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1462
mockit is for unit test, not for system test. It is just like
[AbstractTestRestApi](https://github.com/apache/zeppelin/blob/master/zeppelin-server/src/test/java/org/apache/zeppelin/rest/AbstractTest
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1462
What I mean to say was, lets mock these till the time we get livy's jar
from public repo, how do you think about that ?
---
If your project is set up for it, you can reply to this email an
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1462
@prabhjyotsingh I dont' get it. This is integration test where I will
launch a real livy server, how can I use mock to simulate that ?
---
If your project is set up for it, you can reply to this
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1462
Any thought about using `org.mockito.Mock`; example
https://github.com/apache/zeppelin/blob/master/python/src/test/java/org/apache/zeppelin/python/PythonInterpreterTest.java#L78
?
---
If
Sherry Xue created ZEPPELIN-1530:
Summary: Change the env variable name DEBUG that Zeppelin used to
be a specific name of ZEPPELIN to avoid
Key: ZEPPELIN-1530
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1462
@prabhjyotsingh These jars are not available in public maven repository. Do
you have any better ideas ?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1369
---
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 feature is e
Github user djKooks commented on the issue:
https://github.com/apache/zeppelin/pull/1472
@AhyoungRyu No problem~thanks for guide! I'll keep work on it.
---
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
37 matches
Mail list logo
