[GitHub] zeppelin issue #2375: [ZEPPELIN-2527] Use thin (normal) cursor.

2017-06-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2375 Thanks @1ambda 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 #2381: [WIP] [gh-pages] [ZEPPELIN-2597] Updating offcial main...

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 Hi, finally I finished the above To-do list. @1ambda @Leemoonsoo Could you confirm this, again? For now, the quick links is connected with 0.8.0-SNAPSHOT documentation. @Leemoonsoo cou

[GitHub] zeppelin issue #2309: [ZEPPELIN-2482] added font size for paragraph

2017-06-04 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2309 Ready to 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 enabled and wishes so

[GitHub] zeppelin issue #2268: [ZEPPELIN-2403] interpreter property widgets

2017-06-04 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2268 Ready to 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 enabled and wishes so

[GitHub] zeppelin issue #2343: [ZEPPELIN-2538] JDBC completer improvements for work w...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2343 LGTM. Will merge it 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 enabled and wis

[GitHub] zeppelin pull request #2343: [ZEPPELIN-2538] JDBC completer improvements for...

2017-06-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2343 --- 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

Re: [DISCUSS] Reducing default interpreters while building and releasing Zeppelin

2017-06-04 Thread Jongyoul Lee
It means we release with some interpreters and deploy all interpreters into maven separately. We already had a install-interpreter script inside it. If someone wants to install specific interpreter not included in default release package, they can use that script to install specific one. On Sun, J

[GitHub] zeppelin issue #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Zeppelin...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 Ready to review. Please make sure it works with latest jupyter format with major version of 4 --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] zeppelin issue #2388: [ZEPPELIN-2492] Use single Dockerfile for each tag (br...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2388 I'll merge 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 have this feature enabled and wishes so, or

[GitHub] zeppelin issue #2369: [ZEPPELIN-2591] Show user info in spark job descriptio...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2369 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 fe

[GitHub] zeppelin issue #2309: [ZEPPELIN-2482] added font size for paragraph

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2309 LGTM. @1ambda Can you verify 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 have this feature enabled

[GitHub] zeppelin issue #2282: [ZEPPELIN-2447] Fix python interpreter as to use max r...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2282 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 the f

[GitHub] zeppelin issue #2388: [ZEPPELIN-2492] Use single Dockerfile for each tag (br...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2388 Can you close it by yourself? --- 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

Re: [DISCUSS] Reducing default interpreters while building and releasing Zeppelin

2017-06-04 Thread Felix Cheung
Sure - I think it will be important to discuss what criteria to use to decide what is included vs what will be released separately. _ From: Jongyoul Lee mailto:jongy...@gmail.com>> Sent: Sunday, June 4, 2017 9:47 AM Subject: Re: [DISCUSS] Reducing default interpreters

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread DuyHai Doan
+1 too On Sun, Jun 4, 2017 at 2:16 AM, Park Hoon <1am...@gmail.com> wrote: > +1 for release 0.7.2 since 0.7.1 has few critical bugs. > > > Thanks. > > On Sun, Jun 4, 2017 at 9:13 AM, Felix Cheung > wrote: > > > +1 > > > > From: moon soo Lee > > Sent: Saturday,

[GitHub] zeppelin issue #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Zeppelin...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 Yes, sure. I'll made another PR for adding docs --- 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

[jira] [Created] (ZEPPELIN-2616) Add docs to describe how to use this feature

2017-06-04 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-2616: -- Summary: Add docs to describe how to use this feature Key: ZEPPELIN-2616 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2616 Project: Zeppelin Issue

[GitHub] zeppelin issue #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Zeppelin...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 Made [ZEPPELIN-2616](https://issues.apache.org/jira/browse/ZEPPELIN-2616) --- 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 p

Re: [DISCUSS] Reducing default interpreters while building and releasing Zeppelin

