[GitHub] zeppelin issue #1511: [ZEPPELIN-1544]upgrade spark version to 2.0.1 as profi...

2016-10-13 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1511 LGTM. Merging if there's 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 fea

[GitHub] zeppelin pull request #1510: [ZEPPELIN-675] "No data available" in invalid g...

2016-10-13 Thread snaveenp
Github user snaveenp closed the pull request at: https://github.com/apache/zeppelin/pull/1510 --- 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 #1517: [MINOR] ui improvement

2016-10-13 Thread baekhoseok
GitHub user baekhoseok opened a pull request: https://github.com/apache/zeppelin/pull/1517 [MINOR] ui improvement ### What is this PR for? * configuration sorting * remove tooltip in job menu * change repository infomation icon ### What type of PR is it?

[GitHub] zeppelin pull request #1510: [ZEPPELIN-675] "No data available" in invalid g...

2016-10-13 Thread snaveenp
GitHub user snaveenp reopened a pull request: https://github.com/apache/zeppelin/pull/1510 [ZEPPELIN-675] "No data available" in invalid graph types for sql queries ### What is this PR for? Overriding graph message 'No data available' to 'Invalid Data, check graph settings'

[jira] [Created] (ZEPPELIN-1550) Import note closing refresh error

2016-10-13 Thread Yunho Maeng (JIRA)
Yunho Maeng created ZEPPELIN-1550: - Summary: Import note closing refresh error Key: ZEPPELIN-1550 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1550 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-13 Thread spektom
Github user spektom commented on the issue: https://github.com/apache/zeppelin/pull/1447 @gss2002 you shouldn't be concerned about performance as this code only runs when command is executed. --- If your project is set up for it, you can reply to this email and have your reply appear

[jira] [Created] (ZEPPELIN-1549) NotebookID in the source code should be NoteID

2016-10-13 Thread Hyeonjin Jung (JIRA)
Hyeonjin Jung created ZEPPELIN-1549: --- Summary: NotebookID in the source code should be NoteID Key: ZEPPELIN-1549 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1549 Project: Zeppelin I

[GitHub] zeppelin issue #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1471 I don't have more comments. \cc @Leemoonsoo @felixcheung if they have comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Reg :: Issue while raising pull request

2016-10-13 Thread Rajarajan Ganesan
Hi, When i was working on an issue, i tried to create pull request. i ended up having a issue like below The steps i followed while doing that 1. git add 'files' 2. git commit 3. git push origin master After that i was following the rebase process, once i resolved the conflicts, i pushed the

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1503 That's correct for LivySparkInterpreter, LivyPySparkInterpreter and LivySparkRInterpreter. Because they use FIFOScheduler. If you check the implenentaion of FIFOScheduler, there's only one thread i

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 Also I have some concerns on using Exceptions to as goto's as there is definite performance issues that occur by using exceptions... http://javarevisited.blogspot.com/2013/03/0-exception-handling-

[GitHub] zeppelin issue #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exceptio...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1516 @zjffdu and @prabhjyotsingh here is the 404 NestedRuntime Fix --- 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 n

[GitHub] zeppelin pull request #1516: ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 E...

2016-10-13 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1516 ZEPPELIN-1546 - Zeppelin Livy Interpreter 404 Exception not caught Kerberos Enabled ### What is this PR for? ZEPPELIN-1546: Zeppelin Livy Interpreter 404 Exception not caught with Kerberos

How to add custom syntax highlight ?

2016-10-13 Thread Jeff Zhang
I am working on Pig Interpreter, but unfortunately https://github.com/ajaxorg/ace-builds/tree/master/src-noconflict doesn't support pig, does anyone know other approaches to support custom syntax hightlight except adding pig support in ACE editors ? Thanks.

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu this may be me misunderstanding the code. but if you have 10 users in Zeppelin all accessing PySpark or SparkR there is no possibility of multiple users stepping on the map? So you are say

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1503 That's should be fine to use HashMap for SparkR and PySpark, as their scheduler is FIFOScheduler which only one thread will access the map at one time. But it would be better to use `ConcurrentHash

[GitHub] zeppelin pull request #1515: [ZEPPELIN-916] Apply new mechanism to KylinInte...

2016-10-13 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1515 [ZEPPELIN-916] Apply new mechanism to KylinInterpreter ### What is this PR for? Applying for new registration mechanism for Apache Kylin ### What type of PR is it? [Feature]

[jira] [Created] (ZEPPELIN-1548) ConcurrentModificationException when trying to create JDBC interpreter

