[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1247 This PR looks like copy and paste from NotebookRestAapi. If you think websocket is a better way to do it, why don't you removing rest api? I don't think it's good to duplicate only. What's the be

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1247 And please add test case if you are going further. --- 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 th

[GitHub] zeppelin pull request #1221: [MINOR] Change url in pom.xml files

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1221 --- 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 pull request #1254: [ZEPPELIN-1257] storage - fix get note revision...

2016-07-31 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/1254 [ZEPPELIN-1257] storage - fix get note revision api ### What is this PR for? Getting revision of note requires only unique revision id and the whole revision object isn't required. Thi

[GitHub] zeppelin issue #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-07-31 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1254 /cc @corneadoug @bzz 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 en

[jira] [Created] (ZEPPELIN-1258) Adding extra libraries to livy interpreter

2016-07-31 Thread Mahmoud Elgamal (JIRA)
Mahmoud Elgamal created ZEPPELIN-1258: - Summary: Adding extra libraries to livy interpreter Key: ZEPPELIN-1258 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1258 Project: Zeppelin I

[GitHub] zeppelin pull request #1255: Adding extra libraries to livy

2016-07-31 Thread mfelgamal
GitHub user mfelgamal opened a pull request: https://github.com/apache/zeppelin/pull/1255 Adding extra libraries to livy ### What is this PR for? Adding extra libraries to livy interpreter which isn't exist by default. ### What type of PR is it? [ Improvement ]

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1247 @jongyoul Yes. this is copied from rest api. We should probably keep both options and use websocket on zeppelin UI. IMHO we all communications on the ui should happen via websockets and avoid mul

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1247 Thanks for the explaining it, but I think you'd better handle those changes into single PR, which is more meaningful. How do you think of it? --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1247 @jongyoul I'm planning to make those changes in this 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 your project does not ha

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1247 That sounds great --- 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 #1255: Adding extra libraries to livy

2016-07-31 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1255 I would not recommend user to do that to add extra library. Because most of time zeppelin user don't know the cluster details like which extra library are located in the cluster machines. I would s

[GitHub] zeppelin issue #1253: [HOTFIX] Add bigquery interpreter to configuration

2016-07-31 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1253 Thank you so much for taking care! --- 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

[GitHub] zeppelin issue #1247: ZEPPELIN-1254 Make get and save Interpreter bindings c...

2016-07-31 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1247 @r-kamath if you are planning to add more changes in this PR, do you think it will make sense to: - update TODO list in PR description with action items that you plan - add `WIP` prefix (wor

[jira] [Created] (ZEPPELIN-1259) ace editor configuration to service metadata restore object

2016-07-31 Thread CloverHearts (JIRA)
CloverHearts created ZEPPELIN-1259: -- Summary: ace editor configuration to service metadata restore object Key: ZEPPELIN-1259 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1259 Project: Zeppelin

[GitHub] zeppelin issue #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-07-31 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1254 @khalidhuseynov The CI is fully red --- 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 en

[GitHub] zeppelin issue #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-07-31 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1254 @khalidhuseynov could you please elaborate on CI failure reason? Also, could you please explain why do you think this is valuable change and what benefits does it bring? --- If your project

[GitHub] zeppelin pull request #1240: [ZEPPELIN-1246] In JDBCInterpreter.getScheduler...

2016-07-31 Thread prasadwagle
Github user prasadwagle closed the pull request at: https://github.com/apache/zeppelin/pull/1240 --- 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 #1240: [ZEPPELIN-1246] In JDBCInterpreter.getScheduler...

2016-07-31 Thread prasadwagle
GitHub user prasadwagle reopened a pull request: https://github.com/apache/zeppelin/pull/1240 [ZEPPELIN-1246] In JDBCInterpreter.getScheduler, use getMaxConcurrentConnection instead of hardcoding maxConcurrency to 10 ### What is this PR for? In JDBCInterpreter.getScheduler, use

[jira] [Created] (ZEPPELIN-1260) R interpreter doesn't work with Spark 2.0

2016-07-31 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1260: -- Summary: R interpreter doesn't work with Spark 2.0 Key: ZEPPELIN-1260 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1260 Project: Zeppelin Issue Type: Bug

[GitHub] zeppelin issue #1244: [ZEPPELIN-1251] Update interpreter artifact option lis...

2016-07-31 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1244 Looks good to 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 does not have this feature enabled and wishes so

[GitHub] zeppelin issue #1244: [ZEPPELIN-1251] Update interpreter artifact option lis...

2016-07-31 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1244 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 #1240: [ZEPPELIN-1246] In JDBCInterpreter.getScheduler, use g...

2016-07-31 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1240 Could you please rebase it? it would be fixed yesterday. --- 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 h

[GitHub] zeppelin issue #1249: [ZEPPELIN-1255] Add cast to string in z.show() for Pan...

2016-07-31 Thread bustios
Github user bustios commented on the issue: https://github.com/apache/zeppelin/pull/1249 Tranks for the revision @felixcheung, @bzz. I have already added the test case. I think it would be nice to show a column with the row number or index if the DataFrame had one. --- If

[jira] [Created] (ZEPPELIN-1261) z.show() height parameter doesn't take effect alone

2016-07-31 Thread Paul Bustios (JIRA)
Paul Bustios created ZEPPELIN-1261: -- Summary: z.show() height parameter doesn't take effect alone Key: ZEPPELIN-1261 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1261 Project: Zeppelin

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-31 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1241 Thanks @AhyoungRyu , the new UI lgtm. BTW, this link helps me in case someone else need this, http://pragmaticstudio.com/blog/2010/9/23/install-rails-ruby-mac --- If your project is set up fo

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-31 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @zjffdu Yeah I also struggled with ruby and my OSX version issues. Anyway thanks for providing the link :) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in ...

2016-07-31 Thread AhyoungRyu
Github user AhyoungRyu closed the pull request at: https://github.com/apache/zeppelin/pull/1241 --- 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 #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in ...

2016-07-31 Thread AhyoungRyu
GitHub user AhyoungRyu reopened a pull request: https://github.com/apache/zeppelin/pull/1241 [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppelin conf table ### What is this PR for? This PR will fix [ZEPPELIN-1217](https://issues.apache.org/jira/browse/ZEPPELIN-1217).

[GitHub] zeppelin issue #1244: [ZEPPELIN-1251] Update interpreter artifact option lis...

2016-07-31 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1244 Looks great to me. :shipit: --- 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

[GitHub] zeppelin issue #1231: [Zeppelin - 683][WIP] BittorrentNotebookRepo

2016-07-31 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/1231 @bzz ping. --- 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

[GitHub] zeppelin issue #1249: [ZEPPELIN-1255] Add cast to string in z.show() for Pan...

2016-07-31 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1249 Looks great to me. >I think it would be nice to show a column with the row number or index if the DataFrame had one. @bustios This makes sense, please feel free to open a separate JIR

[jira] [Created] (ZEPPELIN-1262) code editor configuration web page

2016-07-31 Thread CloverHearts (JIRA)
CloverHearts created ZEPPELIN-1262: -- Summary: code editor configuration web page Key: ZEPPELIN-1262 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1262 Project: Zeppelin Issue Type: Sub

[GitHub] zeppelin pull request #1256: [ ZEPPELIN-1259 ] create ace.editor configurati...

2016-07-31 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/zeppelin/pull/1256 [ ZEPPELIN-1259 ] create ace.editor configuration manager service object ### What is this PR for? https://github.com/apache/zeppelin/pull/1218 Establish a foundation to personalize th

[GitHub] zeppelin issue #1254: [ZEPPELIN-1257] storage - fix get note revision api

2016-07-31 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1254 @bzz CI failure was due to test, fixed now. Further, regarding the flexibility and narrowing scope. Revision class is implemented [here](https://github.com/apache/zeppelin/blob/mas

[GitHub] zeppelin pull request #1246: [MINOR] Fix missing validation on interpreter p...

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1246 --- 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 pull request #1228: ZEPPELIN-1175. AM log is not available for yarn...

2016-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1228 --- 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 #1236: [ZEPPELIN-1237] Auto-suggestion of notebook permission...

2016-07-31 Thread rconline
Github user rconline commented on the issue: https://github.com/apache/zeppelin/pull/1236 @felixcheung i think this will continue to remain a problem, till the time we allow all user roles to setup/modify these permissions. Unless we can recognize and separate this as an admin (superu