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

2017-06-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2309 I'll merge it if there's 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

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

2017-06-18 Thread Jongyoul Lee
age 1] > > > On Wed, Jun 14, 2017 at 8:48 PM, Raffaele S wrote: > >> I believe it's a good idea to select which interpreters to keep in the >> "default release", should we track this through JIRA? >> >> 2017-06-11 17:05 GMT+02:00 Jongyoul Lee : >

[GitHub] zeppelin pull request #2418: [ZEPPELIN-2645] Adding way to register RemoteIn...

2017-06-17 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/2418 [ZEPPELIN-2645] Adding way to register RemoteInterpreterServer's port into InterpreterProcess ### What is this PR for? This is a thrift version related to #2410 ###

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

2017-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 Merging if there's 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 fe

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

2017-06-15 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

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

2017-06-15 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

[GitHub] zeppelin issue #2398: [ZEPPELIN-2622] Add Zeppelin Git info REST API

2017-06-14 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2398 Merging if there's 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 fe

[GitHub] zeppelin pull request #2410: [ZEPPELIN-2645] Adding way to register RemoteIn...

2017-06-14 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/2410 [ZEPPELIN-2645] Adding way to register RemoteInterpreterServer's port into InterpreterProcess ### What is this PR for? Changing the way to register interpreter to server. interpreter

[jira] [Created] (ZEPPELIN-2645) Adding way to register RemoteInterpreterServer's port into InterpreterProcess

2017-06-13 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-2645: -- Summary: Adding way to register RemoteInterpreterServer's port into InterpreterProcess Key: ZEPPELIN-2645 URL: https://issues.apache.org/jira/browse/ZEPPELIN

[GitHub] zeppelin issue #2329: [WIP][ZEPPELIN-2040] ClusterManager to support launchi...

2017-06-13 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu I agree with you. It's very basic and minimum feature to run interpreters in Zeppelin. I'll take that issue ASAP. --- If your project is set up for it, you can reply to this

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

2017-06-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2309 @1ambda Thanks. @tinkoff-dwh Please catch up the 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

[GitHub] zeppelin issue #2404: fix bugs for: when interpreter setting restart of one ...

2017-06-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2404 Let me test it. I wonder if it works when we restart Zeppelin itself or not. --- 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 #2404: fix bugs for: when interpreter setting restart of one ...

2017-06-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2404 @wary Thanks for additional description. Let's check it. It's very fragile part of Apache Zeppelin for now. Did you test other behaviors related this features? --- If your project

[GitHub] zeppelin issue #2404: fix bugs for: when interpreter setting restart of one ...

2017-06-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2404 Hi, It's intended behaviours. If you want to restart your interpreter only, you can click the restart button on the note page. --- If your project is set up for it, you can reply to this

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

2017-06-11 Thread Jongyoul Lee
> https://github.com/kubernetes/kubernetes/tree/master/ > examples/spark#known-issues-with-zeppelin > > On Mon, Jun 5, 2017, 06:54 Jongyoul Lee wrote: > > > Felix, > > Yes, I said a bit confused. I want to release Zeppelin with some - > not-all > > - interpre

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

2017-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 Passed. https://travis-ci.org/jongyoul/zeppelin/builds/241524191 --- 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 #2403: ZEPPELIN-2637: Make useLegacySql configurable, in Bigq...

2017-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2403 Thanks. Merge it if there's 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 #2402: [ZEPPELIN-2636] User role lookup via interfaces

2017-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2402 Thanks for contribution. I suggest you'd better divide this PR into three or more different PRs as you mentioned in description. This PR includes many changes. I hope every PR changes as

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

2017-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2282 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 enabled and wishes so, or

[GitHub] zeppelin issue #2403: ZEPPELIN-2637: Make useLegacySql configurable, in Bigq...

2017-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2403 @karuppayya Can you re-trigger the last profile of CI? https://travis-ci.org/karuppayya/zeppelin/builds/241501388 --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #2398: [ZEPPELIN-2622] Add Zeppelin Git info REST API

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

