Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1775
For other reviewers
- If you are using git 2.10+, Notebook revision feature will not work.
Checkout [ZEPPELIN-1737](https://issues.apache.org/jira/browse/ZEPPELIN-1737)
---
If your proje
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1780
@bzz I agree with you and I will follow your word!
Thanks.
---
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 do
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1694
CI is all green now! Ready to review.
Here are some brief summaries what I did.
Since from this PR `anonymous`(non authenticated) user can't set note
permission, related test cases
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1787
@bikassaha zeppelin use spark repl api heavily. And spark repl is not
stable as RDD/DataFrame api (It happens in spark 1.x), so it is possible to
have changes in spark repl for spark feature relea
Github user bikassaha commented on the issue:
https://github.com/apache/zeppelin/pull/1787
In a different jira, should we consider removing this check for 2.1, 2.2
since its unlikely that Spark will break common APIs in dot releases. Perhaps
the policy can change to adding a check whe
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1759
As I posted the reason of CI failure in the above, it's not relevant. So
will merge into master if there are no more comments. Thanks @DrIgor!
---
If your project is set up for it, you can re
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1757
@lresende Thanks! Looks nice. BTW the [Build from
Source](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/install.html#building-from-source)
section went to too down side. How about put
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1749
---
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1788
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/1787
and btw, could you please test with the last 2.1 RC5 instead of snapshot
builds.
---
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 felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1787
Looks fine. Would you like to get this in before Spark 2.1 artifacts are in
maven?
I think we should reword the JIRA if that's the case - I don't think we
should say we support 2.1 until w
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1788
@Leemoonsoo @jongyoul @bzz Please help 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 feat
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1788
ZEPPELIN-1826. Flaky Test: ZeppelinSparkClusterTest.zRunTest
### What is this PR for?
Fix the flaky test of ZeppelinSparkClusterTest.zRunTest. The issue is that
z.run is not a blocking call so
Github user lresende commented on the issue:
https://github.com/apache/zeppelin/pull/1757
@AhyoungRyu @felixcheung Please take a look and let me know if I should
update any other pieces.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1787
@Leemoonsoo @felixcheung Please help 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 featu
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1787
ZEPPELIN-1815. Support Spark 2.1
### What is this PR for?
This PR is to update the max supported spark version to be 2.1 as Spark 2.1
is released.
I didn't change pom file and travis scrip
Github user lresende commented on the issue:
https://github.com/apache/zeppelin/pull/1757
No worries @AhyoungRyu I will make the suggested changes and add
ZEPPELIN-1811 to this pr as well.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user pmccaffrey6 commented on the issue:
https://github.com/apache/zeppelin/pull/1776
Hey @bzz,
Great catch! Sorry about that. I removed the unnecessary import. The
error seems unrelated. It comes from:
```
1470 21:03:56,259 ERROR org.apache.zeppelin.Abstra
Github user DrIgor commented on the issue:
https://github.com/apache/zeppelin/pull/1759
Ping-ping! :-)
What should I do to complete pull request?
---
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 do
Github user rawkintrevo commented on the issue:
https://github.com/apache/zeppelin/pull/1758
@AlexanderShoshin I think what @zjffdu and @bzz meant was that you could
possibly call out this notebook in some where like
[`docs/interpreter/flink.md`](https://github.com/apache/zeppelin/blo
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1782
Those are other flaky tests already reported. You can search JIRA
[here](https://issues.apache.org/jira/issues/?jql=project%20%3D%20ZEPPELIN%20and%20text%20~%20%22testShowDescriptionOnInterpreterCre
Github user DrIgor commented on the issue:
https://github.com/apache/zeppelin/pull/1782
@bzz @1ambda I attached CI logs
Failures are:
* `ZeppelinIT.testAngularDisplay:157->AbstractZeppelinIT.waitForText:84 »
StaleElementReference`
* `AngularElemTest:`
`AngularEl
Github user AlexanderShoshin commented on the issue:
https://github.com/apache/zeppelin/pull/1758
@bzz, I am not sure. We already have a *Word Count* example there to
describe `Flink` usage. Where should we place the link to this example?
---
If your project is set up for it, you can
Github user FireArrow commented on the issue:
https://github.com/apache/zeppelin/pull/1783
I agree with @AhyoungRyu. What she describes is the common way to link to
paragraphs, so it seems reasonable to assume that is what users are used to.
---
If your project is set up for it, you
Github user bitchelov commented on the issue:
https://github.com/apache/zeppelin/pull/1738
@AhyoungRyu ok, ill update jira issue.
---
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 bitchelov commented on the issue:
https://github.com/apache/zeppelin/pull/1738
@Leemoonsoo
https://issues.apache.org/jira/secure/ViewProfile.jspa?name=Sergey+Sokur
---
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 DrIgor reopened a pull request:
https://github.com/apache/zeppelin/pull/1782
[ZEPPELIN-1837] Fix possible reason of DepInterpreterTest failure
### What is this PR for?
Fix possible reason of DepInterpreterTest failure
### What type of PR is it?
[Bug Fix]
Github user DrIgor closed the pull request at:
https://github.com/apache/zeppelin/pull/1782
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1758
@AlexanderShoshin Right that would be better. So 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. If you
Github user AlexanderShoshin commented on the issue:
https://github.com/apache/zeppelin/pull/1758
I am confused a bit :)
@AhyoungRyu, do I need to drop my last commit or not?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1716
Hello @bzz ,
could you please review?
ci is green.
I think ready for merge on this pr.
Thank you :)
Have a nice day.
---
If your project is set up for it, you can reply t
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1775
good 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 project does not have this feature
enabled and wishes so,
Github user DrIgor closed the pull request at:
https://github.com/apache/zeppelin/pull/1782
---
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 DrIgor reopened a pull request:
https://github.com/apache/zeppelin/pull/1782
[ZEPPELIN-1837] Fix possible reason of DepInterpreterTest failure
### What is this PR for?
Fix possible reason of DepInterpreterTest failure
### What type of PR is it?
[Bug Fix]
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1786
this is ready for review, /cc @AhyoungRyu for help in 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
GitHub user khalidhuseynov opened a pull request:
https://github.com/apache/zeppelin/pull/1786
[ZEPPELIN-1842] sync on start when anonymous allowed
### What is this PR for?
this is to keep backward compatibility when it's possible to sync on start
in anonymous mode.
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1765
---
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 zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1765
[WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pySpark* fail the build
### What is this PR for?
Add more logging as I could not reproduce it locally.
### What type of PR is
Khalid Huseynov created ZEPPELIN-1842:
-
Summary: Sync mechanism doesn't work on start if anonymous mode is
on
Key: ZEPPELIN-1842
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1842
Project: Z
Github user DrIgor commented on the issue:
https://github.com/apache/zeppelin/pull/1782
@1ambda yes, of course
* https://travis-ci.org/apache/zeppelin/builds/185383249
* https://travis-ci.org/apache/zeppelin/builds/185389810
* https://travis-ci.org/apache/zeppelin/build
GitHub user DrIgor reopened a pull request:
https://github.com/apache/zeppelin/pull/1782
[ZEPPELIN-1837] Fix possible reason of DepInterpreterTest failure
### What is this PR for?
Fix possible reason of DepInterpreterTest failure
### What type of PR is it?
[Bug Fix]
Github user DrIgor closed the pull request at:
https://github.com/apache/zeppelin/pull/1782
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1782
Could you file CI links? :) also
---
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 a
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1765
[WIP] ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pySpark* fail the build
### What is this PR for?
Add more logging as I could not reproduce it locally.
### What type of PR is
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1736
Ping Ping ! Ci is green.
It's time to ready to review this PR :smile:
---
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 zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1765
---
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 DrIgor closed the pull request at:
https://github.com/apache/zeppelin/pull/1782
---
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 DrIgor reopened a pull request:
https://github.com/apache/zeppelin/pull/1782
[ZEPPELIN-1837] Fix possible reason of DepInterpreterTest failure
### What is this PR for?
Fix possible reason of DepInterpreterTest failure
### What type of PR is it?
[Bug Fix]
48 matches
Mail list logo
