Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1270
Looks great to me, thank you @praagarw
Merging to only to master, if there is no further discussion
Or do you think we should also apply this to upcoming 0.6.1 as well?
---
If your
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1258
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 sarutak opened a pull request:
https://github.com/apache/zeppelin/pull/1271
[Z
### What is this PR for?
Some of the doc files say "SparkSQL" but the collect spelling is "Spark
SQL" (need a white space between "Spark" and "SQL").
Lets's replace it with the
Kousuke Saruta created ZEPPELIN-1274:
Summary: Write "Spark SQL" in docs rather than "SparkSQL"
Key: ZEPPELIN-1274
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1274
Project: Zeppelin
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1263
CI failure
```
Results :
Failed tests:
ZeppelinSparkClusterTest.sparkRTest:116 expected:<[[1] 3]> but
was:<[Error in getSparkSession(): SparkSession not initialized
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1258
@bzz
I removed the parts which you say.
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
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1258
@corneadoug
I did modification.
Please review for this pr.
---
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1267
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
I refactor method name.
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1266
@bzz Really appreciate for your all precise comments! Updated several docs
as you suggested :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user prasadwagle opened a pull request:
https://github.com/apache/zeppelin/pull/1270
[ZEPPELIN-1273] Use Math.abs to determine if custom formatter should be
called for y axis values
### What is this PR for?
Use Math.abs to determine if custom formatter should be called
Prasad Wagle created ZEPPELIN-1273:
--
Summary: d3.format not called for large negative numbers
Key: ZEPPELIN-1273
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1273
Project: Zeppelin
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1207
CI build failure is irrelevant with this PR i think. Can this be merged
then?
---
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 asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1268
---
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/1259
---
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/1268
@zjffdu you have good eyes :) 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 this
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1267
Sounds great! Apparently CI failure has been addressed in #1259 as well
---
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/932
A quick search over JIRA suggests that the failure you reffer was taked
care of in [ZEPPELIN-1264](https://issues.apache.org/jira/browse/ZEPPELIN-1264)
and is resolved in latest master, please rebase.
lichenglin created ZEPPELIN-1272:
Summary: Zeppelin spark innterpreters throws
java.lang.StackOverflowError after running a while
Key: ZEPPELIN-1272
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1272
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1266
Great improvement @AhyoungRyu ! Posted few suggestions above.
---
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/1263
CI failed due to
[ZEPPELIN-1264](https://issues.apache.org/jira/browse/ZEPPELIN-1264)
Rebasing on latest master and merging, if there is no further discusison
---
If your project is set up
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1267
Looks great to me, thank you @bustios !
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1269
\cc @Leemoonsoo Please help review, 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 does not have this
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1269
ZEPPELIN-1270. Remove getting SQLContext from SparkSession.wrapped()
### What is this PR for?
SparkSession.wrapped is only for spark 2.0 preview, it is not supported in
spark 2.0 release. So
Jeff Zhang created ZEPPELIN-1270:
Summary: Remove getting SQLContext from SparkSession.wrapped()
Key: ZEPPELIN-1270
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1270
Project: Zeppelin
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1268
@AhyoungRyu Please help review. 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 does not have this feature
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1260
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 have this feature
enabled and wishes
GitHub user bustios opened a pull request:
https://github.com/apache/zeppelin/pull/1267
[ZEPPELIN-1261] Bug fix in z.show() for matplotlib graphs
### What is this PR for?
Bug fix in z.show() for matplotlib graphs and code refactoring
### What type of PR is it?
Bug
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1259
Merge it into master and branch-0.6 if there're 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
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1261
---
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/1261
Merge it into master and branch-0.6 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
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/1266
[ZEPPELIN-1219] Add searching feature to Zeppelin docs site
### What is this PR for?
As more and more document pages are added, it's really hard to find
specific pages. So I added
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/1259
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 echarles commented on the issue:
https://github.com/apache/zeppelin/pull/932
Failing with
org.apache.zeppelin.interpreter.InterpreterException:
org.apache.thrift.TApplicationException: Internal error processing
createInterpreter
at
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/1265
Quickly went through the changes, but not sure to get it.
In which way is this related to the various intepreter modes we already
have?
In which way is this related to ZEPPELIN-1000
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1247
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
Github user rawkintrevo commented on the issue:
https://github.com/apache/zeppelin/pull/928
[Build 6281](https://travis-ci.org/apache/zeppelin/builds/149199571)
Profiles
[1](https://s3.amazonaws.com/archive.travis-ci.org/jobs/149199572/log.txt),
GitHub user rawkintrevo reopened a pull request:
https://github.com/apache/zeppelin/pull/928
[ZEPPELIN-116] Add Apache Mahout Interpreter
### What is this PR for?
This PR adds Mahout functionality for the Spark Interpreter.
### What type of PR is it?
Github user rawkintrevo closed the pull request at:
https://github.com/apache/zeppelin/pull/928
---
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 jojurajan opened a pull request:
https://github.com/apache/zeppelin/pull/1264
Updated path to point to latest Maven binaries
### What is this PR for?
To update link of Maven binary provided in the installation steps.
### What type of PR is it?
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1261
Have created
[ZEPPELIN-1269](https://issues.apache.org/jira/browse/ZEPPELIN-1269)
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1259
Thanks @echarles for the review. I have exposed sqlContext as well
---
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 corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@kavinkumarks Thanks for the contribution, the Paragraph ID is actually
already shown in the UI
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1239
@Leemoonsoo @minahlee Could you please test 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
GitHub user kavinkumarks opened a pull request:
https://github.com/apache/zeppelin/pull/1262
[ZEPPELIN-1268]
### What is this PR for?
This pull request is to add the below options so when someone develops a
custom UI it would ease them.
1. Add a new option "Get paragraph
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/1259
@minahlee Just saw you comment on ZEPPELIN-1260, but @Leemoonsoo has been
fast...
I have tested the changes, and now sparkr dataframes are working well (with
the `spark` variable which
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1261
@bzz I see the same error on my local test. However somehow travis does not
have the error.
Anyway, do you mind file a new issue for the new error?
---
If your project is set up for it,
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1258
I would limit this PR to the first commit, for me the rest is out of scope.
Also, in the first PR, there is no need to make a if before calling
`setParagraphMode`, the conditions you added
Github user SachinJanani commented on the issue:
https://github.com/apache/zeppelin/pull/1234
Travis checks are failing but are not related to this changes.@jongyoul Can
you please have a look.
---
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/1261
Have updated the comment, you might need to reload the page
---
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/1261
Local build results are - it hangs for last 4 minutes with the error below
```
20:56:59,101 INFO org.apache.zeppelin.notebook.Paragraph:282 - run
paragraph 20160802-205659_1961452200
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1261
Not sure why CI is not triggered in this PR.
Here's CI result with my travis account
https://travis-ci.org/Leemoonsoo/zeppelin/builds/149153003
CI is not green yet, but at least
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/932
@Leemoonsoo Just pushed the asked changes regarding licenses.
---
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 Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/1261
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1261
CI seems to not beeing triggered on this branch :\
Trying locally with `time mvn package -pl "\!zeppelin-distribution"`
---
If your project is set up for it, you can reply to this email and
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1248
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 project does not have this feature
enabled and wishes so, or
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/1261
[ZEPPELIN-1264] [HOTFIX] Fix CI test failure with Failed to create
interpreter: org.apache.zeppelin.interpreter.remote.mock.MockInterpreterA
### What is this PR for?
Fix CI test failure
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1259
@bzz Thanks for quick review. Updated the indent.
This should be merged to branch-0.6 as well
---
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/1227
Looks great to me! Merging if there is no further discussion.
> Does this needs documentation? no
I think it does ;)
---
If your project is set up for it, you can reply to this
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1255
You can use --conf spark.jars= and --conf
spark.jars.packages= instead
---
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 mfelgamal commented on the issue:
https://github.com/apache/zeppelin/pull/1255
I think that the livy itself doesn't support add (--jars/--packages
property in spark-submit)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user onkarshedge commented on the issue:
https://github.com/apache/zeppelin/pull/1231
@bzz . I will post a diagram of communication for clear view.
`BittorrentNotebookRepo` does not implement/override any `NotebookRepo`
methods. It uses all VFSNotebookRepo methods.
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1254
@bzz i believe we decided not to merge it for now. I'll revise it and let
you know
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1231
@onkarshedge on the architecture side - could you please elaborate on the
design that you choose, especially regarding `TorrentServlet` - why do you need
a new servlet? What are the parts that
Github user onkarshedge commented on the issue:
https://github.com/apache/zeppelin/pull/1231
@khalidhuseynov thanks for helping.
To actually see `TorrentSocket` in action I have to create
a`WebSocketCreator` and a `Servlet`, then add this to jetty server , [at this
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1254
@khalidhuseynov thank you!
Do you suggest merging this guy now and taking care of Revision separation
later?
---
If your project is set up for it, you can reply to this email and have your
Kavin Kumar created ZEPPELIN-1268:
-
Summary: As an enduser, I would like to embed paragraph and
refresh the content of it in my custom UI
Key: ZEPPELIN-1268
URL:
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1260
\cc @jongyoul 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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1248
ð for having a test!
Looks good to me, modulo the issue raised above.
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1260
ZEPPELIN-1267. PySparkInterpreter doesn't work in spark 2.0
### What is this PR for?
PySparkInterpreter doesn't work in spark 2.0 because pyspark and py4j is
not distributed to executors.
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1259
Looks great to me, modulo question above.
Thank you for prompt fix @Leemoonsoo !
Shall this be merged to master or branch-0.6 as well? Asking as JIRA fix
version is not set for this one.
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1258
Would it make sense to change `isParagraphSaveAlready` =>
`wasParagraphAlreadySaved` ?
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1259
[ZEPPELIN-1260] R interpreter doesn't work with Spark 2.0
### What is this PR for?
This PR fixes R interpreter doesn't work with Spark 2.0
### What type of PR is it?
Bug
Jeff Zhang created ZEPPELIN-1267:
Summary: PySparkInterpreter doesn't work in spark 2.0
Key: ZEPPELIN-1267
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1267
Project: Zeppelin
Issue
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1258
---
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 cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1258
ci trigger.
---
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 cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1258
[ ZEPPELIN-1266 ] Code editor Optimization
### What is this PR for?
If the result of many para- graph, the code editor is very slow.
### What type of PR is it?
Improvement
GitHub user cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/1258
[ ZEPPELIN-1266 ] Code editor Optimization
### What is this PR for?
If the result of many para- graph, the code editor is very slow.
### What type of PR is it?
Improvement
CloverHearts created ZEPPELIN-1266:
--
Summary: If the result of many paragraph, the code editor is very
slow
Key: ZEPPELIN-1266
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1266
Project:
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/673
Thanks @minahlee, it works. But any reason to make it hidden by default ?
But it is not intuitive for me that I can click the gear button to display it.
---
If your project is set up for it, you
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1255
No, I mean add interpreter property to allow user to specify jars/packages.
---
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 jimdowling commented on the issue:
https://github.com/apache/zeppelin/pull/1255
Do you mean by modifying the UI on notebook creation or using a param to
selecting the interpreter like
'%livy --jars [path]
---
If your project is set up for it, you can reply to this
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1251
@felixcheung I added description 99d91bf for the user who uses Spark
version earlier than 2.0.0. And planning to add [zeppelin-spark_2.10
installation
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1231
@onkarshedge thanks for this contribution. as i can see, the design issue
on passing `BittorrentNotebookRepo` to `TorrentSocket` was solved by passing
through constructor. also better have
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1257
[Zeppelin-945] Interpreter authorization
### What is this PR for?
This PR is derived from https://github.com/apache/zeppelin/pull/1223
### What type of PR is it?
Sami Jaktholm created ZEPPELIN-1265:
---
Summary: Value of zeppelin.pyspark.python not reflected the python
version Spark executors use on YARN
Key: ZEPPELIN-1265
URL:
86 matches
Mail list logo