[GitHub] zeppelin pull request #1250: [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160...

2016-07-30 Thread prabhjyotsingh
GitHub user prabhjyotsingh reopened a pull request: https://github.com/apache/zeppelin/pull/1250 [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160502.230123-45 ### What is this PR for? This is to fix CI which is mostly failing for resource org.apache.apache.resources:apache-jar

[GitHub] zeppelin pull request #1250: [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160...

2016-07-30 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/1250 --- 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 #1250: [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160502.230...

2016-07-30 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1250 Sure let me try that as well. --- 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 #1250: [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160...

2016-07-30 Thread prabhjyotsingh
GitHub user prabhjyotsingh reopened a pull request: https://github.com/apache/zeppelin/pull/1250 [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160502.230123-45 ### What is this PR for? This is to fix CI which is mostly failing for resource org.apache.apache.resources:apache-jar

[GitHub] zeppelin pull request #1250: [HOTFIX] [WIP] change 1.5-SNAPSHOT to 1.5-20160...

2016-07-30 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/1250 --- 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 #1250: [HOTFIX] change 1.5-SNAPSHOT to 1.5-20160502.230123-45

2016-07-30 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1250 @Leemoonsoo should we have this change, to make CI 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

[GitHub] zeppelin pull request #1250: [HOTFIX] change 1.5-SNAPSHOT to 1.5-20160502.23...

2016-07-30 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1250 [HOTFIX] change 1.5-SNAPSHOT to 1.5-20160502.230123-45 ### What is this PR for? This is to fix CI which is mostly failing for resource org.apache.apache.resources:apache-jar-resource

[GitHub] zeppelin issue #1228: ZEPPELIN-1175. AM log is not available for yarn-client...

2016-07-30 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1228 CI failure looks unrelated, 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

[GitHub] zeppelin issue #1246: [MINOR] Fix missing validation on interpreter page

2016-07-30 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1246 CI failure is unrelated, 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

[GitHub] zeppelin issue #1246: [MINOR] Fix missing validation on interpreter page

2016-07-29 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1246 Good catch, 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 #1237: ZEPPELIN-1242. Should set property SPARK_YARN_MODE and...

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

[GitHub] zeppelin issue #1236: [ZEPPELIN-1237] Auto-suggestion of notebook permission...

2016-07-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1236 @r-kamath have implement your feedback. --- 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 #1233: [ZEPPELIN-1235] ignore rules for files generated by ze...

2016-07-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1233 Merging this if no more discussion. CI fails for unrelated issue. --- 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 #1228: ZEPPELIN-1175. AM log is not available for yarn-client...

2016-07-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1228 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 #1235: ZEPPELIN-1227 change spark-2.0.0-preview to spark-2.0....

2016-07-27 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1235 +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 wishes so

[GitHub] zeppelin pull request #1236: [ZEPPELIN-1237] Auto-suggestion of notebook per...

2016-07-27 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1236 [ZEPPELIN-1237] Auto-suggestion of notebook permissions should list roles as well ### What is this PR for? Auto-suggestion of notebook permissions should list roles as well

[GitHub] zeppelin pull request #1233: [ZEPPELIN-1235] ignore rules for files generate...

2016-07-27 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1233 [ZEPPELIN-1235] ignore rules for files generated by zeppelin-examples ### What is this PR for? add `/helium` in .gitignore ### What type of PR is it? [Improvement

[GitHub] zeppelin issue #1206: ZEPPELIN-1199. Need to login using keytab and principa...

2016-07-27 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1206 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 #1206: ZEPPELIN-1199. Need to login using keytab and principa...

2016-07-26 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1206 @zjffdu can you resolve conflict ? --- 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 #1220: [MINOR] Make scala version definition consistent in Tr...

2016-07-26 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1220 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 #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-25 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1223 @Leemoonsoo I would say, since the use case for this is different than that of notebook, and idea for this is to map access to users with interpreters. While reading the title

[GitHub] zeppelin issue #1216: [ZEPPELIN-919] Apply new mechanism to Markdown

2016-07-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1216 CI green. 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

[GitHub] zeppelin issue #1216: [ZEPPELIN-919] Apply new mechanism to Markdown

2016-07-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1216 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 #1163: [ZEPPELIN-1149] %sh interpreter kerberos support

2016-07-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1163 CI fail is unrelated, which fails for Build #6048.2 with `No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build

[GitHub] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1155 Will merge 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

