[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread tinkoff-dwh
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 w

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread prabhjyotsingh
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 12

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2347 I think is better without this parameter, what to do with the incorrect result. if there are problems, then think how to solve it. Postgres by default, and many use it and all users will have

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread prabhjyotsingh
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 statemen

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2347 single mode is default and the user will always get an incorrect result if he writes multiple queries to in paragraph --- If your project is set up for it, you can reply to this email and hav

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2347 Which I think is fine, because in that case, we will not try to use this logic, and at least there will be a workaround if the user is facing any problem because of this issue, and we can h

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2347 @felixcheung @prabhjyotsingh have a problem with single mode, if user execute multiple queries (all text executes as single query) then result set is incorrect. I think this se

[GitHub] zeppelin pull request #2145: [ZEPPELIN-2271] encoding password of credential...

2017-05-22 Thread tinkoff-dwh
Github user tinkoff-dwh closed the pull request at: https://github.com/apache/zeppelin/pull/2145 --- 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] zeppelin pull request #2317: [ZEPPELIN-2493] Visualization class should thro...

2017-05-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2317 --- 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 e

[jira] [Created] (ZEPPELIN-2577) No graphs visible on IE 11

2017-05-22 Thread Danish Khan (JIRA)
Danish Khan created ZEPPELIN-2577: - Summary: No graphs visible on IE 11 Key: ZEPPELIN-2577 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2577 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2347 yes for interpreter settings --- 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] zeppelin pull request #2357: [ZEPPELIN-2429] Improve `import notebook` dialo...

2017-05-22 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2357 --- 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 e

[GitHub] zeppelin pull request #2357: [ZEPPELIN-2429] Improve `import notebook` dialo...

2017-05-22 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2357 [ZEPPELIN-2429] Improve `import notebook` dialog style ### What is this PR for? Improve `import notebook` dialog style. See the attached screenshots. ### What type of PR is it?

[GitHub] zeppelin issue #2349: [ZEPPELIN-2214] Set npm installer default npm registry

2017-05-22 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2349 @1ambda, latest updates: - Renamed "registry" to "installer.url" as suggested in your comment - Fixed CI - "ZEPPELIN-2214 is about setting proxy for Helium. But Actually, this PR it

[GitHub] zeppelin issue #2268: [ZEPPELIN-2403] interpreter property widgets

2017-05-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2268 @Leemoonsoo done --- 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 wis

[GitHub] zeppelin issue #2309: [ZEPPELIN-2482] added font size for paragraph

2017-05-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2309 Ready 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 enabled and wishes so

[GitHub] zeppelin issue #2357: [ZEPPELIN-2429] Improve `import notebook` dialog style

2017-05-22 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2357 LGTM! ![image](https://cloud.githubusercontent.com/assets/8110458/26305492/750c4c58-3f2a-11e7-92a3-1809160a790b.png) --- If your project is set up for it, you can reply to this ema

[jira] [Created] (ZEPPELIN-2576) dynamic querying of Temp Table in Spark R returns Error: is.character(x) is not TRUE

2017-05-22 Thread Prasanth (JIRA)
Prasanth created ZEPPELIN-2576: -- Summary: dynamic querying of Temp Table in Spark R returns Error: is.character(x) is not TRUE Key: ZEPPELIN-2576 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2576

[GitHub] zeppelin issue #2359: [ZEPPELIN-2575] Improve `rename note` dialog style

2017-05-22 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2359 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 fe

[GitHub] zeppelin issue #2361: [ZEPPELIN-2131] Restrict `shift + arrow` key in focuse...

2017-05-22 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2361 @1ambda Thanks for checking and improvement this! Let me test it out ASAP! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] zeppelin issue #2362: [ZEPPELIN-2469] HeliumBundleFactoryTest fails on Travi...

2017-05-22 Thread AlexanderShoshin
Github user AlexanderShoshin commented on the issue: https://github.com/apache/zeppelin/pull/2362 We also need to enable the `HeliumBundleFactoryTest` on CI by removing it from `-Dtests.to.exclude` variable in `.travis.yml` --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #2362: [ZEPPELIN-2469] HeliumBundleFactoryTest fails on Travi...

2017-05-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2362 @pravin-dsilva Thanks for the contribution. To enable jenkins, you have to setup your own travis for your zeppelin fork repository. --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2361: [ZEPPELIN-2131] Restrict `shift + arrow` key in focuse...

2017-05-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2361 @soralee 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, o

[GitHub] zeppelin pull request #2362: [ZEPPELIN-2469] HeliumBundleFactoryTest fails o...

2017-05-22 Thread pravin-dsilva
GitHub user pravin-dsilva opened a pull request: https://github.com/apache/zeppelin/pull/2362 [ZEPPELIN-2469] HeliumBundleFactoryTest fails on Travis ### What is this PR for? There are 6 errors occuring with error: "yarn config set cache-folder /tmp/ZeppelinLTest_1494573109020/h

[GitHub] zeppelin issue #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

2017-05-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2347 IMO property in settings of interpreter makes more sense, with a bit of a documentation on how to enable/disable it. cc: @felixcheung --- If your project is set up for it, you can rep