Re: [VOTE] Release Apache Zeppelin 0.7.2 (RC1)

2017-06-10 Thread Jongyoul Lee
epos/dist/release/zeppelin/KEYS > > > > Release notes available at > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa? > version=12340231&projectId=12316221 > > > > Vote will be open for next 72 hours (close at 01:00 12/Jun PDT). > > > > [ ] +1 approve > > [ ] 0 no opinion > > [ ] -1 disapprove (and reason why) > > > -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

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

2017-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 @cloverhearts Can you tested it? I've pushed some commit and deployed it to snapshots. And also please test it with v3 format :-) --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiTest.tes...

2017-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2206 Can you fix it to fit with current master 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

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 I found it's your first contribution. Thank 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 project

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @felixcheung Yes, I already checked it but I was confused the branch name is master. :-) Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 Can you 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 does not have this feature enabled and

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 Thanks, Will merge it after CI become 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 this

[GitHub] zeppelin issue #2397: [ZEPPELIN-2592] Ensure open stream is closed

2017-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2397 @andreaTP Thanks for fixing it. BTW, I think we'd better use try-with-resources of outputStream --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] zeppelin issue #2397: [ZEPPELIN-2592] Ensure open stream is closed

2017-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2397 @andreaTP try() doesn't call close when it occurs exception? --- 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 #2397: [ZEPPELIN-2592] Ensure open stream is closed

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

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-06-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 Oh, I'm sorry for the late reply. I think you'd better to move those classes into another PR --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2383 Thanks, I just wonder if it works becase it was developed when spark was 1.5. One more question. Did you test -sparkR ? If you did, can you compare which one is proper for you? We had two

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

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

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

2017-06-05 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2250 I'm a bit confused because there's some unrelated changes regarding this issue. I don't know why you change zeppelin-interpreter package and why you make hash code and equals

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

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

[GitHub] zeppelin issue #2390: [ZEPPELIN-2589]Let zeppelin show more than one line of...

2017-06-05 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2390 @qwemicheal Thanks for the contribution. Can you handle the issues @zjffdu mentioned? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin pull request #2395: [ZEPPELIN-2515] After 100 minutes R process qui...

2017-06-05 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/2395 [ZEPPELIN-2515] After 100 minutes R process quits silently and spark.r interpreter becomes unresponsive. ### What is this PR for? Enabling to set timeout value for R backend

[GitHub] zeppelin issue #2206: [ZEPPELIN-1856] Flaky Test: InterpreterRestApiTest.tes...

2017-06-05 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2206 @AlexanderShoshin Can you rebase it from current master? --- 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

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

2017-06-05 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2393 @felixcheung Thanks for the review. I'll fix somethings and will remain others for the community. I think there's Python expert in the community and they will help to improve th

