GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1832
[ZEPPELIN-1884] Prevent NullPointerException when JDBC query.
### What is this PR for?
Sometimes users don't want to set the properties which they don't want to
care
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1813
[ZEPPELIN-1868]fix to not shows up the login button on Anonymous mode.
### What is this PR for?
This PR fixs to not shows up the login button on Anonymous mode.
### What type
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1780
@AhyoungRyu 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
enabled and wishes
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1780
CI has passed after rebase.
---
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 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1780
In my case, It's normal output of results. @AhyoungRyu Did you build this
PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1780
@AhyoungRyu Thank you for review and let me check that problem.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1777
Thanks for contribution!
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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1780
[BugFix] Tutorial note json format.
### What is this PR for?
Since https://github.com/apache/zeppelin/pull/1658, the output format had
changed but note.json of Tutorials didn't ch
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1735
CI is green now. :)
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1752
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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1735
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1735
[ZEPPELIN-1767] Blank paragraph should not be run.
### What is this PR for?
This PR fixes not to run blank paragraph.
### What type of PR is it?
Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1744
Thank you @Leemoonsoo and @bzz help to review.
CI failure is not irrelevant to this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1566
I agree with @khalidhuseynov's opinion.
I was keep wondering it's possible impersonation on `shared` or `scoped`
mode.
My PR is just created for the testing and only teste
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1735
[ZEPPELIN-1767] Blank paragraph should not be run.
### What is this PR for?
This PR fixes not to run blank paragraph.
### What type of PR is it?
Improvement
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1735
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1694
I agree with @prabhjyotsingh and @cloverhearts blocking to change the
permission on a backend side too.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1735
Testcase also added. @AhyoungRyu Please 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1735
Thank you @AhyoungRyu ! I didn't catch the 3rd.
I just fix that and let me add testcase for this.
---
If your project is set up for it, you can reply to this email and have your
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1744
Better output of JDBC Interpreter.
### What is this PR for?
Currently, the output format of the JDBC Interpreter is the table format
with all the results except EXPLAIN statement, It is
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1736
@soralee Thank you for contribution of useful function.
but I am usually using the `Shift + Enter` to run paragraph,
How can be presented this shortcut?
and it seems necessary a
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1735
I just fixed typo.
Thanks @rhymoid and @cloverhearts to review 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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1735
[ZEPPELIN-1767] Blank paragraph should not be run.
### What is this PR for?
This PR fixes not to run blank paragraph.
### What type of PR is it?
Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1719
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1721
Working as expected. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1717
Working well. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1716
It's working as expected. Thank you for great improvement.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1715
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 astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1284
---
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 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/zeppel
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1706
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1707
@Leemoonsoo It was my mistake. I just fixed.
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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1707
Add PythonDockerInterpreter to interpreter group
### What is this PR for?
Recently PythonDockerInterpreter added to PythonInterpreter in
https://github.com/apache/zeppelin/pull/1654
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1679
@zjffdu It's my misunderstanding. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1679
Sorry for late review.
User name is already passed to interpreter.
We can get the user id on
interpreterContext.getAuthenticationInfo().getUser()
I think passing password
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1699
[ZEPPELIN-1724] fix conda version
### What is this PR for?
Because `conda run` command removed since version `4.0.9`,
PythonCondaInterpreter not working after the `conda-4.0.9`.
This
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1685
@AhyoungRyu Thank you for reviewing. I am using Zeppelin master on CentOS-7.
---
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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1685
[ZEPPELIN-1714] Build error of scio Intepreter on Centos
### What is this PR for?
This PR fixes build error of scio Intepreter on CentOS.
### What type of PR is it?
Bug
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1617
@randerzander Could you rebase?
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1671
@minahlee Thank you for proper pointing out! I just fixed 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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1656
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1656
Fix Yarn configuration of Spark2.0
### What is this PR for?
`yarn-client` is deprecated since Spark 2.0, so SparkInterpreter also
support `yarn` as `application master
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1671
@tae-jun Thank you for your 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1656
@felixcheung It makes sense. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1638
Thank you for fixing bug.
It's working as expected and adding test is also great!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1671
Clean unnecessary codes
### What is this PR for?
This PR is for cleaning unnecessary codes.
### What type of PR is it?
Refactoring
### Questions:
* Does the
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1670
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1670
@Leemoonsoo Oh sorry, I didn't know that the value is configurable.
If modification is possible, It's better to close this PR because 30s is
okay in normal case.
Thank you!
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1670
[ZEPPELIN-1700] ConnectException between ZeppelinServer and
RemoteInterpreterProcess.
### What is this PR for?
The default value of interpreter connection timeout is 30s but sometimes 30
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1656
review please~
---
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 astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1656
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1656
Fix Yarn configuration of Spark2.0
### What is this PR for?
`yarn-client` is deprecated since Spark 2.0, so SparkInterpreter also
support `yarn` as `application master
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1656
Fix Yarn configuration of Spark2.0
### What is this PR for?
`yarn-client` is deprecated since Spark 2.0, so SparkInterpreter also
support `yarn` as `application master`.
### What
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
CI is green!
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1636
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1644
[ZEPPELIN-1670] Disabling login button double click.
### What is this PR for?
This PR is for preventing double click login button.
### What type of PR is it?
Bug Fix
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
@1ambda Thank you 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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1636
Could you also update the `yarn` docker file path to
`$ZEPPELIN_HOME/scripts/docker/spark-cluster-managers/spark_yarn_cluster` ?
---
If your project is set up for it, you can reply to this
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
@1ambda I added `placeholder` message for `Entity`. Does it make sense?
` in mysql, `auto commit` should be `false`
otherwise it throws `SQLException` like following.
```
java.sql.SQLException: Can't call commit when autocommit
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1465
please review 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 project does not have this feature
enabled and
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1465
[ZEPPELIN-1500] pyspark completion doesn't work.
### What is this PR for?
@fanlu reported bug on https://github.com/apache/zeppelin/pull/530.
This PR fixes bug of che
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1465
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1465
re-trigger CI
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1465
[ZEPPELIN-1500] pyspark completion doesn't work.
### What is this PR for?
@fanlu reported bug on https://github.com/apache/zeppelin/pull/530.
This PR fixes bug of che
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1465
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1465
rebuild CI
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1465
[ZEPPELIN-1500] pyspark completion doesn't work.
### What is this PR for?
@fanlu reported bug on https://github.com/apache/zeppelin/pull/530.
This PR fixes bug of che
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1465
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1339
PySparkInterpreter need a [python library for
pyspark](https://github.com/apache/zeppelin/blob/master/spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java#L483)
so I think we
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@mfelgamal Could you tell me more detail about the issue on `from 0.6.0v to
0.6.2v` ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
CI is green now. 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
enabled and
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
re-build CI
---
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 astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1433
I did rebase and CI has passed. 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@mfelgamal I got following error. Did i missed something?
```
$ docker pull mahmoudelgamal/zeppelin-release:0.6.1
0.6.1: Pulling from mahmoudelgamal/zeppelin-release
985c5f84712b
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
rebuild CI
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1539
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
please 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 so
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@mfelgamal Ping.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1564
@felixcheung I fixed it and CI has passed. Thank you very much!
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1564
[ZEPPELIN-1585] Testcase for PySparkInterpreter.
### What is this PR for?
This PR adds PySparkInterpreter testcase.
### What type of PR is it?
Improvement
### What
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1564
---
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1564
[ZEPPELIN-1585] Testcase for PySparkInterpreter.
### What is this PR for?
This PR adds PySparkInterpreter testcase.
### What type of PR is it?
Improvement
### What
101 - 200 of 541 matches
Mail list logo
