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

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2268 If i list all combinations of widget-type in this PR, | widget | type | | - | - | | input | **string** | | input | **number** | | **textarea**

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

Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2268 @Leemoonsoo first three comments ... I think it's not complicated. IMO, it's the part we should design carefully since it's hard to change (backward compatibility) ...

[GitHub] zeppelin issue #2322: [ZEPPELIN-2502] RemoteInterpreterServer hang forever d...

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2322 Merge to master and branch-0.7 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 hav

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

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2268 > the separation discussed above in the comments maybe i missed, @tinkoff-dwh could you point out the comment? --- If your project is set up for it, you can reply to this email and hav

[GitHub] zeppelin issue #2227: [ZEPPELIN-2359] Support Spell as Display

Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2227 Hi @echarles. Sorry for late comment. I agree it's quite hard to implement. Since `InterpreterOutput` class is widely used. 1. We need to add a variable to `RemoteInterpreterConte

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

Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2349 @necosta Thanks for quick updating! I have one more question. **Is there a proxy option for npm packages? (not npm itself)** Since if we an enable helium package, it will download spe

[jira] [Created] (ZEPPELIN-2573) Improve Navbar

Hoon Park created ZEPPELIN-2573: --- Summary: Improve Navbar Key: ZEPPELIN-2573 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2573 Project: Zeppelin Issue Type: Improvement Repor

[GitHub] zeppelin issue #2352: [ZEPPELIN-2570] fix: indistinguishable directory icon ...

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2352 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 fea

[GitHub] zeppelin issue #2352: [ZEPPELIN-2570] fix: indistinguishable directory icon ...

Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2352 appveyor failed. but irrelevant. --- 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 a

[GitHub] zeppelin issue #2323: [ZEPPELIN-2411] Improve Table

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2323 GTM. Thanks @1ambda for great improvement. 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 Gi

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

Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2347 hmm, I see, 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] zeppelin issue #2352: [ZEPPELIN-2570] fix: indistinguishable directory icon ...

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2352 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] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Hi, I have a question. I'm not sure but classes under org.apache.zeppelin.interpreter.graph is used only for testing it. Can you verify it? If it does, can you move these classes under

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

Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2347 @felixcheung +1 for defaulting to the single query. And for all of your questions answer is yes. ![1845](https://cloud.githubusercontent.com/assets/674497/21741685/1cbfb62e-d50

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

Github user conker84 commented on the issue: https://github.com/apache/zeppelin/pull/2125 No news? --- 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 th

[GitHub] zeppelin issue #2351: [ZEPPELIN-2544] clean task doesn't work in zeppelin-we...

Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2351 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 #2347: [Bug Fix][ZEPPELIN-2554] sql parser fix (backslash)

Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2347 Perhaps multiple queries was the reason we tried to parse the text (we really need to document these things better) - though I'm not sure how it would work exactly? Do you get multiple table i

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 @sohaibiftikhar Sure, I didn't test it yet as I had a talk in ApacheCon last Wednesday. I'll handle it soon. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2341: ZEPPELIN-2543: Add ability to change the log level at ...

build [job](https://s3.amazonaws.com/archive.travis-ci.org/jobs/233987208/log.txt?X-Amz-Expires=30&X-Amz-Date=20170519T142353Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170519/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host

[GitHub] zeppelin issue #2326: [ZEPPELIN-2519] Disable watchers not in viewport

Date=20170519T140437Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170519/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host&X-Amz-Signature=528cbb815c291c20ab8f4a2418e762fee0d63d63af8cbf652a1262ce8dd2ef9a --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2354: Allow group/role based authentication using LdapRealm ...

Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2354 \cc @khalidhuseynov --- 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 pull request #2354: Allow group/role based authentication using Lda...

GitHub user sohaibiftikhar opened a pull request: https://github.com/apache/zeppelin/pull/2354 Allow group/role based authentication using LdapRealm [ZEPPELIN-2539] ### What is this PR for? Currently allowing authentication for selected roles/groups of an LDAP realm is not possi

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

Github user sohaibiftikhar commented on the issue: https://github.com/apache/zeppelin/pull/932 @jongyoul I wish to open a dependent pull request for review. Is it possible to merge this branch if everything is working fine. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin pull request #2353: missing try catch

GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/2353 missing try catch ### What is this PR for? Currently we can not build on branch-0.7 The cause is a missed try catch. So, I fixed this part. ### What type of PR is it?

[jira] [Created] (ZEPPELIN-2572) Select multiple paragraphs for running, deleting, moving, etc

Lucas Partridge created ZEPPELIN-2572: - Summary: Select multiple paragraphs for running, deleting, moving, etc Key: ZEPPELIN-2572 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2572 Project:

[jira] [Created] (ZEPPELIN-2571) Add a "Run to here" option on paragraphs

Lucas Partridge created ZEPPELIN-2571: - Summary: Add a "Run to here" option on paragraphs Key: ZEPPELIN-2571 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2571 Project: Zeppelin Iss

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

Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2349 Thanks @andreaTP , @1ambda , @felixcheung @andreaTP , I confused everyone with soft-coding the npm value. "npm" is the npm package name. So it should remain hard-coded. @1ambda ,

[GitHub] zeppelin pull request #2326: [ZEPPELIN-2519] Disable watchers not in viewpor...

GitHub user vipul1409 reopened a pull request: https://github.com/apache/zeppelin/pull/2326 [ZEPPELIN-2519] Disable watchers not in viewport ### What is this PR for? Currently all the watchers are enabled by default. I came across this github project https://github.com/wix/angul

[GitHub] zeppelin pull request #2326: [ZEPPELIN-2519] Disable watchers not in viewpor...

Github user vipul1409 closed the pull request at: https://github.com/apache/zeppelin/pull/2326 --- 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 i

[GitHub] zeppelin issue #2348: ZEPPELIN-2555: Run all paragraphs API does not run all...

Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2348 Merging this if 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. If your project does not have this featur