Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1686
@AhyoungRyu I updated this PR :smiley:
---
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/1661
@minahlee Looks great to me!
Could you please rebase on latest master?
I assume that for testing, it should be enough to build `mvn clean package
-Pbuild-distr -DskipTests` and make
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1695
@PetroKostyuk thank you for improvement!
It looks like your work need to be rebased on the latest master in order to
resolve conflicts, before merging.
---
If your project is set up for
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1441
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1668
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1668
2 Scala 2.11 profiles failed on IgniteSQL tests - a flaky test tracked
under [ZEPPELIN-1738](https://issues.apache.org/jira/browse/ZEPPELIN-1738l)
```
Tests run: 2, Failures: 1, Errors:
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1672
Merging this if no more 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/1668
Waiting for CI to finish on https://travis-ci.org/1ambda/zeppelin/
Will merge to master right after that, if there is no further discussion.
---
If your project is set up for it, you can
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1682
Looks great to me, thank you @felixcheung let's merge if there is no
further discussion!
CI failure of tests in `zeppelin-zengine` on 1 profile looks not relevant
and will be taken care
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1566
@khalidhuseynov How about using `System.setProperty("HADOOP_USER_NAME",
getUserName());` instead of `proxy-user` setting?
Could you review https://github.com/astroshim/zeppelin/pull/15?
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1689
Go it, that sounds great.
Then shall we finish and merge this guy first, and then get back to #1668
merge it?
---
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/1500
Looks great to me and CI is green.
Thank you @Peilin-Yang for prompt responce and @felizbear for reviews!
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/1668
Looks like it need a final re-base here
---
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 tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1677
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1688
If i assume your remote name for Zeppelin repository is `apache-github`
(you can check it by running `git remote -v`),
First, fetch Zeppelin repo (to get latest master) with `git
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1691
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1699
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1708
Thanks @Tagar @astroshim for the review. Merge to master if there're no
further discussions.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user zjffdu commented on the pull request:
https://github.com/apache/zeppelin/commit/b6cd47996038bafd3c05caf8f17b26f0d30f3224#commitcomment-20026294
In testing/downloadSpark.sh:
In testing/downloadSpark.sh on line 23:
Seems should return non-zero status, @minahlee
You can get ui by
sc.uiWebUrl
And community is working on to display it in paragraph.
https://github.com/apache/zeppelin/pull/1663
Ruslan Dautkhanov 于2016年12月1日周四 上午8:58写道:
> Any easy way to get Spark Driver's URL (i.e. from sparkContext )?
> I always have to go to CM ->
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/1711
[ZEPPELIN-1721] Let each visualization define 'setting'.
### What is this PR for?
ZEPPELIN-1003 made an abstraction layer for visualization and now
visualizations are moved to
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1658
---
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
Any easy way to get Spark Driver's URL (i.e. from sparkContext )?
I always have to go to CM -> YARN applications -> choose my Spark job ->
click Application Master etc. to get Spark's Driver UI.
Any way we could derive driver's URL programmatically from SparkContext
variable?
ps. Long haul - it
GitHub user rhymoid opened a pull request:
https://github.com/apache/zeppelin/pull/1710
CI test for #1729
Just a CI test for #1729.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rhymoid/zeppelin
backport-hotfixes-ZEPPELIN-1729
Github user rhymoid commented on the pull request:
https://github.com/apache/zeppelin/commit/b6cd47996038bafd3c05caf8f17b26f0d30f3224#commitcomment-20023614
In testing/downloadSpark.sh:
In testing/downloadSpark.sh on line 23:
This seems off. If the script exits because it was
Github user rhymoid commented on the issue:
https://github.com/apache/zeppelin/pull/1705
Looks like the builds for branch-0.6 are failing because some Spark
binaries were removed from all mirrors. I think this was fixed in master with
#1599 + #1709, but not for branch-0.6.
---
If
Github user conker84 commented on the issue:
https://github.com/apache/zeppelin/pull/1582
My devloping evironment is Windows (10) if it can help
---
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
Jerzy J. Gangi created ZEPPELIN-1741:
Summary: JAR's specified with spark.jars in spark-defaults.conf
does not affect %pyspark interpreter
Key: ZEPPELIN-1741
URL:
Github user krskoondra commented on the issue:
https://github.com/apache/zeppelin/pull/1688
Someone help me. How can I resolve "This branch has conflicts that must be
resolved"?
I'm beginner with opensorce and github.
Thanks.
---
If your project is set up for it, you can
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/1582
hmm, I'm not sure
```
17:29:54,140 INFO
org.apache.zeppelin.interpreter.InterpreterOutputChangeWatcher:71 - watch
/tmp/ZeppelinLTest_1480267793139
Tests run: 1, Failures: 0,
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1708
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 Tagar commented on the issue:
https://github.com/apache/zeppelin/pull/1708
LGTM. Thank you!
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1702
Merge to master if there're no further 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 project does not have
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1709
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1707
Merge to master if there're no further 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 not have
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1709
+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 feature
enabled and wishes so, or if the
Hoon Park created ZEPPELIN-1739:
---
Summary: Flaky Test in zeppelin-display (AngularElem)
Key: ZEPPELIN-1739
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1739
Project: Zeppelin
Issue
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1709
CI is green, merging to master as a HOTFIX
---
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
Alexander Bezzubov created ZEPPELIN-1738:
Summary: IgniteSqlInterpreterTest fails the build \w Scala 2.11
Key: ZEPPELIN-1738
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1738
Project:
Hoon Park created ZEPPELIN-1737:
---
Summary: Zengine tests failed in local (Scala 2.11)
Key: ZEPPELIN-1737
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1737
Project: Zeppelin
Issue Type:
Github user karuppayya commented on the issue:
https://github.com/apache/zeppelin/pull/1663
Ready for 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
enabled and wishes
Tae Jun Kim created ZEPPELIN-1736:
-
Summary: Introduce trash & enable remove folder
Key: ZEPPELIN-1736
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1736
Project: Zeppelin
Issue Type:
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1658
@agoodm merged master and addressed tests added from #1632.
CI failure is not related. Merge to master.
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@tae-jun It is very AWESOME ð and thanks you for the accept my
suggestion.
Let me check again after CI is green ð
---
If your project is set up for it, you can reply to this email
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1696
This will be closed (duplicated with #1689)
---
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 tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1689
@bzz I will work on here and add some small details from #1696 later
(e.g adding profile for checkstyle not to skip checkstyle in all other
profiles)
---
If your project is set up for it,
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1696
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1709
This is awesome and thanks for great, great effort of @bzz
Let me add 1 small detail for who not to see the full history.
archive.apache has download limitation thus it is safe
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1668
It is safe to centralise `${plugin.download.version}` only in
`spark-dependency` to the root `pom.xml`
```
$ spark-dependencies git:(chore/centralize-maven-pom-versions) ag
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1696
rebasing on #1709 should solve the CI 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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1689
@1ambda I have created a HOTFIX for CI Spark download in #1709 so after
merging it, let's rebase and see if that fixed the issue.
I'm a bit confused, do you want to continue working here or
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1682
@felixcheung I think after #1709 there might be no need for `mvn -q` option
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1709
[HOTFIX] Make Spark download stable on CI
### What is this PR for?
There has been issues with downloading\caching Spark, esp in #1689 #1696
This is hotfix for Spark download on CI.
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1668
@bzz I will add right now and push 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 not have this feature
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1668
Will you also add `plugin.download.version` to
[spark-dependencies/pom.xml](https://github.com/apache/zeppelin/pull/1668/files#diff-457fb4186c9640eac4e0808bbd0d849e)
?
---
If your project is set up
Github user Peilin-Yang commented on the issue:
https://github.com/apache/zeppelin/pull/1500
@felizbear
The code of this PR uses `momentjs` to parse the string to date.
Specifically, http://momentjs.com/docs/#/parsing/string/
Integers are not of ISO8601 format so
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Always thanks for the 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
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1686
@AhyoungRyu Thanks for guidance.
Okay, Let me notify you after updating PR description.
And my JIRA username is `sora`.
---
If your project is set up for it, you can reply to this email
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@soralee What a nice suggestion! @tae-jun I think it's cool!!
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@soralee Hi! I added the feature you suggested. What do you think? Is title
or content of warning message appropriate? Thanks for the cool suggestion!
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1690
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1686
@soralee I just merged this into master branch. Could you update PR
description with latest change? Maybe one day, someone wants to refer this PR
to add another keyboard shortcut I think :)
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1686
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1682
@bzz
It is definitely better to do. I will create new JIRA issue for it.
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1703
@khalidhuseynov Thanks!
CI had been failed during downloading Spark
https://cloud.githubusercontent.com/assets/10060731/20750140/b0186c98-b738-11e6-8687-6697ced72a1e.png;>
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@soralee Thanks for the review! ð That's a great idea! I will give some
effect on the front-end and ping you again. Thanks :)
---
If your project is set up for it, you can reply to this email
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1696
I will rebase this
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1682
The test above is non-deterministic actually because network websequence
API is required to get the rendered result.
---
If your project is set up for it, you can reply to this email and have
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1682
I also see this error in #1612, but successfully build it locally.
---
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
75 matches
Mail list logo