[GitHub] zeppelin issue #1172: [Zeppelin-1161] Revert precise execution time

2016-07-18 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1172 @SachinJanani There should be a thread about 0.6.1. If we want it to be part of it, I guess it should be added to https://issues.apache.org/jira/browse/ZEPPELIN-1203 --- If your

Re: [jira] [Created] (ZEPPELIN-1207) to build Spark Interpreter occur problem

2016-07-18 Thread Hyung Sung Shim
hi. Maybe it must be the "-Phadoop-2.5" not "-Phadoop 2.5" .(you missed the '-') 2016-07-19 0:20 GMT+09:00 polar wang (JIRA) : > polar wang created ZEPPELIN-1207: > > > Summary: to build Spark Interpreter occur problem >

[GitHub] zeppelin issue #1192: [ZEPPELIN-1189] Get note revision websocket api

2016-07-18 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1192 @bzz fixed the issue with CI. regarding change in api, you're right. it's better not to change api as a part of different PR, so I reverted it back. The main point in changing

[GitHub] zeppelin issue #1157: [ZEPPELIN-1146] Zeppelin JDBC interpreter should work ...

2016-07-18 Thread rja1
Github user rja1 commented on the issue: https://github.com/apache/zeppelin/pull/1157 Thanks for the tip @prabhjyotsingh and for your work, hive is working! I'm running into an issue with jdbc(phoenix) now though, hoping you can help. My interpreter config is listed below.

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-18 Thread babupe
Github user babupe commented on the issue: https://github.com/apache/zeppelin/pull/1170 I think excluding constants.json helped. Let me know if you need anything else from my side. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-18 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1170 Great, thank you! Please, let me make a final pass and I'll get back to you. --- 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

[GitHub] zeppelin pull request #1188: [ZEPPELIN-1183] Replace $window.location by $lo...

2016-07-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1188 --- 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 #1194: [ZEPPELIN-1193] Update Node JS related dependencies to...