[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

[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

[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

[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

Re: [DISCUSS] Release 0.7.2

2017-06-04 Thread Jongyoul Lee
> > > > > > Since 0.7.1 has major problems like ZEPPELIN-2336 [2], i'd like to > > > > proceed > > > > > release process for 0.7.2 at this point and provide stable release. > > > > > > > > > > Remaining issue can be resolved in 0.8.0 release (or 0.7.3 if > another > > > > > maintenance release is necessary). > > > > > > > > > > Shell we proceed to 0.7.2? > > > > > > > > > > Thanks, > > > > > moon > > > > > > > > > > [1] https://issues.apache.org/jira/browse/ZEPPELIN-2276 > > > > > [2] https://issues.apache.org/jira/browse/ZEPPELIN-2336 > > > > > > > > > > > > > > > -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

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 is

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

2017-06-04 Thread Jongyoul Lee
rpreters) package but didn't conclude the criteria and how we make a >> decision. >> >> Jongyoul, do you have any idea about criteria? >> >> Thanks, >> moon >> >> On Sun, Jun 4, 2017 at 10:47 AM Felix Cheung >> wrote: &g

Release 0.7.2

2017-06-04 Thread Jongyoul Lee
0. Regards, JL -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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 wish

[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

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

2017-06-04 Thread Jongyoul Lee
, Jun 4, 2017 at 9:11 AM, Felix Cheung wrote: > Are we proposing some interpreters to be built and released separately? > > Is this going to be separate packaging? Or separate release pipeline? > > > _ > From: Jongyoul Lee mailto:jongy...@gmail.com&g

[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

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

2017-06-03 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/2393 [WIP][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

[GitHub] zeppelin issue #2383: [ZEPPELIN-2606] Fix compilation with R interpreter ena...

2017-06-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2383 I have general question. Does -Pr works at the latest spark distribution? --- 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 #2384: [ZEPPELIN-2577] No graphs visible on IE11

2017-06-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2384 After merging this PR, let's proceed to release 0.7.2 --- 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 doe

[DISCUSS] Reducing default interpreters while building and releasing Zeppelin

2017-06-02 Thread Jongyoul Lee
s the most important factor. If someone wants to have an idea to deal with it, please feel free to talk about it. Thanks, Jongyoul Lee -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

[GitHub] zeppelin issue #2182: [HOLD][ZEPPELIN-2248] Fix javax.ws.rs and cxf version ...

2017-06-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2182 Will close it as resolved by #932 --- 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 #2182: [HOLD][ZEPPELIN-2248] Fix javax.ws.rs and cxf v...

2017-06-02 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/2182 --- 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 issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2017-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 Tested and LGTM. Sorry for the late reply. I'll 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

[jira] [Created] (ZEPPELIN-2590) Convert jupyter file format to zeppelin's note format

2017-05-26 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-2590: -- Summary: Convert jupyter file format to zeppelin's note format Key: ZEPPELIN-2590 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2590 Project: Zep

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

2017-05-26 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2368 Generally, it makes sense 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

Re: Can not start zeppelin on master branch

2017-05-26 Thread Jongyoul Lee
t; org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114) > at > > org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61) > at org.eclipse.jetty.server.Server.doStart(Server.java:354) > at > > org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68) > at > org.apache.zeppelin.server.ZeppelinServer.main(ZeppelinServer.java:195) > -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

Re: JdbcInterpreter might be too generic

2017-05-24 Thread Jongyoul Lee
le when the jdbc interpreter is > > implemented. if user create interpreter setting for each database, then > he > > doesn't need to specify `(mysql)` as point 1. > > > > Welcome any comments and thoughts on this. > > > -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-05-19 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2125 @conker84 Hi, I have a question. I'm not sure but classes under org.apache.zeppelin.interpreter.graph is used only for testing it. Can you verify it? If it does, can you move these classes

[GitHub] zeppelin issue #2351: [ZEPPELIN-2544] clean task doesn't work in zeppelin-we...

2017-05-19 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2351 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

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2017-05-19 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 @sohaibiftikhar Sure, I didn't test it yet as I had a talk in ApacheCon last Wednesday. I'll handle it soon. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2017-05-13 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 @sohaibiftikhar Thanks for verifying 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

Re: Spark Context in custom interpreter

2017-05-13 Thread Jongyoul Lee
raphs. > > Can anyone advise me on the best way to get hold of that Context please? > -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

Re: Help verify yarn client mode without SPARK_HOME

2017-05-13 Thread Jongyoul Lee
cluster and succeed to > launches > SparkInterpreter with local mode in yarn cluster. > Do you mean you can run spark intepreter in yarn cluster mode with > SPARK_HOME set ? > > IIUC, running spark interpreter in yarn-client or yarn-cluster mode require > SPARK_HOME and HADOOP_CONF_DIR

Re: What is the user case for remote of interpreter option is false

2017-05-13 Thread Jongyoul Lee
ages. > >>> However, keeping code base for remote=false and remote=true introduces > >>> more > >>> work. > >>> > >>> What do you think? > >>> > >>> Thanks, > >>> moon > >>> > >>> On

Reverse commit of 15ecbb49db17f3938ca0b4d584193354ff243429

2017-05-12 Thread Jongyoul Lee
Hi dev, Recently, Ci has failed because some profiles changed. It's necessary to enable it but now some tests fails. Then, we revert it first, fix that problem and merge it later. Regards, Jongyoul Lee -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

[GitHub] zeppelin issue #2300: [ZEPPELIN-2467] Restoring second travis job functional...

2017-05-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2300 Thanks for your opinion. Then I'll revert it. Please reopen this PR and rebase it from current master. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #2300: [ZEPPELIN-2467] Restoring second travis job functional...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2300 @AlexanderShoshin After merging this PR, we continued to got failed in CI. Can you handle it urgently? If it took some times, it would be better to revert this commit and merge it after fixing

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @Tagar Thanks for reaching it out. In case of Spark, I feel like we need to focus on external spark mode more. In case of `spark-defaults.conf`, AFAIK, because we don't need to deploy spa

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu Personally, I agree on your opinion. Before I start to do it, I discussed a lot how to support yarn-cluster mode including spark. I concluded it's not much weird to launch spark

[GitHub] zeppelin issue #1446: ZEPPELIN-1263. Should specify zeppelin's spark configu...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1446 I didn't dig into it but have simple question. Does it support embedded mode? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu I'm adding to read HADOOP_CONF_DIR. Concerning spark, I recommend, for now, to set local[*], and yarn-client. In case of yarn-cluster, I think native approach would be better. I&#x

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 And what I meant ux is about supporting spark's native yarn-cluster mode in Zeppelin :-) --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 I mean 'not in case of spark'. And this implementation satisfies your needs. You can launches some interpreters like md in local mode and some interpreters in yarn. --- If your proj

[GitHub] zeppelin issue #2336: [MINOR] Change `case` indent

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2336 This is minor issue and CI failure isn't related to 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 pr

[GitHub] zeppelin pull request #2336: [MINOR] Change `case` indent

2017-05-10 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/2336 [MINOR] Change `case` indent ### What is this PR for? Changing style in case of 'case'. Zeppelin has two spaces as default indentation but checkstyle has different styl

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 It makes our logic much difficult, but it's improve users' experiences. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 In case of Spark, it has two yarn apps. I agree that it's a bit weird for users. Otherwise, all other interpreters like python get better to launch it in yarn cluster. I think it would be b

[GitHub] zeppelin issue #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to support lau...

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2329 @zjffdu Thanks for quick review. I know spark already have yarn-cluster by itself. In case of Zeppelin, SparkInterpreter will run yarn-client mode which means driver is still in a process

[GitHub] zeppelin issue #2333: Notebook: (Web) HDFS as a backend storage (Read & Writ...

2017-05-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2333 I don't think it's good idea to include some interpreters as dependencies onto zeppelin-zengine. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #2329: [WIP][PoC] ZEPPELIN-2040 ClusterManager to supp...

2017-05-09 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/2329 [WIP][PoC] ZEPPELIN-2040 ClusterManager to support launching interpreter in a cluster ### What is this PR for? Launching interpreters into yarn cluster. This is PoC level now and has many

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2017-05-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 Actually, I'm not sure I didn't dig into it. My current assumption is to support anonymous and authorized one --- If your project is set up for it, you can reply to this email and

Re: Help verify yarn client mode without SPARK_HOME

2017-05-09 Thread Jongyoul Lee
Thanks for testing it. On Wed, May 10, 2017 at 2:16 AM, Jeff Zhang wrote: > yarn-client mode doesn't work for embeded spark. But does it work before ? > I think embeded spark should only work with local mode > > > > > Jongyoul Lee 于2017年5月9日周二 上午10:02写道: > > > Hi

Help verify yarn client mode without SPARK_HOME

2017-05-09 Thread Jongyoul Lee
ode in current master. Can anyone test it and let me know the result? Thanks in advance, Jongyoul -- 이종열, Jongyoul Lee, 李宗烈 http://madeng.net

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

2017-05-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/932 I also had a similar problem. We need to handle it. On Mon, May 8, 2017 at 11:31 PM, Sohaib Iftikhar wrote: > @echarles <https://github.com/echarles> I tested

<    2   3   4   5   6   7   8   9   10   11   >