[GitHub] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1426 @felixcheung DataFrame won't be affected, as it is handled in %livy.sql (`LivySparkSQLInterpreter`). And since `ZeppelinContext` is not available in livy, so you can not user `z.show(df)` in livy

[GitHub] zeppelin issue #1452: ZEPPELIN-1442. UDF can not be found due to 2 instances...

2016-09-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1452 LGTM @zjffdu Do you mind trigger CI one more time? --- 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

[GitHub] zeppelin issue #1453: [ZEPPELIN-1482] Load updated dependency library on int...

2016-09-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1453 Tested and 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

[GitHub] zeppelin pull request #1452: ZEPPELIN-1442. UDF can not be found due to 2 in...

2016-09-23 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1452 ZEPPELIN-1442. UDF can not be found due to 2 instances of SparkSession is created ### What is this PR for? The issue is that we create 2 SparkSession in zeppelin_pyspark.py (Because we

[GitHub] zeppelin issue #1440: ZEPPELIN-1427. Scala z.show() doesn't work on v.0.6.1

2016-09-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1440 Thanks for the fix. LGTM and merge to master and branch-0.6 if there're no more discussions. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin pull request #1439: [ZEPPELIN-1423] Allow users to specify pre/post...

2016-09-23 Thread agoodm
GitHub user agoodm reopened a pull request: https://github.com/apache/zeppelin/pull/1439 [ZEPPELIN-1423] Allow users to specify pre/post-execute callbacks for interpreters ### What is this PR for? One feature built-in to Jupyter's ipykernel that is currently missing in

[GitHub] zeppelin pull request #1439: [ZEPPELIN-1423] Allow users to specify pre/post...

2016-09-23 Thread agoodm
Github user agoodm closed the pull request at: https://github.com/apache/zeppelin/pull/1439 --- 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] zeppelin issue #1429: ZEPPELIN-1440 Notebook clone: prefix name with "Copy o...

2016-09-23 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1429 @felixcheung I think either we remove "Copy of" or make it localizable. Will wait comments from others before making the choice. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1439: [ZEPPELIN-1423] Allow users to specify pre/post-execut...

2016-09-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1439 That's great - would you like to update the rest of the file to make the style consistent? I understand the default value - I'm only pointing out that there are many users and use

[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-23 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1408 @kavinkumarks This look like a good start, but is it the only case we wan't to handle here? Maybe it's because of the way we test this PR, but if I change the system notebook

[jira] [Created] (ZEPPELIN-1479) Blocking message pending 10000 for BLOCKING

2016-09-23 Thread SanAh Kang (JIRA)
SanAh Kang created ZEPPELIN-1479: Summary: Blocking message pending 1 for BLOCKING Key: ZEPPELIN-1479 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1479 Project: Zeppelin Issue

[GitHub] zeppelin issue #1439: [ZEPPELIN-1423] Allow users to specify pre/post-execut...

2016-09-23 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1439 @felixcheung Sure thing, I can do that in a bit. As for the issue of the default value for `replName`, are you simply referring to the use case for different interpreters, or some variants

[GitHub] zeppelin issue #1445: [Zeppelin-1001] Take care of comma/tab escape in csv/t...

2016-09-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1445 I believe so. --- 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 pull request #1415: [ZEPPELIN-1026] set syntax highlight based on d...

2016-09-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1415 --- 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] zeppelin issue #1446: ZEPPELIN-1263. Should specify zeppelin's spark configu...

2016-09-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1446 @Leemoonsoo @jongyoul @felixcheung Please help 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

[jira] [Created] (ZEPPELIN-1480) Blocking message pending 10000 for BLOCKING

2016-09-23 Thread SanAh Kang (JIRA)
SanAh Kang created ZEPPELIN-1480: Summary: Blocking message pending 1 for BLOCKING Key: ZEPPELIN-1480 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1480 Project: Zeppelin Issue

[jira] [Created] (ZEPPELIN-1481) Blocking message pending 10000 for BLOCKING

2016-09-23 Thread SanAh Kang (JIRA)
SanAh Kang created ZEPPELIN-1481: Summary: Blocking message pending 1 for BLOCKING Key: ZEPPELIN-1481 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1481 Project: Zeppelin Issue

[GitHub] zeppelin issue #1429: ZEPPELIN-1440 Notebook clone: prefix name with "Copy o...

2016-09-23 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1429 Except for the folder issue that @AhyoungRyu reported, I found another bug where the number do not increment: * Make a clone * Go back to the original note * Make another clone, the

[jira] [Created] (ZEPPELIN-1482) Reload local dependencies to interpreter on restart if the library has changed

