[GitHub] zeppelin issue #2333: Notebook: (Web) HDFS as a backend storage (Read & Writ...

2017-05-11 Thread hayssams
Github user hayssams commented on the issue: https://github.com/apache/zeppelin/pull/2333 @jongyoul moved HDFSCommand to zeppelin-interpreter --- 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 t

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

2017-05-11 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2323 @guptarajat Hi, refer the last GIF. "persist" means that all state of table is saved in the note.josn. For example, sorting state will be saved after this PR but it is not in current master. ---

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

2017-05-11 Thread vipul1409
Github user vipul1409 commented on the issue: https://github.com/apache/zeppelin/pull/2326 Even latest master is breaking with exactly same error. I am not able to figure out the issue. Any help or suggestions will be great. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #2334: [HOTFIX] Set interpreter context in zeppelin context f...

2017-05-11 Thread FireArrow
Github user FireArrow commented on the issue: https://github.com/apache/zeppelin/pull/2334 I looked at the tests, and they fail at the same place even after re-running them, but it seems unrelated to my changes. Could someone else also take a look and see if I'm missing something? -

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

2017-05-11 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2268 CI red https://travis-ci.org/tinkoff-dwh/zeppelin/builds/231079773 another tests SecurityRestApiTest.testGetUserList:69 » JsonSyntax java.lang.IllegalStateExce... Se

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

2017-05-11 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2322 am I supposed to do anything else? --- 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 enabl

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

2017-05-11 Thread sohaibiftikhar
Github user sohaibiftikhar commented on the issue: https://github.com/apache/zeppelin/pull/932 @jongyoul Should I raise this as a separate issue because this seems unrelated to this PR? Apparently the ticket is not held when using LdapRealm while with LdapGroupRealm it is fine. Howeve

[jira] [Created] (ZEPPELIN-2530) Zeppelin user impersonation with domain name suffix is failing

2017-05-11 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-2530: - Summary: Zeppelin user impersonation with domain name suffix is failing Key: ZEPPELIN-2530 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2530 Project:

[GitHub] zeppelin pull request #2337: ZEPPELIN-2530: Zeppelin user impersonation with...

2017-05-11 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/2337 ZEPPELIN-2530: Zeppelin user impersonation with domain name suffix is failing ### What is this PR for? Basically what happens is, if a user login using full name with suffix then the

[GitHub] zeppelin issue #2334: [HOTFIX] Set interpreter context in zeppelin context f...

2017-05-11 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2334 Yeah, that's seem flaky test. Thanks @FireArrow I will merge it if no more comments. ``` Failed tests: NotebookSecurityRestApiTest.testThatOtherUserCannotAccessNoteIfPermission

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

2017-05-11 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2326 Additionally, Zeppelin supports angular API both in backend, frontend. Could you make sure that this PR doesn't break existing feature? - [Back-end Angular API](https://zeppelin.ap

[jira] [Created] (ZEPPELIN-2531) Angular API in frontend is broken

2017-05-11 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2531: --- Summary: Angular API in frontend is broken Key: ZEPPELIN-2531 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2531 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 @jongyoul I could not run your PR, seems they only work in your enviroment. There are several issue, one of them is * HADOOP_CONF_DIR is not in zeppelin server classpath, so it could not conn

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu I'm adding to read HADOOP_CONF_DIR. Concerning spark, I recommend, for now, to set local[*], and yarn-client. In case of yarn-cluster, I think native approach would be better. I'll push m

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 @jongyoul Thanks for quick response. I am thinking that we could do the yarn cluster mode support in 2 steps: 1. Support yarn-cluster mode for spark interpreter via native spark-submit. 2.

[jira] [Created] (ZEPPELIN-2532) User should get noticed who is editing also

2017-05-11 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2532: --- Summary: User should get noticed who is editing also Key: ZEPPELIN-2532 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2532 Project: Zeppelin Issue Type:

[GitHub] zeppelin pull request #2334: [HOTFIX] Set interpreter context in zeppelin co...

