[GitHub] zeppelin issue #2446: [ZEPPELIN-2699] Helium REST API bug-fixes. Also update...

2017-07-12 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2446 @1ambda 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 if

[GitHub] zeppelin pull request #2490: Zeppelin 2718 Find and replace code in note

2017-07-12 Thread tinkoff-dwh
GitHub user tinkoff-dwh opened a pull request: https://github.com/apache/zeppelin/pull/2490 Zeppelin 2718 Find and replace code in note ### What is this PR for? Add feature for find and replace code in all paragraphs of note ### What type of PR is it? [Feature]

[GitHub] zeppelin issue #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2486 LGTM. I think we can have separate discussion for JDK version support, thanks for the explain. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin pull request #2467: [ZEPPELIN-2730] compare with maxLength after ge...

2017-07-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2467 --- 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 #2470: [ZEPPELIN-2648] Fix spark module build problem ...

2017-07-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2470 --- 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 #2374: [ZEPPELIN-2593] Add storage settings to persist on run...

2017-07-12 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2374 @1ambda 1. separate tab with `Global Settings` contains the settings which are applied to all storages, so that we don't duplicate in each storage setting. 2. note persisted when us

[GitHub] zeppelin issue #2489: [Refactoring] some opportunities to use diamond operat...

2017-07-12 Thread desmorto
Github user desmorto commented on the issue: https://github.com/apache/zeppelin/pull/2489 already signed with travis --- 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 w

[GitHub] zeppelin pull request #2489: [Refactoring] some opportunities to use diamond...

2017-07-12 Thread desmorto
GitHub user desmorto opened a pull request: https://github.com/apache/zeppelin/pull/2489 [Refactoring] some opportunities to use diamond operator A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide - https://ze

[GitHub] zeppelin issue #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2486 One workaround for spark 2.2 is that user could specify JAVA_HOME in its interpreter setting. I have verified it. ![image](https://user-images.githubusercontent.com/164491/28148316-76461c

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2238 Looks like code need to handle null return of `getProperty(CASSANDRA_WITH_SSL)` at `org.apache.zeppelin.cassandra.CassandraInterpreter.open(CassandraInterpreter.java:200)` --- If your project

[GitHub] zeppelin issue #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2486 For Spark 2.2.0, yes... Zeppelin could run a different jdk version? The interpreter jvm hosting spark would need to be 8. --- If your project is set up for it, you can reply to this emai

[GitHub] zeppelin issue #2238: 0.7.1 with ssl

2017-07-12 Thread alourie
Github user alourie commented on the issue: https://github.com/apache/zeppelin/pull/2238 @Leemoonsoo I've tried to rerun them few times. Only one now fails - build #4; fails with NPE in C* setup/tearDown: ```Tests run: 2, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 40.22

[GitHub] zeppelin issue #2488: Add z.show() feature to docs

2017-07-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2488 Thanks @TwUxTLi51Nus for the contribution! LGTM. merge to master if no further comment. CI failure is okay while this PR updates doc only CI does not build and verify doc. If y

[GitHub] zeppelin pull request #2488: Add z.show() feature to docs

2017-07-12 Thread TwUxTLi51Nus
GitHub user TwUxTLi51Nus opened a pull request: https://github.com/apache/zeppelin/pull/2488 Add z.show() feature to docs I was unaware of this really nice feature, which led me to writing my own "zeppelin_show" function. ### What is this PR for? (tiny) improvement of t

[GitHub] zeppelin issue #2483: [ZEPPELIN-2766] Make online resources url configurable...

2017-07-12 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2483 yes, :-) done! @felixcheung --- 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 pull request #2483: [ZEPPELIN-2766] Make online resources url confi...

2017-07-12 Thread andreaTP
Github user andreaTP closed the pull request at: https://github.com/apache/zeppelin/pull/2483 --- 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 #2483: [ZEPPELIN-2766] Make online resources url confi...

2017-07-12 Thread andreaTP
GitHub user andreaTP reopened a pull request: https://github.com/apache/zeppelin/pull/2483 [ZEPPELIN-2766] Make online resources url configurable at compile time ### What is this PR for? At compile time Zeppelin is downloading several external resources. I want to be able to

[GitHub] zeppelin issue #2374: [ZEPPELIN-2593] Add storage settings to persist on run...

2017-07-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2374 @khalidhuseynov Thanks for the update. 1. Could you describe the reason why we need to have separated setting tabs? I mean, what note persistence setting should be under `Global Settings`

[GitHub] zeppelin issue #2477: [ZEPPELIN-2342] Improving Test Environment for zeppeli...

2017-07-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2477 Thanks for the review. @soralee. It enables us to write integration tests easily. I will add some integration tests into zeppelin-web later based on this. Merge if no more discussion.