[GitHub] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1155 Have uploaded the GIF again, I think it would have failed last 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

[GitHub] zeppelin issue #1211: [ZEPPELIN-910] New interpreter registration mechanism ...

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1211 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 #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-21 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1155 @karup1990 have tried to implement the behaviour that you were asking about, can you take a look. Have added a GIF in PR description. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1205: [ZEPPELIN-1212] User impersonation support in JDBC

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

[GitHub] zeppelin pull request #1205: [ZEPPELIN-1212] User impersonation support in J...

2016-07-19 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1205 [ZEPPELIN-1212] User impersonation support in JDBC interpreter for Hive… ### What is this PR for? Add impersonation support to JDBC interpreters, in addition to Kerberos

[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 pull request #1184: [ZEPPELIN-1159] Livy interpreter gets "404 not ...

2016-07-14 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1184 [ZEPPELIN-1159] Livy interpreter gets "404 not found" error ### What is this PR for? RestTemplate throws HttpClientErrorException exception thrown when an HTTP 4xx i

[GitHub] zeppelin issue #1176: [Zeppelin 1165 : improve] Paragraph dynamic relationsh...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1176 Cool feature, even I wanted to have something similar. Will test it, and give more feedback, in the mean while, and referring to the gif posted in screenshot segment, can the rest

[GitHub] zeppelin issue #1181: ZEPPELIN-1168 Add http header X-Requested-By in post r...

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

[GitHub] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

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

[GitHub] zeppelin issue #1140: [ZEPPELIN-1125] Application does not logout user when ...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1140 @corneadoug should we merge this, so that, `grunt serve` starts working, and tune this later ? --- If your project is set up for it, you can reply to this email and have your reply appear

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

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1157 You can either use %hive or %jdbc(hive), for the same. --- 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 issue #1171: [ZEPPELIN-1150] Table contents disappear 2nd run onwar...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1171 👍 --- 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 #1182: ZEPPELIN-1170 Handsontable fails to display data on se...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1182 @Leemoonsoo Yes, I agree, we should merge #1171. --- 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 issue #1171: [ZEPPELIN-1150] Table contents disappear 2nd run onwar...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1171 I missed this (which solve the same problem as https://github.com/apache/zeppelin/pull/1182), let me check this. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1182: [HOTFIX] ZEPPELIN-1170 Handsontable fails to display d...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1182 LGTM! This has been happening in my machine as well for quite some time. Thanks @r-kamath for the fix. @corneadoug, @Leemoonsoo Please review will merge this a hot fix

[GitHub] zeppelin issue #1181: ZEPPELIN-1168 Add http header X-Requested-By in post r...

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1181 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 #1175: use static field logger instead of function

2016-07-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1175 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 #1157: [ZEPPELIN-1146] Zeppelin JDBC interpreter should work ...

2016-07-12 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1157 @rja1 Sure, attaching both screenshot and json. I have tested this for hive and phoenix. https://cloud.githubusercontent.com/assets/674497/16776893/11020f28-4885-11e6-878b

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

2016-07-12 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1157 Yes fair point, let me do a quick test and do it right away. --- 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 #993: [ZEPPELIN-987] Enable user to secure interpreter settin...

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

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

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

[GitHub] zeppelin issue #1159: [Test branch] Logout

2016-07-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1159 I don't have a strong view on any, fine either way. Just that I thought authcBasic should work. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1159: [Test branch] Logout

2016-07-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1159 Agreed @corneadoug. Thought we don't have `authcBasic` in our doc, but whoever knows or had used shiro in past can choose to use it, and IMO whatever features shiro provides we should

[GitHub] zeppelin issue #1153: Block all routes except Home when user isn't logged

2016-07-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1153 Tried new changes works well. --- 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 #1153: Block all routes except Home when user isn't logged

2016-07-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1153 LGTM! minor suggestion. --- 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 #1147: ZEPPELIN-1130 Make Livy create session retries configu...

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

[GitHub] zeppelin issue #1158: [ZEPPELIN-1131] Does not initialize login page values.

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

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

2016-07-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1157 @jongyoul i couldn't figure out a easier way, than having a actual kerberos environment, and then testing it there. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1155: [ZEPPELIN-1143] Interpreter dependencies are not downl...

2016-07-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1155 Yes, so far that is the only way we can download the dependency again. --- 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 #993: [ZEPPELIN-987] Enable user to secure interpreter settin...

2016-07-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/993 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 #1140: [ZEPPELIN-1125] Application does not logout user when ...

2016-07-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1140 > Basic Auth is getting logout after only one call. Kind of strange it, doesn't work for me. Have attached gif's where in console, first I'm trying to call it once, then tw

[GitHub] zeppelin issue #1154: [ZEPPELIN-1136] NPE in Zeppelin Logs

2016-07-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1154 Yes, agreed, added a unit test case for getUserList `userlist/{searchText}` API. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #993: [ZEPPELIN-987][WIP] Enable user to secure interpreter s...

2016-07-09 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/993 Sure this will work. Thanks for the 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 not have

[GitHub] zeppelin issue #1156: ZEPPELIN-1145 Zeppelin UI fails to load page with HTTP...

2016-07-09 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1156 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 pull request #1157: [ZEPPELIN-1146] Zeppelin JDBC interpreter shoul...

2016-07-09 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1157 [ZEPPELIN-1146] Zeppelin JDBC interpreter should work in a Kerberos environment ### What is this PR for? Zeppelin JDBC interpreter should work in a Kerberos environment

[GitHub] zeppelin pull request #1155: [ZEPPELIN-1143] Interpreter dependencies are no...

2016-07-08 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1155 [ZEPPELIN-1143] Interpreter dependencies are not downloaded on zeppelin… ### What is this PR for? While saving interpreter setting, if zeppelin server crashed/killed/restarted

[GitHub] zeppelin issue #1154: [ZEPPELIN-1136] NPE in Zeppelin Logs

2016-07-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1154 CI fails for `Job #5690.1` ``` 13:05:55,335 INFO org.apache.zeppelin.notebook.Paragraph:254 - run paragraph 20160708-130555_1605317382 using r

[GitHub] zeppelin pull request #1154: [ZEPPELIN-1136] NPE in Zeppelin Logs

2016-07-08 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1154 [ZEPPELIN-1136] NPE in Zeppelin Logs ### What is this PR for? Access Zeppelin without configuring any security, Zeppelin shows anoymous user and notice the NPE in Zeppelin logs

[GitHub] zeppelin issue #1140: [ZEPPELIN-1125] Application does not logout user when ...

2016-07-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1140 Yes, basic-auth was not getting logout on first call, hence I had to call it twice. Advantage of keeping `UNAUTHORIZED` is that we don't have to bother about $http.post's success

[GitHub] zeppelin issue #1140: [ZEPPELIN-1125] Application does not logout user when ...

2016-07-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1140 @corneadoug I tried your approach, it didn't worked for me. --- 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 #1139: [ZEPPELIN-235] jscs in zeppelin-web

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

[GitHub] zeppelin issue #1142: [ZEPPELIN-1127] Show error message in case of exceptio...

2016-07-07 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1142 Sure @Leemoonsoo I'll do that. --- 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 #1142: [ZEPPELIN-1127] Show error message in case of exceptio...

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

[GitHub] zeppelin issue #1143: [ZEPPELIN-1085] Make dropdown menu scrollbar always vi...

2016-07-07 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1143 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 #1147: ZEPPELIN-1130 Make Livy create session retires configu...

2016-07-07 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1147 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 pull request #1142: [ZEPPELIN-1127] Show error message in case of e...

2016-07-07 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1142 [ZEPPELIN-1127] Show error message in case of exception with JDBC ### What is this PR for? When there is exception while executeSql in JDBC interpreter, only stack trace is sent back

[GitHub] zeppelin issue #1139: [ZEPPELIN-235] jscs in zeppelin-web

2016-07-07 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1139 @corneadoug, have addressed your feedback. --- 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 #1139: [ZEPPELIN-235] jscs in zeppelin-web

2016-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1139 @corneadoug, have made the relevant changes. It was coming because `function($scope.` was is next line, and because of which build was failing with `validateIndentation

[GitHub] zeppelin issue #1140: Application does not logout user when authcBasic and `...

2016-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1140 @corneadoug, have made a fix for https://github.com/apache/zeppelin/pull/1071#issuecomment-230736598. Can you please review. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1139: [ZEPPELIN-235] jscs in zeppelin-web

2016-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1139 CI fails for `Failed to run task: 'npm install --color=false' failed.`, re-triggered. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #1139: [ZEPPELIN-235] jscs in zeppelin-web

2016-07-06 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1139 [ZEPPELIN-235] jscs in zeppelin-web ### What is this PR for? Reopening the PR https://github.com/apache/zeppelin/pull/229 for jscs in zeppelin-web, using following in .jscsrc

[GitHub] zeppelin issue #1071: [ZEPPELIN-1052] Application does not logout user when ...

2016-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1071 @corneadoug, will surly look into it, in the mean while have created this a new jira for tracking the same https://issues.apache.org/jira/browse/ZEPPELIN-1125 --- If your project is set

[GitHub] zeppelin issue #1132: [ZEPPELIN-1088] Add Htmlhint

2016-07-06 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1132 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 #1116: minor doc fix

2016-07-05 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1116 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 #1088: [ZEPPELIN-1062] Get original InterpreterSetting when t...

2016-07-05 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1088 Thank you for taking care of 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

[GitHub] zeppelin issue #1128: [HOTFIX] NPE when there's no properties field in inter...

2016-07-04 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1128 Good catch, agreed with @bzz, let's merge asap. --- 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 issue #1127: [HOTFIX] Remove duplicate closing brace in publish_rel...

2016-07-04 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1127 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 pull request #480: Option to restart interpreter from paragraph.

2016-07-03 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/480 --- 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 #1111: [ZEPPELIN-1094] Run all paragraphs keeps appending emp...

2016-07-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/ Sure, thank you @jongyoul I'll open up a separate PR for fixing that. --- 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 #1111: [ZEPPELIN-1094] Run all paragraphs keeps appending emp...

2016-07-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/ @jongyoul Sure, now I understand what you mean, but before that what do we call an empty paragraph; should it be `Strings.isNullOrEmpty(text)` or `"%getInterpreterName()".equals

[GitHub] zeppelin issue #1111: [ZEPPELIN-1094] Run all paragraphs keeps appending emp...

2016-07-01 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/ Yes, its working for me, if you refer to the gif attached above; if paragraph content is only interpreter name (like in this example "%md ") it wouldn't add a paragraph below,

[GitHub] zeppelin issue #1109: [Zeppelin-1090][Hot Fix] LivySparkSQLInterpreter doesn...

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

[GitHub] zeppelin issue #1111: [ZEPPELIN-1094] Run all paragraphs keeps appending emp...

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

[GitHub] zeppelin pull request #1111: [ZEPPELIN-1094] Run all paragraphs keeps append...

2016-06-30 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/ [ZEPPELIN-1094] Run all paragraphs keeps appending empty paragraphs ### What is this PR for? Every time user clicks on "Run all paragraphs" button system keeps appendin

[GitHub] zeppelin issue #1109: [Zeppelin-1090] LivySparkSQLInterpreter doesn't work i...

2016-06-30 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1109 @bzz CI still fails https://github.com/apache/zeppelin/pull/1094 ``` --- T E S T S

[GitHub] zeppelin issue #1101: [ZEPPELIN-1072] Saving Interpreter Setting dosen't not...

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

[GitHub] zeppelin pull request #1109: [Zeppelin-1090] LivySparkSQLInterpreter doesn't...

2016-06-30 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1109 [Zeppelin-1090] LivySparkSQLInterpreter doesn't work in FIFO. ### What is this PR for? LivySparkSQLInterpreter should work in FIFO just like SparkSqlInterpreter ### What

[GitHub] zeppelin issue #1106: [ZEPPELIN-1082] Restore Interpreter page layout change

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

[GitHub] zeppelin issue #1106: [ZEPPELIN-1082] Restore Interpreter page layout change

2016-06-29 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1106 Tested, 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 issue #1101: [ZEPPELIN-1072] Saving Interpreter Setting dosen't not...

2016-06-29 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1101 CI green, ready 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

[GitHub] zeppelin issue #1099: ZEPPELIN-1071 ] Ace-editor hidden auto-complete additi...

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1099 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 #1103: [ZEPPELIN-1076] Set hbase.client.retries.number for JD...

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1103 Yes, `phoenix.hbase.client.retries.number`, and have tested with and without this string. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1094: ZEPPELIN-1063: fix flaky python interpreter test

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1094 Yes, I've seen this, this happens when spark takes longer that 60sec to start up. Will open up a new jira and fix this as well. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1088: [ZEPPELIN-1062] Get original InterpreterSetting when t...

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1088 Also what happens when user clicks on close ("x") button, instead of "OK" or "Close" ? --- If your project is set up for it, you can reply to this em

<    3   4   5   6   7   8   9   >