2017-06-04 Thread moon soo Lee
Following is last discussion related to release package size. https://lists.apache.org/thread.html/69f606409790d7ba11422e8c6df941a75c5dfae0aca63eccf2f840bf@%3Cusers.zeppelin.apache.org%3E at this time, we have discussed about having bin-all (every interpreters), bin-min (selected interpreters), b

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread 18637156...@163.com
+1 for release 0.7.2 18637156...@163.com From: moon soo Lee Date: 2017-06-04 01:47 To: dev Subject: [DISCUSS] Release 0.7.2 Hi, Since 0.7.1 release, we have resolved 38 issues and that includes most of critical issues in 0.7.2 umbrella issue [1]. Since 0.7.1 has major problems like ZEPPELI

[GitHub] zeppelin issue #2356: [ZEPPELIN-2460] Highlight active line in editor

2017-06-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2356 This PR makes unnecessary margin above the Editor inside of Paragraph. before ![image](https://cloud.githubusercontent.com/assets/1540981/26764299/4640bb22-4919-11e7-8997-9851783825

[GitHub] zeppelin issue #2381: [WIP] [gh-pages] [ZEPPELIN-2597] Updating offcial main...

2017-06-04 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2381 Thank you for great work! Especially big +1 for adding this section https://cloud.githubusercontent.com/assets/10060731/26764287/e5b90aa2-4918-11e7-908c-1fdfd4870037.png";> I've

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread Ahyoung Ryu
+1 for the release On Sat, Jun 3, 2017 at 11:32 PM, 18637156...@163.com <18637156...@163.com> wrote: > +1 for release 0.7.2 > > > > 18637156...@163.com > > From: moon soo Lee > Date: 2017-06-04 01:47 > To: dev > Subject: [DISCUSS] Release 0.7.2 > Hi, > > Since 0.7.1 release, we have resolved 38 i

[GitHub] zeppelin issue #1970: ZEPPELIN-2045. Pass interpreter properties with "spark...

2017-06-04 Thread orenpai
Github user orenpai commented on the issue: https://github.com/apache/zeppelin/pull/1970 @zjffdu @Leemoonsoo I believe this is a mistake, and poses a real problem for me and other Zeppelin users. While it's true that the properties file is being skimmed to remove any non- spark.*

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread Jeff Zhang
+1 Ahyoung Ryu 于2017年6月5日周一 上午4:00写道: > +1 for the release > > On Sat, Jun 3, 2017 at 11:32 PM, 18637156...@163.com <18637156...@163.com> > wrote: > > > +1 for release 0.7.2 > > > > > > > > 18637156...@163.com > > > > From: moon soo Lee > > Date: 2017-06-04 01:47 > > To: dev > > Subject: [DISCUSS

[GitHub] zeppelin issue #2388: [ZEPPELIN-2492] Use single Dockerfile for each tag (br...

2017-06-04 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2388 Thanks @jongyoul. --- 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, o

[GitHub] zeppelin pull request #2388: [ZEPPELIN-2492] Use single Dockerfile for each ...

2017-06-04 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2388 --- 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

Re: [DISCUSS] Reducing default interpreters while building and releasing Zeppelin

2017-06-04 Thread Jeff Zhang
If possible, we need to integrate the install script in zeppelin UI. As I would expect many users would ask why some interpreter is missing and how to install them. moon soo Lee 于2017年6月5日周一 上午2:06写道: > Following is last discussion related to release package size. > > > https://lists.apache.org

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread Hyung Sung Shim
+1 2017-06-05 8:42 GMT+09:00 Jeff Zhang : > +1 > > Ahyoung Ryu 于2017年6月5日周一 上午4:00写道: > > > +1 for the release > > > > On Sat, Jun 3, 2017 at 11:32 PM, 18637156...@163.com < > 18637156...@163.com> > > wrote: > > > > > +1 for release 0.7.2 > > > > > > > > > > > > 18637156...@163.com > > > > > > Fr

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread Khalid Huseynov
+1 On Mon, Jun 5, 2017 at 9:49 AM, Hyung Sung Shim wrote: > +1 > > 2017-06-05 8:42 GMT+09:00 Jeff Zhang : > > > +1 > > > > Ahyoung Ryu 于2017年6月5日周一 上午4:00写道: > > > > > +1 for the release > > > > > > On Sat, Jun 3, 2017 at 11:32 PM, 18637156...@163.com < > > 18637156...@163.com> > > > wrote: > >

[GitHub] zeppelin issue #2381: [gh-pages] [ZEPPELIN-2597] Updating offcial main websi...

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 @AhyoungRyu Thanks for your opinions! But, Sorry for sending disagreement message. First, `Navbar` part is that the basis is #2371 PR. We should match this new feature and, I think fo

[GitHub] zeppelin pull request #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Z...

2017-06-04 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/2393 [ZEPPELIN-2590] Convert jupyter's notebook to Zeppelin's note ### What is this PR for? Converting jupyter's notebook to Zeppelin's note ### What type of PR is it? [Feature]

[GitHub] zeppelin pull request #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Z...

2017-06-04 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/2393 --- 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 #1970: ZEPPELIN-2045. Pass interpreter properties with "spark...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1970 @zjffdu Can you handle this 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 project does not have this feature enabl

Release 0.7.2

2017-06-04 Thread Jongyoul Lee
Hi all, We have umbrella issue for 0.7.2. See https://issues.apache.org/jira/browse/ZEPPELIN-2276 I think it's almost done and there're some trivial issues which don't look like blockers. How about starting release process for 0.7.2? And I suggest moving remaining issues into 0.8.0. Regards, JL

[GitHub] zeppelin issue #1970: ZEPPELIN-2045. Pass interpreter properties with "spark...

2017-06-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1970 @orenpai IIUC, spark won't propagate non spark.* properties to executor. How does ElasticsEarch use these configuration. Does it only use it in driver side ? --- If your project is set up for it,

[GitHub] zeppelin issue #2371: [ZEPPELIN-2596] Improving documentation page

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2371 Sorry for my late response I quickly checked some navbar bug about https://github.com/apache/zeppelin/pull/2381#issuecomment-305451690 and https://github.com/apache/zeppelin/pull/2381#issueco

Re: [DISCUSS] Reducing default interpreters while building and releasing Zeppelin

2017-06-04 Thread Jongyoul Lee
Felix, Yes, I said a bit confused. I want to release Zeppelin with some - not-all - interpreters, but deploy all interpreters into maven to install them if users want to use them. Moon, I think it's the best to fit the list as same as homepage by default as it makes users confused less. But if we

Re: Release 0.7.2

2017-06-04 Thread Jongyoul Lee
I saw another mailing thread now. Ignore it. On Mon, Jun 5, 2017 at 1:45 PM, Jongyoul Lee wrote: > Hi all, > > We have umbrella issue for 0.7.2. See https://issues.apache.org/ > jira/browse/ZEPPELIN-2276 > > I think it's almost done and there're some trivial issues which don't look > like blocke

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread Jongyoul Lee
+1 On Mon, Jun 5, 2017 at 10:57 AM, Khalid Huseynov wrote: > +1 > > On Mon, Jun 5, 2017 at 9:49 AM, Hyung Sung Shim wrote: > > > +1 > > > > 2017-06-05 8:42 GMT+09:00 Jeff Zhang : > > > > > +1 > > > > > > Ahyoung Ryu 于2017年6月5日周一 上午4:00写道: > > > > > > > +1 for the release > > > > > > > > On Sat,

[GitHub] zeppelin issue #2381: [gh-pages] [ZEPPELIN-2597] Updating offcial main websi...

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 ![image](https://cloud.githubusercontent.com/assets/8110458/26771309/5e6b8b8e-49f8-11e7-941c-8efd7ef7c1e9.png) this part is different between #2371. Let me update to match it. --- If you

[GitHub] zeppelin issue #2381: [gh-pages] [ZEPPELIN-2597] Updating offcial main websi...

2017-06-04 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2381 splitting this PR into multiples - might be painful for the author. but it would help to review easily. - review process might be slower. so hope @AhyoungRyu would help review all sepa

[GitHub] zeppelin issue #2381: [gh-pages] [ZEPPELIN-2597] Updating offcial main websi...

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 Um.. So, for quickly review, I'll separate this PR to 2 PR. 1. updating navbar and main sections. 2. adding technologies and deployments --- If your project is set up for it, you can rep

[GitHub] zeppelin issue #2356: [ZEPPELIN-2460] Highlight active line in editor

2017-06-04 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2356 @Leemoonsoo Hi, Thanks for the feedback. - it is actually intended. otherwise highlight color will override the paragraph setting menus. - let me find an another way (e.g applying `z-

[GitHub] zeppelin issue #2356: [ZEPPELIN-2460] Highlight active line in editor

2017-06-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2356 Yeah, that was my guess. I think keeping layout unchanged is more important than prevent overlapping paragraph control and text editor. --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2372: [ZEPPELIN-2581] Shell ansi codes

2017-06-04 Thread DrIgor
Github user DrIgor commented on the issue: https://github.com/apache/zeppelin/pull/2372 Any other questions? --- 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 #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Z...

2017-06-04 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/2393 [ZEPPELIN-2590] Convert jupyter's notebook to Zeppelin's note ### What is this PR for? Converting jupyter's notebook to Zeppelin's note ### What type of PR is it? [Feature]

[GitHub] zeppelin pull request #2393: [ZEPPELIN-2590] Convert jupyter's notebook to Z...

2017-06-04 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/2393 --- 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 #2372: [ZEPPELIN-2581] Shell ansi codes

2017-06-04 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2372 I failed to test this feature since terminal i am using doesnt support that option. others can help review. sorry for the late reply. --- If your project is set up for it, you can reply to this em

[GitHub] zeppelin issue #2369: [ZEPPELIN-2591] Show user info in spark job descriptio...

2017-06-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2369 Merge to master if no further 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

[GitHub] zeppelin issue #2250: [ZEPPELIN-2085] Interpret scala code in paste mode

2017-06-04 Thread DrIgor
Github user DrIgor commented on the issue: https://github.com/apache/zeppelin/pull/2250 Please explain what issues I need to 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 this feature

[GitHub] zeppelin issue #2250: [ZEPPELIN-2085] Interpret scala code in paste mode

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2250 I think we'd better take care of the comments left. --- 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 t

[GitHub] zeppelin issue #2354: Allow group/role based authentication using LdapRealm ...

2017-06-04 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2354 LGTM. Thanks @sohaibiftikhar for the contribution. Merge to master if no further discussions here. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #2394: [gh-pages] [ZEPPELIN-2597] adding technologies ...

2017-06-04 Thread soralee
GitHub user soralee opened a pull request: https://github.com/apache/zeppelin/pull/2394 [gh-pages] [ZEPPELIN-2597] adding technologies and deployments (Main Page) ### What is this PR for? This PR is for Improving more comfortable guidance for point of user view in the main zeppe

[GitHub] zeppelin issue #2381: [gh-pages] [ZEPPELIN-2597] Updating offcial main websi...

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2381 @1ambda, @Leemoonsoo, @AhyoungRyu, I created a PR regrading `Technologies and Deployments` #2394. Let me make a remainder PR. --- If your project is set up for it, you can reply to this emai

[GitHub] zeppelin issue #2368: Improved comment, removed admin user definition per de...

2017-06-04 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2368 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 the fe

[GitHub] zeppelin issue #2394: [gh-pages] [ZEPPELIN-2597] adding technologies and dep...

2017-06-04 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2394 @AhyoungRyu @1ambda @Leemoonsoo, please review again! --- 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 #2394: [gh-pages] [ZEPPELIN-2597] adding technologies and dep...

2017-06-04 Thread sravan-s
Github user sravan-s commented on the issue: https://github.com/apache/zeppelin/pull/2394 I think it would be better if the font sizes inside top box(technologies) and bottom(deployments) were of same size and had a uniform margin / padding scheme. Also about **use now** and **learn m