GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/2875
ZEPPELIN-3343. Interpreter Hook is broken
### What is this PR for?
Currently the interpreter hook mechanism is broken. This PR fix this issue
and also add unit test for interpreter hook, (incl
Maxim Belousov created ZEPPELIN-3344:
Summary: Revert comments in JDBC interpreter
Key: ZEPPELIN-3344
URL: https://issues.apache.org/jira/browse/ZEPPELIN-3344
Project: Zeppelin
Issue Type
Github user r-kamath commented on the issue:
https://github.com/apache/zeppelin/pull/2872
LGTM
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2875
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@tinkoff-dwh I just realised that zeppelin already support hooks for
interpreter. I think you don't need to do it specifically for jdbc interpreter,
instead we could introduce one general property
Github user sanjaydasgupta commented on the issue:
https://github.com/apache/zeppelin/pull/2834
Here is one approach: define two new [zeppelin configurable
properties](http://zeppelin.apache.org/docs/latest/install/configuration.html)
as follows:
1) A boolean property named _
GitHub user mebelousov opened a pull request:
https://github.com/apache/zeppelin/pull/2876
ZEPPELIN-3344. Revert comments in queries in JDBC interpreter
### What is this PR for?
The original purpose of https://github.com/apache/zeppelin/pull/2158 was
correct processing of ';'. T
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2834
Thanks for the contribution. Regarding the interpolation property, I think
it might be better to use paragraph level property instead of interpreter level
property. e.g. here's what I imagine
Jeff Zhang created ZEPPELIN-3345:
Summary: Don't load interpreter setting where it exist
interpreter.json but no interpreter-setting.json found
Key: ZEPPELIN-3345
URL: https://issues.apache.org/jira/browse/ZEPPELI
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2872
Thanks for the review will merge this if no more discussion.
---
GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/2877
ZEPPELIN-3345. Don't load interpreter setting when it exist
interpreter.json but no interpreter-setting.json found
### What is this PR for?
It usually happens in dev enviroment. e.g. I have li
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2873
Tried on local, works well. LGTM!
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2877
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2867
@herval
@Leemoonsoo
---
14 matches
Mail list logo