GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1116
minor doc fix
### What is this PR for?
java doc fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zjffdu/incubator
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1116
right, I miss the newbie label. Add it in the next commit.
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1175
use static field logger instead of function
### What is this PR for?
Create static field logger in Paragraph instead of function to make the
code style consistent. This is a trivial and
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1190
ZEPPELIN-1187. Redirect output of the process running interpreter.sh to
log4j
### What is this PR for?
For now the output of process running interpter.sh is lost, it would be
nice to
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1190
\cc @Leemoonsoo Could you help take look at this PR ? Since you are the
original author of this file. Thanks.
---
If your project is set up for it, you can reply to this email and have your
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1184
@prabhjyotsingh
```this is result in proper error message.```
What does this mean ? Is the behavior livy interpreter give proper error
message to client or just recreate a new session
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1190
@jongyoul I just feel it might be a little complicated to redirect it to
zeppelin-.out, as I don't find api to redirect log4j to another file at
runtime.
---
If your project is set up f
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1184
@prabhjyotsingh understood.
But I feel the better solution is to either recreate a new session for user
implicitly or tell the user to restart the livy interpreter as the session is
expired
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1184
@prabhjyotsingh Got it, I would prefer to create new session for user
implicitly and don't expose any livy concept to users. Just try to make it
transparent to users. Anyway, we can do it
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1206
ZEPPELIN-1215. Need to login using keytab and principal before creating
SparkContext in secured cluster
### What is this PR for?
Need to login using keytab and principal before creating
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1206
Might need to check whether it works in pyspark/sparkr.
---
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/1206
pyspark and sprakr both create SparkContext, so this fix also works in
pyspark & sparkr
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1212
ZEPPELIN-1222. ClassNotFoundException of SparkJLineCompletion in Spark
Interpreter
### What is this PR for?
ClassNotFoundException happens because not fully qualified class name is
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1212
@Leemoonsoo Could you help take a look at it ? 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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1163
agree with @felixcheung, use the zeppelin service keytab/principal might
bring some other potential security problems. I think user A would expect to
run shell script as himself, but actually it
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1163
Sorry, I mistake it as hdfs interpreter. For shell interpreter, do we must
have kerberos support ? I think here kinit is just for the hadoop command, so
actually it is for running hdfs command in
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1215
ZEPPELIN-1225. Errors before the last shell command are ignored
### What is this PR for?
The problem is that command "bash -c " will always return 0
as long as the last line of sh
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1215
Find ShellInterpreterTest has some format issue (it is windows format), I
have changed it to unix format in this PR as well.
---
If your project is set up for it, you can reply to this email and
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1215
After checking ShellInterpreter.java, I found other 2 issues.
* Should remove entry from executors after the shell execution is done
otherwise it would accumulate more and more
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1215
Travis build fails due to network issue I think,
```
[INFO] bower@1.7.2 node_modules/bower
[INFO] âââ semver-utils@1.1.1
[ERROR] npm ERR! Linux 3.13.0-40-generic
[ERROR
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1163
@prabhjyotsingh Thanks for clarification.
---
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1215
@Leemoonsoo, mind to take look at it ? 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 commented on the issue:
https://github.com/apache/zeppelin/pull/1189
@Leemoonsoo I pushed another commit to add migration section in
upgrade.md, please help review. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1228
ZEPPELIN-1175. AM log is not available for yarn-client mode
### What is this PR for?
For now, we share the same class path for zeppelin server and remote
interpreter process. The cause the
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1228
@Leemoonsoo
* `$ZEPPELIN_HOME/interpreter/` is not in the classpath,
actually the jars under this folder are in the classpath. Although I can put
all the files under this folder on the
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1189
Thanks @AhyoungRyu , I have updated the doc.
---
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1189
BTW, I will do the windows related change after we make consensus on this
approach.
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1221
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1228
Build fails due to some network issue.
```
[ERROR] Failed to execute goal
com.github.eirslett:frontend-maven-plugin:0.0.25:npm (npm install) on project
zeppelin-web: Failed to run task
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1232
ZEPPELIN-1197. Should print output directly without invoking function print
in pyspark interpreter
### What is this PR for?
For now, user need to invoke print to make the output displayed on
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1206
@prabhjyotsingh Conflicts resolved.
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1237
ZEPPELIN-1242. Should set property SPARK_YARN_MODE and do login beforâ¦
### What is this PR for?
We run zeppelin on spark when spark authentication is turned on, but got
the following
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1237
@prabhjyotsingh I close #1206, and merge it here. 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
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1206
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1206
Close it and merge it to #1237
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1237
Build fails due to network issue.
```
[INFO]
[ERROR] Failed to execute goal
org.apache.maven.plugins:maven
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1232
@Leemoonsoo Thanks for the careful checking. I compare it with jupyter,
only the second case is different. Let me investigate how to fix it.
---
If your project is set up for it, you can reply
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1232
I also compare it with native python repl, the second case is consistent.
So I think this behvior is fine, although it is different from jyputer.
![2016-07-29_0746](https
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1241
@AhyoungRyu I follow the instruction here to build the doc
https://github.com/apache/zeppelin/blob/master/docs/README.md#build-documentation,
but fails. Here's the error after `bundle in
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1241
yes, I run it under `ZEPPELIN_HOME/docs`
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1245
ZEPPELIN-1225. Errors before the last shell command are ignored
What is this PR for?
The problem is that command "bash -c " will always return 0 as long as the
last line of sh
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1245
@Leemoonsoo this is for branch-0.6, the test fail seems irrelevant.
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1255
I would not recommend user to do that to add extra library. Because most of
time zeppelin user don't know the cluster details like which extra library are
located in the cluster machines. I
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1241
Thanks @AhyoungRyu , the new UI lgtm. BTW, this link helps me in case
someone else need this,
http://pragmaticstudio.com/blog/2010/9/23/install-rails-ruby-mac
---
If your project is set up
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1255
@jimdowling Right, there's no ZeppelinContext in livy right now. Would
allowing user to specify jars/packages when creating livy session more general
? (like --jars/--packages in spark-s
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/673
@minahlee Is there any follow up change about this ? I didn't find the
repository management section in the interpreter page.
### What is this PR for?
SparkSession.wrapped is only for spark 2.0 preview, it is not supported in
spark 2.0 release. So
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1245
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1232
Sure, let me do it for python interpreter 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 project does not have
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1232
Just take a look at python interpreter, it uses a different way with
pyspark interpreter. Might need to take more time to investigate that.
---
If your project is set up for it, you can reply to
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1278
ZEPPELIN-1287. No need to call print to display output in PythonInterâ¦
### What is this PR for?
It is not necessary to call print to display output in PythonInterpreter. 2
main changes
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1260
I also use the latest master and build zeppelin with this command:
```mvn clean package -Pspark-2.0 -Ppyspark -Psparkr -DskipTests
```, and then export `SPARK_HOME` to where spark-2.0 is
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1189
\\cc @Leemoonsoo 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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1278
\\cc @bzz @Leemoonsoo ready for review, not sure whether the test fail is
relevant. CI is very unstable recently, we might need to fix it asap :smiley:.
---
If your project is set up for it
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1269
@Leemoonsoo PR rebased.
---
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 zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1189
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1300
ZEPPELIN-1305. Fix bug of ZEPPELIN-1215
### What is this PR for?
It is my bad, Before commit ZEPPELIN-1215, I change
`getProperty().getProperty(key) != null` to `getProperty().contains(key
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1300
\cc @prabhjyotsingh
---
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 zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1189
---
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 zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1189
---
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 zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1189
ZEPPELIN-1185. ZEPPELIN_INTP_JAVA_OPTS should not use ZEPPELIN_JAVA_OPTS
### What is this PR for?
Don't use ZEPPELIN_JAVA_OPTS as the default value of ZEPPELIN_INTP_JAVA
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1278
---
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 zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1278
ZEPPELIN-1287. No need to call print to display output in PythonInterpreter
### What is this PR for?
It is not necessary to call print to display output in PythonInterpreter. 2
main changes
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1307
ZEPPELIN-1311. Typo in ZEPPELIN-1197
### What is this PR for?
My bad, it should be `Exception` rather than `Execution`
---
If your project is set up for
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1307
oh, sorry, I didn't realize `ZEPPELIN-1197` is not in 0.6. Then let's merge
this to master only.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1278
+1 for merging this into branch-0.6
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1316
One suggestion is asking the 3rd party to specify their zeppelin version
for this demo. If they update the demo, they should also update the version
accordingly.
---
If your project is set up
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1265
@jongyoul Does that mean the login user is the process owner ? Because it
matters for security reason.
e.g.
* For spark interpreter, it would be better to launch the yarn app as the
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1320
Just one minor code style issue, rest LGTM, Thanks @minahlee for taking
over this.
---
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1301
Definitely should remove profile yarn. I got classpath issue when enabling
yarn profile.
And we need to update the docs accordingly.
---
If your project is set up for it, you can reply
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1333
[ZEPPELIN-1334] Environment variable defined in interpreter setting doesn't
take effect
### What is this PR for?
I define SPAKR_HOME in interpreter setting, but it doesn't take eff
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1333
@Leemoonsoo Please help to 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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1265
@sourav-mazumder 's suggestion is about another point of multiple user
support, this is might not be a trivial task to implement in this PR. I think
we have a discussion about the multiple
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1318
@astroshim I can use spark 2.0 and hadoop 2.7 successfully. I hit this
issue when building zeppelin with profile yarn enabled. So please don't enable
yarn profile otherwise you will get h
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1265
Create umbrella ticket for multiple user support.
https://issues.apache.org/jira/browse/ZEPPELIN-1337
---
If your project is set up for it, you can reply to this email and have your
reply
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1189
ping @Leemoonsoo in case you miss 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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1318
The jira title seems a little confusing to me. The PR is for running spark
on yarn by docker, but I don't think users will use docker for production.
---
If your project is set up for it
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1318
It would be better to change the title to reflect the docker. I think we
should mention docker is only for small experimental enviroment rather than
production environment. Besides that, I don&#
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1318
Thanks @astroshim, I have no other concerns.
---
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1333
@bzz Sure, let me know if I need to create another PR for 0.6
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1347
ZEPPELIN-1284. Unable to run paragraph with default interpreter
### What is this PR for?
This issue happens when SPARK_HOME is not defined. In this case, you are
using spark 2.0 and scala
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1333
@Leemoonsoo I have added unit test, please help review. failed test seems
irrelevant.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1333
---
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 zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1333
[ZEPPELIN-1334] Environment variable defined in interpreter setting doesn't
take effect
### What is this PR for?
I define SPAKR_HOME in interpreter setting, but it doesn't t
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1364
[MINOR] Remove unnecessary question mark
### What is this PR for?
Just remove the unnecessary question mark.
### What type of PR is it?
[Documentation]
### Todos
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1364
\cc @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 project does not have this feature
enabled and wishes so, or
1 - 100 of 2003 matches
Mail list logo