2016-10-13 Thread Xi Shen (JIRA)
duler org.apache.zeppelin.interpreter.remote.RemoteInterpretershared_session1000964802 INFO [2016-10-14 02:35:28,584] ({pool-1-thread-7} Paragraph.java[jobRun]:252) - run paragraph 20161013-101834_778249242 using jdbc org.apache.zeppelin.interpreter.LazyOpenInterpreter@3e43ac44 INFO [2016-10-14 02:35:28,591] ({pool-1-thread-7} RemoteInterpreterProcess.java[reference

[GitHub] zeppelin pull request #1514: ZEPPELIN-1516. NPE LivySparkSQLInterpreter thro...

2016-10-13 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1514 ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with %livy.sql interpreter ### What is this PR for? This PR fixed NPE issue in `LivySparkSQLIntepreter`, also it resolves the multiple thread

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu Not a problem. Also I assume we will want to fix SparkR and PySpark also to use ConcurrentHashMaps since they could have multiuser's hitting it. I noticed some of the other interpreters us

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1503 Thanks @gss2002 for checking ZEPPELIN-1430, I will update that in another PR. --- 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 pull request #1512: Zeppelin 1516 1546

2016-10-13 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1512 --- 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 #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu Also those new maps added with ZEPPELIN-1430 those should be changed to ConcurrentHashMap. Basically any map that is static with gets/puts/removes etc should be ConcurrentHashMap if not tr

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

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1426 @zjffdu and @prabhjyotsingh the new static Maps can these be converted to ConcurrentHashMaps's to prevent possible future contention with multiple users/threads. https://dzone.com/article

[GitHub] zeppelin issue #1447: [ZEPPELIN-1293] Re-create Livy session if it's lost

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1447 @spektom and @zjffdu is it possible to add a null check also. I've been doing some debugging over the past few days and certain situations can cause nulls to be returned and in theory if a null is

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu The change you made solved the issue. I see difference now. Your quick workaround is good to go. I am going to fix the LivyHelper to handle the 404 with NestedException if that works under

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu testing the changes again with another build. I see a slight difference between your's and mine. Testing again ill advise shortly --- If your project is set up for it, you can reply to th

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1503 @gss2002 Isn't [above change](https://github.com/zjffdu/zeppelin/commit/4a24a254b21795cae727fde90cd22643a793ff09) use `Initialization on Demand` mentioned in the stackoverflow post ? And could you

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 @zjffdu I tried that same code multiple times and it did not work to be completely honest.. I have an email out to a JDK Developer/Debugger for one of the large JDK vendors to find out why this is

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1503 @gss2002 Do you think [this change](https://github.com/zjffdu/zeppelin/commit/4a24a254b21795cae727fde90cd22643a793ff09) work for you ? Even this is a temp solution IMO. As I mentioned before, ther

[GitHub] zeppelin issue #1513: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1513 @khalidhuseynov made the requested changes and updated documentation. Please let me know what you think. Also I will be willing to create a jira and move the other Realms. --- If your project is

[GitHub] zeppelin pull request #1513: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-13 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1513 ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm ### What is this PR for? ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm: Provides LdapRealm Functional

[GitHub] zeppelin issue #1493: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1493 @khalidhuseynov I'm going to make the changes and move this under realms. I will also close this pull request and open a new one --- If your project is set up for it, you can reply to this e

[GitHub] zeppelin pull request #1493: ZEPPELIN-1472 - Create new LdapRealm based on A...

2016-10-13 Thread gss2002
Github user gss2002 closed the pull request at: https://github.com/apache/zeppelin/pull/1493 --- 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 #1502: [gh-pages] Separate "Supported Interpreters" table fro...

2016-10-13 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1502 Merge if there is 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 feature ena

[GitHub] zeppelin issue #1503: NPE LivySparkSQLInterpreter thrown with %livy.sql inte...

2016-10-13 Thread gss2002
Github user gss2002 commented on the issue: https://github.com/apache/zeppelin/pull/1503 Also Static HashMaps are not threadsafe with multiple puts in the case of multiple users. Hence why I built out Singletons to keep track of these objects to guarantee only one instance gets create

[GitHub] zeppelin issue #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-13 Thread ravwojdyla
Github user ravwojdyla commented on the issue: https://github.com/apache/zeppelin/pull/1471 @AhyoungRyu @zjffdu code was reviewed by @nevillelyh, build looks green. Any other comments? --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[jira] [Created] (ZEPPELIN-1547) Zeppelin cannot be accessed from the internet

2016-10-13 Thread Ramiro-Ernesto (JIRA)
Ramiro-Ernesto created ZEPPELIN-1547: Summary: Zeppelin cannot be accessed from the internet Key: ZEPPELIN-1547 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1547 Project: Zeppelin

[GitHub] zeppelin pull request #1512: Zeppelin 1516 1546

2016-10-13 Thread gss2002
GitHub user gss2002 opened a pull request: https://github.com/apache/zeppelin/pull/1512 Zeppelin 1516 1546 ### What is this PR for? Livy Interpreter gets a NestedRuntimException when running with Kerberized components. ERROR [2016-10-11 22:44:47,769] ( {pool-2-thread-11}

[jira] [Created] (ZEPPELIN-1546) Zeppelin Livy Interpreter Session Management Recovery

2016-10-13 Thread Greg Senia (JIRA)
Greg Senia created ZEPPELIN-1546: Summary: Zeppelin Livy Interpreter Session Management Recovery Key: ZEPPELIN-1546 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1546 Project: Zeppelin

[jira] [Created] (ZEPPELIN-1545) Cannot select data from table

2016-10-13 Thread Pavel Genevski (JIRA)
Pavel Genevski created ZEPPELIN-1545: Summary: Cannot select data from table Key: ZEPPELIN-1545 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1545 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-13 Thread ravwojdyla
Github user ravwojdyla commented on the issue: https://github.com/apache/zeppelin/pull/1471 @nevillelyh (the creator of scio) - could you 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

[GitHub] zeppelin pull request #1511: [ZEPPELIN-1544]upgrade spark version to 2.0.1 a...

2016-10-13 Thread LantaoJin
GitHub user LantaoJin opened a pull request: https://github.com/apache/zeppelin/pull/1511 [ZEPPELIN-1544]upgrade spark version to 2.0.1 as profile spark-2.0 ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check

[jira] [Created] (ZEPPELIN-1544) upgrade default spark-2.0 version to spark-2.0.1

2016-10-13 Thread Lantao Jin (JIRA)
Lantao Jin created ZEPPELIN-1544: Summary: upgrade default spark-2.0 version to spark-2.0.1 Key: ZEPPELIN-1544 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1544 Project: Zeppelin Issue

[GitHub] zeppelin issue #1500: [ZEPPELIN-1371]add text/numeric conversion support to ...

2016-10-13 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1500 @corneadoug I have changed the test case. Could you 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 proje

Re: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)

2016-10-13 Thread DuyHai Doan
Hello, the link https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0. 6.2-rc2/ is a Zeppelin compiled with Scala 2.11. Do you have the same for Scala 2.10 ? On Thu, Oct 13, 2016 at 9:59 AM, Jeff Zhang wrote: > +1, Run tutorial note successfully in embedded mode and yarn-client. > > > > Min

[GitHub] zeppelin issue #1502: [gh-pages] Separate "Supported Interpreters" table fro...

2016-10-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1502 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 i

[GitHub] zeppelin issue #1509: [Zeppelin-1540] fix note url input placeholder

2016-10-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1509 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

[jira] [Created] (ZEPPELIN-1543) update jackson's version to 2.6.5

2016-10-13 Thread lichenglin (JIRA)
lichenglin created ZEPPELIN-1543: Summary: update jackson's version to 2.6.5 Key: ZEPPELIN-1543 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1543 Project: Zeppelin Issue Type: Wish

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

2016-10-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1426 --- 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 #1265: [ZEPPELIN-1210] Run interpreter per user

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1265 @jongyoul I can see the perUser on interpreter setting page and shiro is enabled, but there's no response when I click it, do I miss anything ? --- If your project is set up for it, you can

[GitHub] zeppelin issue #1265: [ZEPPELIN-1210] Run interpreter per user

2016-10-13 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1265 @jongyoul I've checked now and `SecurityUtils` didn't have correct information, and normally `message` should be having the correct information since routine for authorization based on `pri

Re: [VOTE] Release Apache Zeppelin 0.6.2 (RC2)

2016-10-13 Thread Jeff Zhang
+1, Run tutorial note successfully in embedded mode and yarn-client. Mina Lee 于2016年10月12日周三 下午6:29写道: > Hi dev, > > I propose the following RC to be released for the Apache Zeppelin 0.6.2 > release. > > The commit id is 091086de9400dd1c02ca02acf4180b1bf1e9ede7 which is > corresponds > to the t

[GitHub] zeppelin issue #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1471 @ravwojdyla I left some comments. And I am afraid no one in zeppelin community familiar with scio, could you ask someone else that know scio to review it ? --- If your project is set up for it, y

[jira] [Created] (ZEPPELIN-1542) Cleanup of Note created during jUnit test in zeppelin-server, zeppelin-zengine

2016-10-13 Thread Rajarajan G (JIRA)
Rajarajan G created ZEPPELIN-1542: - Summary: Cleanup of Note created during jUnit test in zeppelin-server, zeppelin-zengine Key: ZEPPELIN-1542 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1542

[GitHub] zeppelin pull request #1510: [ZEPPELIN-675] "No data available" in invalid g...

2016-10-13 Thread snaveenp
GitHub user snaveenp opened a pull request: https://github.com/apache/zeppelin/pull/1510 [ZEPPELIN-675] "No data available" in invalid graph types for sql queries ### What is this PR for? Overriding graph message 'No data available' to 'Invalid Data, check graph settings'

[GitHub] zeppelin issue #1476: ZEPPELIN-335. Pig Interpreter

2016-10-13 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1476 merging if no more 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 this feature enabled an