Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1662
Thanks!
Looks great to me, will merge to master as soon as CI is green, if there is
no further discussion.
---
If your project is set up for it, you can reply to this email and have your
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1668
Thank you for double-checking and explanations. Looks like CI is failing
right now though
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1441
@cloverhearts looks like CI is failing. Do you know the reason? If you try
open\close the PR, can it be reproduced?
---
If your project is set up for it, you can reply to this email and have your
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1538
Great job @mfelgamal !
The idea was to try to avoid building separate artefacts for Docker and use
official convenience binaries from Apache release.
Am I mimssing something, or
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1339
Thank you @AhyoungRyu for great job and taking care in addressing the [user
experience
concerns](https://github.com/apache/zeppelin/pull/1339#issuecomment-259683752)!
---
If your project is
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1639
CI is green now.
Looks great to me, merging to master if there is no further discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1662
@1ambda thank you for great effort! It's a bit hard to review as
re-formatting is mixed with pom.xml modifications.
How hard would be to have 2 separate PRs - one with improvements
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@aspen01 thank you for contribution!
Could you please double-check that all new files, as any other file in
Apache Zeppelin, have [ASF's Apache 2.0 license
header](http://www.apach
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1603
@Leemoonsoo do you think it's worth re-basing on latest master?
Tried `mvn clean package -pl \!zeppelin-distribution` on this branch again,
got:
```
Results :
Tes
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1620
@wang28103 looks like PR to the wrong destination
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1668
Thank you - it's a great effort, a bit long to review so it took a while
and have diverged from master.
@1ambda could you rebase and also make sure it does not affect automation
like
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1632
@agoodm thank you, looks great! There has been some changes in
`.travis.yaml`, could you please rebase?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1639
@felizbear great!
From my experience, if you, as PR author, elaborate on the CI failure, in
case of any, it usually speeds merging things up.
---
If your project is set up for it, you can
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1638
Looks great to me, except for code style noted above.
Thank you @1ambda for tests improvements, would be happy to merge as soon
as the style issue addressed, if there is no further discussion
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1168
@jongyoul how do you think, could you interactively rebase this branch to
have only `./_tools/*` changes? Or is it better to start a JIRA issue and close
this guy?
@1ambda do you know any
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1639
Thank you @1ambda !
@felizbear as soon as #1659 with CI improvements was merged, let's rebase
and see if we can merge this guy
---
If your project is set up for it, you can reply to
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1659
Great, thank you @1ambda !
Looks great to me, @asanjar if there is no more input on Node version (this
PR uses v6, latest LTS) - merging it to master.
---
If your project is set up for it
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1659
@asanjar node version is bumped to 6.x TLS one, please let us know if that
works for you you on Power arch.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1666
ð Looks good to me, merging if there is no further discussion
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1666
Indeed, if you could check #1659 - AFAIK it does all the same, only the
node version is different there.
How about we merge this guy as it does not bring major changes and then
rebase #1659
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1659
Thanks for improving CI @1ambda
Looks great to me,
will be happy to merege asap, right after
[zeppelin-web/README.md](https://github.com/apache/zeppelin/blob/master/zeppelin-web
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1659
@1ambda great effort!
As mentioned in #1639 - could you please re-purpose (interactive rebase,
basically) this PR to contain only CI improvements?
One question on Node version update
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1639
@1ambda great job!
Could you please re-purpose (interactive rebase, basically) #1639 to
contain only CI improvements, so it does not include anything from this PR?
Then we can
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1639
Experimenting with heap size sounds good, if that is the reason CI fails.
But it would be much easier to judge, if somebody could summarize current
CI failure here.
On the frontend build
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1639
Thank you for good effort and contribution @felizbear !
Please, expect that usually reviews do take some time.
One thing that @1ambda brought that would be nice to understand better
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1594
Looks great to me,
merging to master if there is no further discussion
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1628
Tested with and without `matplotlib` - both cases work fine!
Also
```
%python
a = 'a'
print "{}".format(a)
```
works as expected
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1628
Will test right now and post back
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1626
Thank you for the prompt update @agoodm
CI failure on Selenium test profile seem un-related:
```
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 129.268 sec
- in
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1534
@agoodm sorry for digging this out, but I have just realized that this PR
changes only [Python
`PyZeppelinContext`](https://github.com/apache/zeppelin/blob/master/python/src/main/resources
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1598
Great work @1ambda ! Do you know if there are any ETA for Elasticsearch 5.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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1618
Looks like CI is stalled during PySpark execution somehow..
```
16/11/09 09:46:13 INFO PySparkInterpreter: File
/tmp/zeppelin_pyspark-7835708002435081304.py created
No
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/989
@onkarshedge thank you for the great work!
How do you think, how hard it would be to rebase it on latest master?
---
If your project is set up for it, you can reply to this email and have
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1194
I guess we better close this guy, or is it still WIP?
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1231
@onkarshedge would you be willing to address the comments on the code style
and rebase it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/932
@echarles I'm terribly sorry for for such a huge communication delay in my
side :\
Would you be willing to re-base it on the latest master so we could merge
it?
---
If your project i
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1339
Guys, what great work here, simplifying the build!
A quick question @AhyoungRyu as it's kind of a big change, and I'm sorry if
that was explained before, but could you pl
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1376
@lresende sorry for huge communication delay :\
It's great improvement on ability to debug failures, merging to master
ASAP, if there is no further discussion.
---
If your project i
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1378
Sorry for massive communication delay :\
Looks awesome to me, merging to master, if there is no further discussion!
---
If your project is set up for it, you can reply to this email and have
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1391
@zjffdu test plan sounds great.
My only concern was - may we can find a better place for this state, rather
than `static` ? Move it somewhere, where if does not require to be static, but
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@ravwojdyla
> Test/build should happen as part of standard build. Is it fine?
Yes, sounds fantastic, thank you for pointing it out.
Looks great to me, shall we merge it, if th
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1594
What should be the next steps here @1ambda?
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1603
on `mvn clean package -pl \!zeppelin-distribution` I got
```
Tests run: 11, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.193 sec
- in org.apache.zeppelin.search.LuceneSearchTest
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1605
Looks great to me, thank you @minahlee for cleaning up in order to avoiding
user confusion!
ð¯ for having a test
---
If your project is set up for it, you can reply to this email and have
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1610
ð Let's merge
---
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 s
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@mfelgamal thank you for your great work and @astroshim @khalidhuseynov
@AhyoungRyu and @1ambda for prompt reviews!
I think @1ambda raised very good points, using `zeppelin.sh` will allow
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1534
Looks great to me, ð for extra tests. Let's merge
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/928
Looks great to me, thank you @rawkintrevo
CI failure looks not relevant to the changes.
Merging to master if there is no further discussion.
---
If your project is set up for it
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/928
@rawkintrevo thank you for addressing the feedback promptly.
I tried examples and everything worked very well!
One thing though - on the second run of `add_mahout.py` I got
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1597
Looks awesome, thank you for cleaning it up!
Let me try 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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/928
Sorry for delay, @rawkintrevo !
Thank you for kind reminder on this great stuff. I want to mention this
work on ApacheCon, so hope we can merge this asap :)
I tied it and got
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1591
Looks great to me, thank you for fix and prompt response!
Let's merge after CI is green, if there is no further discussions.
---
If your project is set up for it, you can reply to
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1594
Websequence are awesome, thank you @1ambda for contributing this
simplification of MD interpreter.
Looks good to me, modulo few minor issues, commented above.
As soon as CI is green
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1471
So cool guys, sorry, I have totally missed out on this - it looks great and
CI is green now!
Looks like every feedback has been addressed by now?
Quick question on test and CI - it
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1589
ð for docs in same PR.
On CI, I think that's relativly minor and if that's too complicated to
configure - we can can opt out for manual test run+instruction in docs.
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1578
Looks great to me.
Nice side-effects is that CI should become faster 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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1590
Got it! Thank you for kind explanation.
CI failure looks not relevant for the change set.
Looks great to me, merging to master if there is no further discussion.
---
If your project
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1589
Thank you for contribution!
There are few things need to be taken care here:
1. make sure the code adhere [project
styleguide](https://zeppelin.apache.org/contribution/contributions.html
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1590
Just wonder if that means that notes are always load as `anonymous`
irregarding Zeppelin mode, because we do not have such permissions yet?
Curious, as I would expect that we would load all
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1586
Thank you @khalidhuseynov for prompt 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 project does not have this feature
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1586
Looks great to me, thank you for updated integration tests @cloverhearts !
CI fails on 2 profiles
```
Results :
Tests in error:
InterpreterRestApiTest.init:57
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1460
Looks good to me. CI is green now.
What do you think, @Leemoonsoo ?
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1532
Looks great to me!
Merging to master, if there is no further discussion.
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1508
Looks good to me, ð for tests!
Merging if there is no further discussion.
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1515
Looks great to me! Let's :shipit:
---
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
en
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1530
Thank you for contributing!
How do you think, how hard would be to add some tests here, to make sure
the change does not break things?
---
If your project is set up for it, you can reply
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1505
Looks great to me.
Thank you Mina for prompt fix!
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1378
@bustios let me know if you still think it's possible to make it optional
and I'll be happy to merge it.
---
If your project is set up for it, you can reply to this email and have your
re
GitHub user bzz opened a pull request:
https://github.com/apache/zeppelin/pull/1495
ZEPPELIN-1325: WIP - Python, advanced interpreter impl
### What is this PR for?
This is different implementation of Python interpreter that communicates to
external Python 2/3 process (not just
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/928
Sounds awesome, @rawkintrevo, thank you for keeping up a good work!
Let me try it this week and get back to you.
---
If your project is set up for it, you can reply to this email and have
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
Thank you so much @mfelgamal !
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
Thank you @mfelgamal !
Merging to master, if there is no further discussion
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
Great, thank you @mfelgamal for great job laying foundation for Apache Beam
integration! I hate to ask but, could you please rebase it on latest master one
more time? Looks like it have diverged a
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1419
Merging to master, if there is no further discussion.
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
Last 2 profile in CI are failing
```
[INFO] Zeppelin: Display system apis .. FAILURE [
2.537 s]
[31m- should provide onclick method *** FAILED ***[0m
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1419
Shall we merge this guy 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 does not have this feature
enabled and
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1422
This code looks very similar to the one we have in [python
interpreter](https://github.com/apache/zeppelin/blob/master/python/src/main/resources/bootstrap.py#L167),
contributed by @bustios and
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1424
Looks good to me.
Will merge to master if there is no further discussion
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1330
Thank you @mwkang @khalidhuseynov !
Will merge to master, if there is no further discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1415
Looks great 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 have this feature
enabled and wishes so, or
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/928
@rawkintrevo thank you for sharing the ideas and keeping it up!
This sounds like very reasonable approach to me, at least until mahout is
Spark2 compliant.
So the idea is to provide an
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
CI is still failing.
Sorry for confusion, after a bit more thinking it looks like simplest way
here actually will be - to follow the same convention as with any other
interpreters, like
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
@mfelgamal to address 1-2 can you please try to update `.travis.yml`: `-pl
\!beam` -> `-pl !beam` ?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
Looks great to me, thank you @mfelgamal !
I think it's ready to be merged.
The only minor thing that looks a bit off - re-formatted `pom.xml` is not
consistent with the other mo
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1411
Merging to master if there is no further discussion
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1411
Looks good to me, thank you @astroshim
I'm a bit confused - in JIRA fix-for version is only 0.6.2 but PR is to
master branch (which implies 0.7.0 as well)
Could you please cl
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1410
Merging it in
---
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 bzz opened a pull request:
https://github.com/apache/zeppelin/pull/1410
Fix download links
### What is this PR for?
In recent changes download links were broken on the project website
### What type of PR is it?
Hot Fix
### Todos
* [x
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
After running it locally, I think it is great but the last thing that would
be very nice to have, is to expand the example in the docs to be
self-contained, meaning easily runnable by the the first
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1384
@1ambda ð fot tests, looks great to me except for licensing issue
raised by Moon and small comment above.
If you could let me know you user name in ASF JIRA I'll be happy to
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1401
Looks good to me, merging if there is no further discussion.
CI fails on single profile with 1 flaky test
```
[INFO] Zeppelin: Server ... FAILURE
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1407
Thank you for improvements, @cloverhearts
Looks good to me.
Although as discussed under
[ZEPPELIN-1360](https://issues.apache.org/jira/browse/ZEPPELIN-1360) - this
approach have
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1376
RAT service taking too much input seems to be the issue here. I think it is
reproducable on local env as under
[ZEPPELIN-278](https://issues.apache.org/jira/browse/ZEPPELIN-278)
---
If your project
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1401
I think having a separate profile it is reasonable approach for CI. Though
I also think there is value in keepinv the default behaviour for local Dev
workflow and make RAT check on by default
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
I think having it as it is now sounds as a good start. It can be always
enhanced it later on, based on user feedback.
Changes look great to me, thank you!
If that is ok - please let
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1333
Very similar test failure as we discussed above happened in un-related
#1363
[here](https://s3.amazonaws.com/archive.travis-ci.org/jobs/155836675/log.txt)
```
Failed tests
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1333
@zjffdu so there must be some state that persists between the runs.
@Leemoonsoo do you have any idea what that state might be that affects
`InterpreterFactoryTest `? There seems to be
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1371
Looks great to me, thank you @Peilin-Yang !
@corneadoug what do you think?
Let's merge to master, if there is no further discussion.
---
If your project is set up for it, yo
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1376
@lresende shall we merge it or do you find @zjffdu suggestion to work and
we can close it for now?
Thank you @doanduyhai ! Let's try to close\open it to trigger the CI and
see if the fa
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1302
Looks great to me, thank you @astroshim !
On @corneadoug point - it should be possible to animate the icon button \w
arrows to be i.e rotating while "restart is in progress" s
101 - 200 of 690 matches
Mail list logo