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
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
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/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
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
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
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
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 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
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/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
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
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
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/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 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/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.
### 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 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-submit
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
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 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/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
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
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/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/1212
ZEPPELIN-1222. ClassNotFoundException of SparkJLineCompletion in Spark
Interpreter
### What is this PR for?
ClassNotFoundException happens because not fully qualified class name
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 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
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 Gi
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
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 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_OPTS
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
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/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
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_OPTS
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
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
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1307
\cc @Leemoonsoo @minahlee Is it possible to merge it into 0.6.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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1307
Thanks @minahlee, the CI fail should be irrelevant, it is due to network
issue.
```
[INFO] âââ semver-utils@1.1.1
[ERROR] npm ERR! Linux 3.13.0-40-generic
[ERROR] npm ERR
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1307
@minahlee Please hold the commit, I will have another improvement soon.
---
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 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 appear
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
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_OPTS
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 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
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_OPTS
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
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 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/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-user
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 hadoop
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/1960
Yes, the PR is ready. CI failure is not relevant. Please help include it
in 0.7
---
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/1970
@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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1960
@del680202 Can you try this on `%spark.sql` as well ? I guess `Output
exceeds...` may also cause issues in displaying table format data in
`%spark.sql`
---
If your project is set up for it, you
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1960
@del680202 Can you paste the full stacktrace ?
---
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/1960
I am not sure whether that would help, because `Output exceeds ...` is
generated by zeppelin, not by livy or spark.
https://github.com/apache/zeppelin/blob/master/zeppelin-interpreter/src/main
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1960
hmm, right. `LivySparkSQLInterpreter` delegate interpret to
`LivySparkInterpreter`. So it is possible to get `Output exceeds...` in
`LivySparkSQLInterpreter`. Thanks @del680202
---
If your
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1960
@del680202 Thanks for checking.
---
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/1981
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
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1574
Thanks @wesm , I updated this PR yesterday, but didn't use `git merge`. I
use `git rebase` and `git commit --amend`, does these also the issue as well ?
---
If your project is set up for it, you
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1970
ZEPPELIN-2045. Pass interpreter properties with "spark." as prefix to
SparkConf
### What is this PR for?
Minor change to only pass interpreter properties with "spark." as
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1970
This doesn't conform with spark. Spark don't allow property that doesn't
start with `spark.` [1]
If user want to use any properties in driver side, they should use
properties of Interpreter
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1975
@Leemoonsoo I still on extra blank between outputs which is inconsistent
with zeppelin 0.6
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1574
@Leemoonsoo @felixcheung I update this PR, please help help review that. My
purpose is to make ZeppelinContext extensible is in 2 perspective.
* All the interpreter can has its own
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1975
@Leemoonsoo I don't think the blank line is caused by scala. Here's what I
see in zeppelin 0.6.2 and zeppelin 0.7, both use spark 1.6.2
Zeppelin 0.7
 ?
---
If your project is set up for it, you can reply to this email
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2021
@prabhjyotsingh @felixcheung Please help review, CI failure is not
relevant.
```
TypeError: namedtuple() missing 3 required keyword-only arguments:
'verbose', 'rename', and 'module
1 - 100 of 1861 matches
Mail list logo
