Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2586
@felixcheung yeah.. it's similar but the differentiation is that the
previous one couldn't solve the releasing connection issue perfectly.
---
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2586
@felixcheung This is not a subset of the other PR.
---
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2586
[ZEPPELIN-2930] Releasing session of JDBC interpreter close().
### What is this PR for?
This PR fixes that the session release issue of close() of JDBC interpreter.
### What type
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/2582
---
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2582
CI fails seems not related to this PR
---
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2582
fixed jdbc connection issue.
### What is this PR for?
This PR fixes JDBC connection release problem.
for example whenever i run not executable command like following
![image
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/2581
---
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2581
Fix/jdbc connection release 0.7
### What is this PR for?
This PR fixes JDBC connection release problem.
for example whenever i run not executable command like following
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2576
[hotfix] JDBC connection does not release when got exception.
### What is this PR for?
This PR fixes JDBC connection release problem.
for example when i run not executable command
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2428
LGTM~
I think this PR can be merged.
---
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/2343
@cloverhearts Did you test 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2343
@tinkoff-dwh Sorry it was my problem.
Tested and working 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2343
I think this feature is really cool.
BTW I got following exception whenever i try to run sql completion.
```
==> logs/zeppelin-interpreter-jdbc-jdbc.log <==
ERROR [2017
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2192
@Remilito Could you make this PR based on branch-0.7 branch?
---
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/2192
@Remilito Could you make this PR based on `branch-0.7` branch?
---
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/2192
LGTM..
Sorry for late catch..
Merge if there is 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2207
@cloverhearts It looks also great! ð
---
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/2207
Looks great.
one thing i worrying about is that the `_zc` can be possibly used by user
easily?
how about assigning more unique key word like `__zc__` ?
---
If your project is set up
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2106
@Leemoonsoo fixed the issues you commented and added PandasSql.
and also added `conda`, `pandasql` interpreter test on PR description.
---
If your project is set up for it, you can reply
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/2106
---
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/2106
[WIP] Rewrite PythonInterpreter.
### What is this PR for?
I've been testing the python interpreter and I found at least 4 major
issues in the current python interpreter.
1
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2106
@Leemoonsoo thanks for reviewing.. Let me fix the issues you commented and
add PandasSql.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2106
I just added conda interpreter.
---
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/2106
@zjffdu Thank you for your opinion!
Of course i checked #1495 and I totally agree of extensibility for python
interpreter.
but As you can see, current python interpreter has lots
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2106
Py4j python interpreter
### What is this PR for?
I've been testing the python interpreter and I found at least 4 major
issues in the current python interpreter.
1. not working
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2080
@Leemoonsoo I did. 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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/2080
---
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/1985
@FireArrow Thank you for confirm your idea and thank @zjffdu 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
@FireArrow 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 opened a pull request:
https://github.com/apache/zeppelin/pull/2080
[ZEPPELIN-1964] Layout info is lost after refresh
### What is this PR for?
This PR is for `branch-0.7` of https://github.com/apache/zeppelin/pull/2053.
### What type of PR
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2080
@AhyoungRyu Could you 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
@zjffdu I tested impersonation mode followed
http://zeppelin.apache.org/docs/0.7.0/manual/userimpersonation.html and worked
well.
BTW for now anybody can interrupt to interpreter process
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
@Leemoonsoo @zjffdu @felixcheung @karuppayya Could you guys help to review
about the impersonation mode?
---
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/1985
@FireArrow I think it seems not can be possible to know if impersonation
mode or not in Interpreter side(I couldn't find it) and checking `cancel`
paragraph for impersonation is also not good
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
@FireArrow Yea, right. I understood. Let me fix 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
@FireArrow Note has permission in the zeppelin so if user doesn't have
`write` permission on that note, user can't cancel job.
---
If your project is set up for it, you can reply to this email
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/2053
[ZEPPELIN-1964] Layout info is lost after refresh
### What is this PR for?
This PR fixes layout info is lost after refreshing issue.
### What type of PR is it?
Bug Fix
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/2052
CI has passed. Thank you @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
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/2052
[ZEPPELIN-2048] Can't run first paragraph when personalize mode on.
### What is this PR for?
Problem of ZEPPELIN-2048 was because not set the user id when inserting new
paragraph
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/2052
---
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/2052
[ZEPPELIN-2048] Can't run first paragraph when personalize mode on.
### What is this PR for?
Problem of ZEPPELIN-2048 was because not set the user id when inserting new
paragraph
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1985
---
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/1985
[ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark
Interpreter.
### What is this PR for?
If following code runs with Pyspark Interpreter, there is no way to cancel
except
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
@zjffdu yes, this PR is not kill the python process.
Thank you for reviewing!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1985
[ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark
Interpreter.
### What is this PR for?
If following code runs with Pyspark Interpreter, there is no way to cancel
except
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1985
Hello. Sorry for late response.
@Leemoonsoo Let me fix CI error.
@karuppayya Let me fix `scala` too in another PR.
---
If your project is set up for it, you can reply to this email
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1985
---
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/1985
[ZEPPELIN-2075] Can't stop infinite `while` statement in pyspark
Interpreter.
### What is this PR for?
If following code runs with Pyspark Interpreter, there is no way to cancel
except
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1886
@sotnich Great Job! but still doesn't work with `mysql`.
---
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1962
Tested and working expectably.
Thank you for contributing but Could you make the CI as green?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1886
@sotnich Thank you for great improvement.
I tested `mysql` and `postgresql` and only `postgresql` is working
expectedly like @cloverhearts.
---
If your project is set up for it, you can
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1953
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1924
@Leemoonsoo I just create JIRA issue that is
https://issues.apache.org/jira/browse/ZEPPELIN-2024.
Thanks.
---
If your project is set up for it, you can reply to this email and have your
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1833
LGTM it's working 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 this feature
enabled
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1924
CI has passed.
---
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/1924
Thank you @felixcheung and @Leemoonsoo for reviewing this.
Let me check the CI.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1927
[ZEPPELIN-1994] bugfix of streaming output.
### What is this PR for?
If you run the following code, then streaming output doesn't work properly
from the second run
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1922
[ZEPPELIN-1991] Can't get the PARAGRAPH_APPEND_OUTPUT from the Interpreter.
### What is this PR for?
This PR fixes the problem of streaming events(PARAGRAPH_APPEND_OUTPUT).
It's
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1922
---
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/1924
[Pyspark] checking nhooks count.
### What is this PR for?
if the `nhooks` value is `0` in
this(https://github.com/apache/zeppelin/blob/master/spark/src/main/resources/python
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1845
@prabhjyotsingh Yea I know you didn't change it. That's because it doesn't
have state before.
I agree that we can take care/discuss it in a different PR.
Thanks.
---
If your
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1845
Sorry for late response @prabhjyotsingh, @jongyoul .
Let me review soon.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1858
@corneadoug 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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1877
[ZEPPELIN-1930](HotFix) PythonInterpreter syntax error.
### What is this PR for?
This PR fixes syntax error of PythonInterpreter.
### What type of PR is it?
Bug Fix
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1877
---
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/1877
@agoodm pyspark interpreter is working well.
![image](https://cloud.githubusercontent.com/assets/3348133/21768320/e79bebe0-d62d-11e6-8878-3c837fb7938f.png)
---
If your project is set up
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1877
[ZEPPELIN-1930](HotFix) PythonInterpreter syntax error.
### What is this PR for?
This PR fixes syntax error of PythonInterpreter.
### What type of PR is it?
Bug Fix
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1858
@corneadoug Let me find different of more efficient way.
---
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1867
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1862
Tested and LGTM!
Thanks @jongyoul
---
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/1853
@felixcheung You're right! `evaluate` is dependency package of `knitr`.
Thanks for pointing out. It's my misunderstood.
Let me close this PR.
---
If your project is set up for it, you
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1853
---
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/1853
[ZEPPELIN-1911] Adding how to install evaluate package to build R.
### What is this PR for?
This PR inform how to avoid build failing with R.
### What type of PR
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1853
---
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 closed the pull request at:
https://github.com/apache/zeppelin/pull/1858
---
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/1858
[ZEPPELIN-1229] Clear browser cache with version hashtag.
### What is this PR for?
This PR avoids browser cache.
### What type of PR is it?
Bug Fix
### What
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1858
@corneadoug I updated screenshot and codes.
Could you 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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1858
[ZEPPELIN-1229] Clear browser cache with version hashtag.
### What is this PR for?
This PR avoids browser cache.
### What type of PR is it?
Bug Fix
### What
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1853
[ZEPPELIN-1911] Adding how to install evaluate package to build R.
### What is this PR for?
This PR inform how to avoid build failing with R.
### What type of PR
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1848
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1832
@felixcheung Thank you for review!
Currently only `COMMON_KEY` uses for max line count in JDBC interpreter but
as like you mentioned it seems more make sense to move the code to `open
GitHub user astroshim reopened 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 about
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1832
---
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/1833
Hi.
In my case it seems doesn`t work properly.. What am i missing?
![test1](https://cloud.githubusercontent.com/assets/3348133/21593751/75fa43be-d0cf-11e6-8f83-aa102458dc29.gif
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1831
In my test, I got a ` INFO [2017-01-02 09:08:12,358] ({Exec Default
Executor} RemoteInterpreterManagedProcess.java[onProcessComplete]:164) -
Interpreter process exited 0` error when i try
GitHub user astroshim reopened 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 about
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1832
---
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/1813
@soralee Thank you for reviewing and commenting!!
---
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/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 about
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 well
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 changed
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
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
1 - 100 of 435 matches
Mail list logo