[GitHub] zeppelin issue #2448: [ZEPPELIN-2702] save notes in reader-friendly format w...

2017-07-12 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2448 well abstracted and tested PR. LGTM except for - Currently, it seems that converting the note names only when creating / renaming. Is it right? I so, can we rename them initially as you sug

[GitHub] zeppelin issue #2448: [ZEPPELIN-2702] save notes in reader-friendly format w...

2017-07-12 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2448 @zjffdu @1ambda addressed the feedback to convert in save, w/o additional interface. probably will need to separate base class with each storage after this pr to handle more logic in there.

[GitHub] zeppelin issue #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2486 I think Jdk 7 is supported by Spark, before the 2.2.0 release. We can choose not to test with it in Travis. Not sure if we need to make sure we test with jdk 7 before making a release.

[GitHub] zeppelin issue #2480: [ZEPPELIN-1249] build all submodules w/ the same scala...

2017-07-12 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2480 sounds good, I’ll do that then _ From: Felix Cheung mailto:notificati...@github.com>> Sent: Wednesday, July 12, 2017 9:34 AM Subject: Re: [apache/z

[GitHub] zeppelin issue #2480: [ZEPPELIN-1249] build all submodules w/ the same scala...

2017-07-12 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2480 I realize 2.10 is EOL but as mentioned, from a while back many platforms are still running only on 2.10. Even Spark still supports 2.10 as of the latest release 2.2.0 from yesterday. The reaso

[jira] [Created] (ZEPPELIN-2773) Console type error on logout: cannot set property 'principal'

2017-07-12 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-2773: - Summary: Console type error on logout: cannot set property 'principal' Key: ZEPPELIN-2773 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2773 Project:

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-12 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2439 thanks for the improvement, this scenario needs to be fixed indeed. after trying the pr, i noticed couple of things * when following scenario and going into note as user2, my browser go

[GitHub] zeppelin issue #2477: [ZEPPELIN-2342] Improving Test Environment for zeppeli...

2017-07-12 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2477 Oh, it looks great improvement! Tested and LGTM. It works well and it's cool! Thanks @1ambda --- If your project is set up for it, you can reply to this email and have your reply app

[jira] [Created] (ZEPPELIN-2772) Snapshot Dynamic Dependency does not update

2017-07-12 Thread Edgardo Vega (JIRA)
Edgardo Vega created ZEPPELIN-2772: -- Summary: Snapshot Dynamic Dependency does not update Key: ZEPPELIN-2772 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2772 Project: Zeppelin Issue

[GitHub] zeppelin issue #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2486 @Leemoonsoo @jongyoul @felixcheung Please help review it. --- 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

[jira] [Created] (ZEPPELIN-2771) Can I download a csv file directly ever day to my specified location. what command can be used to domnload.

2017-07-12 Thread tp (JIRA)
tp created ZEPPELIN-2771: Summary: Can I download a csv file directly ever day to my specified location. what command can be used to domnload. Key: ZEPPELIN-2771 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2771

[jira] [Created] (ZEPPELIN-2770) Zeppelin is not working in IE-11

2017-07-12 Thread thevenin yamma (JIRA)
thevenin yamma created ZEPPELIN-2770: Summary: Zeppelin is not working in IE-11 Key: ZEPPELIN-2770 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2770 Project: Zeppelin Issue Type: B

[GitHub] zeppelin pull request #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/2486 --- 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 #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-12 Thread zjffdu
GitHub user zjffdu reopened a pull request: https://github.com/apache/zeppelin/pull/2486 [ZEPPELIN-2768]. Bump up Spark version to 2.2.0 ### What is this PR for? Spark 2.2.0 is just released, this PR is to support spark 2.2.0. ### What type of PR is it? [Improvement

[GitHub] zeppelin issue #2485: [ZEPPELIN-2758]. NumberFormatException on importing no...

2017-07-12 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2485 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 fea

[GitHub] zeppelin pull request #2487: [ZEPPELIN-2769] Prevent SQL injection for GetUs...

2017-07-12 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/zeppelin/pull/2487 [ZEPPELIN-2769] Prevent SQL injection for GetUserList.getUserList. ### What is this PR for? Prevent SQL injection for ```GetUserList.getUserList```. ### What type of PR is it?

[jira] [Created] (ZEPPELIN-2769) Prevent SQL injection for GetUserList

2017-07-12 Thread Yanbo Liang (JIRA)
Yanbo Liang created ZEPPELIN-2769: - Summary: Prevent SQL injection for GetUserList Key: ZEPPELIN-2769 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2769 Project: Zeppelin Issue Type: Im