The current JdbcInterpreter implementation seems too generic for me.
Two main reasons
1. The logic of executing sql in JdbcInterpeter is a little weird. Here's
is what jdbc interpreter would do for "%jdbc(mysql) show tables"
The script part is actually `(mysql) show tables` rather than `show
t
Github user tinkoff-dwh closed the pull request at:
https://github.com/apache/zeppelin/pull/2347
---
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 tinkoff-dwh reopened a pull request:
https://github.com/apache/zeppelin/pull/2347
[Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)
### What is this PR for?
Fix of parser to correctly parse backslash in quotes.
### What type of PR is it?
Bug Fix
Yesha Vora created ZEPPELIN-2579:
Summary: Fix the error message when interpreter is not configured
in notebook
Key: ZEPPELIN-2579
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2579
Projec
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1577
Any updated?
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2343
\cc @cloverhearts @astroshim
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2355
Merge to master if no further discussions
---
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 feat
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2356
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2357
LGTM and merge to master if no further comment
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2358
LGTM and merge to master if no further comment
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2359
LGTM and merge to master if no further comment
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2360
LGTM and Merge to master if no further comment
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2362
Flaky test failure https://issues.apache.org/jira/browse/ZEPPELIN-2361 is
not related to this contribution.
```
Tests in error:
ParagraphActionsIT.testRunOnSelectionChange:3
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2361
Tested and LGTM.
Merge to master if no further comment.
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2338
LGTM and merge to master if no further comment
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2338
@Leemoonsoo No. I have just changed the title again.
---
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 th
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2338
@1ambda is this pr WIP?
---
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 wi
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2363
One more suggestion. Can we support proxy for helium registry file?
https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/HeliumOnlineRegi
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2341
Thanks @karuppayya for the contribution!
LGTM and merge to master if no further comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2351
ping @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
enabled and wishes so, or i
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2356
Good! What about others?
---
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 wish
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2363
Can we add documentation for this? under the security section or somewhere
appropriate.
.
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 you
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2349
If we have both, then it's better.
---
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 necosta commented on the issue:
https://github.com/apache/zeppelin/pull/2349
Thanks @1ambda,
Allow me to disagree :) In my opinion should be: "I hope we have a proxy
option as well as configurations for those URLs"
The advantage of having configurable URLs is that i
GitHub user necosta opened a pull request:
https://github.com/apache/zeppelin/pull/2363
[ZEPPELIN-2152] Added proxy config support in Helium bundle factory
### What is this PR for?
Proxy configuration implementation in Helium bundle factory. Used by Node,
NPM and YarnPkg install
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2349
Personally, I hope we have a proxy option instead of configurations for
those URLs.
That's because, it's hard to configure every URLs for users. (not scalable
solution) There might be more URLs
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
Tried the above:
With multiple queries;
https://cloud.githubusercontent.com/assets/674497/26358329/d46cf470-3fef-11e7-9afc-76e3bb448270.png";>
With single query;
htt
GitHub user necosta reopened a pull request:
https://github.com/apache/zeppelin/pull/2349
[ZEPPELIN-2214] Set npm installer default npm registry
### What is this PR for?
Complements ZEPPELIN-2278, this is a fix for "connection timeout" bug when
trying to run "npm install" behind
Github user necosta closed the pull request at:
https://github.com/apache/zeppelin/pull/2349
---
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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
if jdbc driver does not implement this method (getMoreResults), it will
work like you said will return the first result
---
If your project is set up for it, you can reply to this email and h
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
Sure, I do see this working for Postgres as posted in the screenshot, do
you think this will work for rest of the data sources listed here
http://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/int
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
another solution, but I think it's better
---
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 fea
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2357
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
Yes, correct.
---
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 s
37 matches
Mail list logo