2016-07-18 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1194 CI is failure [logs](https://s3.amazonaws.com/archive.travis-ci.org/jobs/145344524/log.txt) is not relevant and is tracked in [ZEPPELIN-862](https://issues.apache.org/jira/browse/ZEPPELIN-862)

[GitHub] zeppelin pull request #1192: [ZEPPELIN-1189] Get note revision websocket api

2016-07-18 Thread khalidhuseynov
Github user khalidhuseynov closed the pull request at: https://github.com/apache/zeppelin/pull/1192 --- 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 #1192: [ZEPPELIN-1189] Get note revision websocket api

2016-07-18 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1192 @bzz addressed the changes and CI is green --- 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

[GitHub] zeppelin issue #1172: [Zeppelin-1161] Revert precise execution time

2016-07-18 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1172 Looks great to me, and CI is green now! --- 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 #1178: [Zeppelin-1167] Group $scope.$on functions

2016-07-18 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1178 Rebased and CI is green. Merging if there is no more 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

[GitHub] zeppelin issue #1187: [ZEPPELIN-1163] Change some parameter name properly in...

2016-07-18 Thread zmhassan
Github user zmhassan commented on the issue: https://github.com/apache/zeppelin/pull/1187 Okay all done. I've committed the fix's for this issue. If there is nothing else then I think this task is complete. @jongyoul Thank you so much --- If your project is set up for it, you can

[GitHub] zeppelin issue #1157: [ZEPPELIN-1146] Zeppelin JDBC interpreter should work ...

2016-07-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1157 @jongyoul Sure let me check. --- 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 #1187: [ZEPPELIN-1163] Change some parameter name properly in...

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1187 @zmhassan Hi, could you please check the style? Zeppelin doesn't have any strict style for now, but basically follows google's guide. Could you please check it? --- If your project is set up

[GitHub] zeppelin pull request #1201: [MINOR] Enable pyspark test in local mode

2016-07-18 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1201 [MINOR] Enable pyspark test in local mode ### What is this PR for? Enabling test for pyspark in local mode ### What type of PR is it? [Improvement] ### Todos * [x]

[GitHub] zeppelin issue #1190: ZEPPELIN-1187. Redirect output of the process running ...

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1190 @zjffdu Ok, I've got it. 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

[GitHub] zeppelin issue #1144: [ZEPPELIN-1128] add try-catch in close() method.

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1144 @voyageth Where have you checked those exceptions? Isn't SqlException enough? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #1190: ZEPPELIN-1187. Redirect output of the process running ...

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1190 I agree to leave logs to file, but how about storing stdout to *.out like zeppelin-*.out? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #930: Make path for .spark-dist folder always accessible [ZEP...

2016-07-18 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/930 No, not ready. None of the envisaged solutions are practicable. --- 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

[GitHub] zeppelin pull request #1200: [ZEPPELIN-1191] Supported legacy way to run par...

2016-07-18 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1200 [ZEPPELIN-1191] Supported legacy way to run paragraph with group name only ### What is this PR for? Preserving legacy way to run paragraph when users use group name only ### What

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-18 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1170 > So, you may have to run the tests manually from a GCE instance after setting a project id in resources/constants.json. >After this, running the test with -Dbigquery.test.exclude='' will start

[GitHub] zeppelin issue #1184: [ZEPPELIN-1159] Livy interpreter gets "404 not found" ...

2016-07-18 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1184 @prabhjyotsingh ```this is result in proper error message.``` What does this mean ? Is the behavior livy interpreter give proper error message to client or just recreate a new session ?

[jira] [Created] (ZEPPELIN-1204) Google Datastore Interpreter for Zeppelin

2016-07-18 Thread Babu Prasad Elumalai (JIRA)
Babu Prasad Elumalai created ZEPPELIN-1204: -- Summary: Google Datastore Interpreter for Zeppelin Key: ZEPPELIN-1204 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1204 Project: Zeppelin

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-18 Thread babupe
Github user babupe commented on the issue: https://github.com/apache/zeppelin/pull/1170 Got it, pushed all the changes in now. --- 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 pull request #1199: [HOTFIX][ZEPPELIN-1169] Fix wrong Guava version

2016-07-18 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/1199 --- 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 #1199: [HOTFIX][ZEPPELIN-1169] Fix wrong Guava version

2016-07-18 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/1199 [HOTFIX][ZEPPELIN-1169] Fix wrong Guava version ### What is this PR for? Fixing the incompatible version for guava ### What type of PR is it? [Hot Fix] ### Todos

[GitHub] zeppelin issue #1197: [ZEPPELIN-1196] Fix for bug ZEPPELIN-1196

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1197 @SachinJanani Hi, can you add a test case for it? It would help understand and test this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1170: BigQuery Interpreter for Apazhe Zeppelin[ZEPPELIN-1153...

2016-07-18 Thread babupe
Github user babupe commented on the issue: https://github.com/apache/zeppelin/pull/1170 I have skipped the failing test and modified licensing and modified the notice at the project root. Let me know if it looks good. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1144: [ZEPPELIN-1128] add try-catch in close() method.

2016-07-18 Thread voyageth
Github user voyageth commented on the issue: https://github.com/apache/zeppelin/pull/1144 @jongyoul I change this just for more stability. I can't find that bug case just now. But if some Connection or Statement's close() method has some bug(like NPE is some circumstance), zeppelin

[GitHub] zeppelin issue #1172: [Zeppelin-1161] Revert precise execution time

2016-07-18 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/1172 @corneadoug Will this changes also be available in 0.6.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

[GitHub] zeppelin issue #1197: [ZEPPELIN-1196] Fix for bug ZEPPELIN-1196

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1197 @SachinJanani Thanks for the quick fix, but if it fails to launch RemoteInterpreterServer with 10 seconds, this test always fails, isn't it? how to deal with that case? I think we need to make

[GitHub] zeppelin issue #1157: [ZEPPELIN-1146] Zeppelin JDBC interpreter should work ...

2016-07-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1157 @prabhjyotsingh Thanks for checking it. I think we should fix all of flaky tests --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] zeppelin issue #1197: [ZEPPELIN-1196] Fix for bug ZEPPELIN-1196

2016-07-18 Thread SachinJanani
Github user SachinJanani commented on the issue: https://github.com/apache/zeppelin/pull/1197 @jongyoul Yes you are right if RemoteInterpreterServer fails to launch within 10 seconds then this test will fail,but we need to have some threshold for that because if say there is some