2016-09-23 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1482: -- Summary: Reload local dependencies to interpreter on restart if the library has changed Key: ZEPPELIN-1482 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1482

[GitHub] zeppelin issue #1448: [MINOR] Remove spaces between intepreter group and nam...

2016-09-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1448 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 #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1426 Thanks for the info, I agree that the long term solution would be using [ support multiple interpreter result](https://issues.apache.org/jira/browse/ZEPPELIN-212). But for livy interpreter, it is

[GitHub] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1426 Add @felixcheung for 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

[GitHub] zeppelin issue #1290: [ZEPPELIN-708]shift positions automatically to ...

2016-09-23 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1290 @mwkang Just tested quickly, I will have to run more test to check the behavior in general. There is some refactoring needed, I left some comments. Otherwise, a more global

[GitHub] zeppelin issue #1449: [MINOR] Change Markdown class name from Markdown to Ma...

2016-09-23 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1449 @Leemoonsoo Thank you for pointing it out. I think changing the class name to `Markdown` is more proper way to avoid compatibility issue. --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-23 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 CI has passed. please 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

[GitHub] zeppelin issue #1429: ZEPPELIN-1440 Notebook clone: prefix name with "Copy o...

2016-09-23 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1429 @r-kamath Thanks for the improvement. It's better than just "Untitled Note #" . But regarding `Copy of`, I agree with @felixcheung. If an user tries to clone a notebook in `this/is`,

[GitHub] zeppelin issue #1339: [ZEPPELIN-1332] Remove spark-dependencies & suggest ne...

2016-09-23 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1339 I think [ZEPPELIN-1101](https://issues.apache.org/jira/browse/ZEPPELIN-1101) can also be resolved by this change. >It looks related to ZEPPELIN-1099 which is about removing

[GitHub] zeppelin pull request #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1431 --- 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] zeppelin pull request #1453: [ZEPPELIN-1482] Load updated dependency library...

2016-09-23 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/1453 [ZEPPELIN-1482] Load updated dependency library on interpreter restart ### What is this PR for? If user specifies library path in interpreter dependency setting, even when the file on this

[GitHub] zeppelin issue #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-09-23 Thread mfelgamal
Github user mfelgamal commented on the issue: https://github.com/apache/zeppelin/pull/1334 @bzz rebased. --- 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 #1416: Rewrite of "Quick Start" page.

2016-09-23 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1416 @rfilmyer Do you need any help? :D --- 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] zeppelin issue #1446: ZEPPELIN-1263. Should specify zeppelin's spark configu...

2016-09-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1446 I think we still support Spark 1.2, 1.3 officially so removing this might be an issue. --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: [Need-Help][Eclipse][Windows] Problem

2016-09-23 Thread Andrea Santurbano
Thanks Lee! Il giorno lun 19 set 2016 alle ore 21:32 moon soo Lee ha scritto: > Hi Andrea, > > I don't know is there a way live debug without rebuild. > > To reduce build time, i leverage IDE, (intellij, eclipse), they can quickly > build single file change into class without

[GitHub] zeppelin issue #1429: ZEPPELIN-1440 Notebook clone: prefix name with "Copy o...

2016-09-23 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1429 @felixcheung removed "Copy of" prefix. @AhyoungRyu fixed folder related issue. @corneadoug fixed sequence bug. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin pull request #1455: Organize env.sh and site.xml config templates a...

2016-09-23 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/zeppelin/pull/1455 Organize env.sh and site.xml config templates and docs ### What is this PR for? Organize env.sh and site.xml config templates and docs ### What type of PR is it? Improvement

[GitHub] zeppelin pull request #1452: ZEPPELIN-1442. UDF can not be found due to 2 in...

2016-09-23 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/1452 ZEPPELIN-1442. UDF can not be found due to 2 instances of SparkSession is created ### What is this PR for? The issue is that we create 2 SparkSession in zeppelin_pyspark.py (Because we

[GitHub] zeppelin issue #1439: [ZEPPELIN-1423] Allow users to specify pre/post-execut...

2016-09-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1439 right, I was referring when the interpreter group is not the default, or there is multiple etc.. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-09-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1350 Thanks for the contribution. @mwkang Could you elaborate little bit more how this code change is related to ZEPPELIN-1351? ZEPPELIN-1351 is about displaying paragraph status when

[GitHub] zeppelin issue #1452: ZEPPELIN-1442. UDF can not be found due to 2 instances...

2016-09-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1452 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] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-23 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1426 I think the problem is if the output is a DataFrame now it can't be turned into a table since %angular is already there because only the first type is picked up? That would seem to me a