2017-05-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2334 --- 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 issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 @jongyoul If you consider to use the native yarn cluster support of spark, then you may need to check ZEPPELIN-1263 (#1446), as for yarn cluster mode, we have to set configuration before launching

[GitHub] zeppelin issue #1446: ZEPPELIN-1263. Should specify zeppelin's spark configu...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1446 I didn't dig into it but have simple question. Does it support embedded mode? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2329 Great job. Exciting to see this new feature in Zeppelin. My two cents: > through thrift in yarn-cluster mode Cloudera's Spark don't have thrift service and Cloudera doesn't reco

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu Personally, I agree on your opinion. Before I start to do it, I discussed a lot how to support yarn-cluster mode including spark. I concluded it's not much weird to launch spark with yarn

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @Tagar Thanks for reaching it out. In case of Spark, I feel like we need to focus on external spark mode more. In case of `spark-defaults.conf`, AFAIK, because we don't need to deploy spark in al

[GitHub] zeppelin issue #1446: ZEPPELIN-1263. Should specify zeppelin's spark configu...

2017-05-11 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1446 I haven't tried that, but I guess it won't work. Because embedded mode is special case for zeppelin, and zeppelin didn't use spark-submit to launch embedded mode, so I guess it won't work in embedd

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2329 > Cloudera's Spark don't have thrift service and Cloudera doesn't recommend to use that (they quote security is one of the concerns there). @Tagar Sorry for making you confused. The thrift

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu @jongyoul got it - thanks for prompt response. --- 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 t

[GitHub] zeppelin issue #2333: Notebook: (Web) HDFS as a backend storage (Read & Writ...

2017-05-11 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2333 it might be important to call this `webHdfs` instead of `hdfs` there are significant and important differences on what is supported esp. with cloud providers --- If your project is set up

[GitHub] zeppelin pull request #2338: [ZEPPELIN-2122] Add execution time for spell re...

2017-05-11 Thread 1ambda
GitHub user 1ambda opened a pull request: https://github.com/apache/zeppelin/pull/2338 [ZEPPELIN-2122] Add execution time for spell results ### What is this PR for? Add execution time for spell results. See the screenshot section. ### What type of PR is it? [Imp

[jira] [Created] (ZEPPELIN-2533) "Clear paragraph" should reset dateFinished, dateStarted in the paragraph

2017-05-11 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2533: --- Summary: "Clear paragraph" should reset dateFinished, dateStarted in the paragraph Key: ZEPPELIN-2533 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2533 Project:

[GitHub] zeppelin issue #2331: [minor] Replace 'SIMPLE' auth with 'simple' auth every...

2017-05-11 Thread VipinRathor
Github user VipinRathor commented on the issue: https://github.com/apache/zeppelin/pull/2331 Thanks @felixcheung for spending time on this and approving. > but we should fix it to lower case always just to be safe? As per Shiro [doc](https://shiro.apache.org/static/1.

[jira] [Created] (ZEPPELIN-2534) Fallback to table if helium package is not available

2017-05-11 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2534: --- Summary: Fallback to table if helium package is not available Key: ZEPPELIN-2534 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2534 Project: Zeppelin Iss

[jira] [Created] (ZEPPELIN-2535) EXPLAIN is not highlighted and EXPLAIN output should be plain text

2017-05-11 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2535: Summary: EXPLAIN is not highlighted and EXPLAIN output should be plain text Key: ZEPPELIN-2535 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2535 Project: Zeppe

[jira] [Created] (ZEPPELIN-2536) Intermitten Classloading error running SQL in Zeppelin

2017-05-11 Thread Vinay Shukla (JIRA)
Vinay Shukla created ZEPPELIN-2536: -- Summary: Intermitten Classloading error running SQL in Zeppelin Key: ZEPPELIN-2536 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2536 Project: Zeppelin

[GitHub] zeppelin pull request #2336: [MINOR] Change `case` indent

2017-05-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2336 --- 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 issue #2337: ZEPPELIN-2530: Zeppelin user impersonation with domain...

2017-05-11 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2337 The issue is the value could have been just @ or @foo.com So after a split on @ the user could become an empty string or just whitespace --- If your project is set up for it, y

[GitHub] zeppelin issue #2300: [ZEPPELIN-2467] Restoring second travis job functional...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2300 @AlexanderShoshin After merging this PR, we continued to got failed in CI. Can you handle it urgently? If it took some times, it would be better to revert this commit and merge it after fixing it

[GitHub] zeppelin issue #2337: ZEPPELIN-2530: Zeppelin user impersonation with domain...

2017-05-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2337 Sure, I see, have added a check before sending it to the AD server. --- 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 p

[GitHub] zeppelin pull request #2339: [ZEPPELIN-2430] The xAxis text in the line and ...

2017-05-11 Thread soralee
GitHub user soralee opened a pull request: https://github.com/apache/zeppelin/pull/2339 [ZEPPELIN-2430] The xAxis text in the line and area chart overlaps ### What is this PR for? Like in the #2133 case, the xAxis text of area chart and line chart is overlapping. For reusing

[GitHub] zeppelin issue #2339: [ZEPPELIN-2430] The xAxis text in the line and area ch...

2017-05-11 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2339 CI fail is irrelevant this PR. ``` Tests in error: SecurityRestApiTest.testGetUserList:69 » JsonSyntax java.lang.IllegalStateExce... SecurityRestApiTest.testTicket:55 » JsonS

[GitHub] zeppelin issue #2330: [minor] Remove incorrect configuration and fix descrip...

2017-05-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2330 Sure 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,

[GitHub] zeppelin issue #2337: ZEPPELIN-2530: Zeppelin user impersonation with domain...

2017-05-11 Thread VipinRathor
Github user VipinRathor commented on the issue: https://github.com/apache/zeppelin/pull/2337 Looks good to me. +1 --- 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