Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
Which is fine, with the document there will be a workaround.
And the case that you are talking is true for Postgres, but for Hive if you
will try to execute multiple SQLs in one
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
if all quieries is valid then the result will return from the first. is it
correct?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2347
IMO there should be a workaround if the logic fails.
And for handling error scenarios, here is what I think will happen.
Before recommendation:
![screen shot 2017-05-23 at
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
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
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"
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
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
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
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;
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2326
Tested and works well (both in frontend, backend angular APIs).
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
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
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
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:
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
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
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
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
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 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
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
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
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
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
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
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2326
Thanks @vipul1409 for great contribution.
Thanks @1ambda for the review.
Merge to master if no further discussions.
---
If your project is set up for it, you can reply to this
Hoon Park created ZEPPELIN-2578:
---
Summary: Angular variable change in frontend is not propagated to
other paragraphs angular display.
Key: ZEPPELIN-2578
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2578
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
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
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.
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?
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
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
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
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
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
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
39 matches
Mail list logo