[GitHub] incubator-zeppelin pull request: ZEPPELIN-496 ] Bug fixed Spark sc...

GitHub user cloverhearts opened a pull request: https://github.com/apache/incubator-zeppelin/pull/528 ZEPPELIN-496 ] Bug fixed Spark scala completion, support other interpreter issue #514 RE-Fixed ZEPPELIN-485 ] Bug Fixed Paragraph Spark Completion --- bug fix

[jira] [Created] (ZEPPELIN-496) Code auto-complete only works only spark-scala

Chae-Sung Lim created ZEPPELIN-496: -- Summary: Code auto-complete only works only spark-scala Key: ZEPPELIN-496 URL: https://issues.apache.org/jira/browse/ZEPPELIN-496 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: ZEPPELIN-268 : Adding Data Valida...

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/262#issuecomment-163519458 My main concern with this PR is performances: * We are parsing the full data to make sure it fit the graph format * The validate function is launche

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-495] Enable running int...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/524#issuecomment-163518787 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-490 Tidying up dynamicfo...

Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/518#issuecomment-163507469 Thank you for review @felixcheung ! ☺ Because of *same language code tab showing* and *code highlighter*, as you mentioned, I made a new javasc

[GitHub] incubator-zeppelin pull request: [TEST] Run CI test

GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/527 [TEST] Run CI test This Pullrequest will never be merged. Run test on CI repeatedly to figure out flaky test and resolutions. You can merge this pull request into a Git repo

[GitHub] incubator-zeppelin pull request: ZEPPELIN-305 Do not add jvm memor...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/526#issuecomment-163494534 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 h

[GitHub] incubator-zeppelin pull request: ZEPPELIN-305 Do not add jvm memor...

GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/526 ZEPPELIN-305 Do not add jvm memory option when using spark-submit https://issues.apache.org/jira/browse/ZEPPELIN-305 When `SPARK_HOME` is defined and `bin/interpreter.sh` is

[GitHub] incubator-zeppelin pull request: ZEPPELIN-305 Do not add jvm memor...

Github user Leemoonsoo closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/526 --- 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] incubator-zeppelin pull request: ZEPPELIN-305 Do not add jvm memor...

GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/526 ZEPPELIN-305 Do not add jvm memory option when using spark-submit https://issues.apache.org/jira/browse/ZEPPELIN-305 When `SPARK_HOME` is defined and `bin/interpreter.sh` is l

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163464015 Then, Merging if there is no more discussions --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] incubator-zeppelin pull request: Edit title fix for Notebook and P...

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/512#issuecomment-163463381 @r-kamath Could you rebase? Seems there is some conflicts --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: [DISCUSS] Graduate Zeppelin from the Incubator

I'd like to see Amos's take on the situation. Amos, could you let us know, please? Thanks, Roman. On Wed, Dec 9, 2015 at 7:24 PM, moon soo Lee wrote: > Hi Amos, Roman, > > I thought we are already sorting out Amos's issue. > > Had a discussion about license and made conclusion. > And now lookin

Re: [DISCUSS] Graduate Zeppelin from the Incubator

Hi Amos, Roman, I thought we are already sorting out Amos's issue. Had a discussion about license and made conclusion. And now looking at CI test error with R Interpreter, and finding and fixing CI problem one by one, while that becomes the priority for this contributions. Also we are addressing

Re: [DISCUSS] Graduate Zeppelin from the Incubator

Like I said -- I would strongly recommend sorting out the issue with Amos ahead of committing to graduation vote. Thanks, Roman. On Wed, Dec 9, 2015 at 3:07 PM, moon soo Lee wrote: > If everything is okay, shall we start community vote? > > fyi, graduation process, > http://incubator.apache.org/

Re: [DISCUSS] Graduate Zeppelin from the Incubator

If everything is okay, shall we start community vote? fyi, graduation process, http://incubator.apache.org/guides/graduation.html#toplevel Thanks, moon On Tue, Dec 1, 2015 at 1:39 PM Alexander Bezzubov wrote: > Apparently it takes a re-build of incubator website >

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163384653 Looks good to me. Tested bookmarklet works really well. FYI, According to issuestat.com that @r-kamath has mentions, Time distributio

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-493] Test fails VFSNote...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/522 --- 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] incubator-zeppelin pull request: Replace standard alert and confir...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/501 --- 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] incubator-zeppelin pull request: Replace standard alert and confir...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-163379597 Tested and LGTM. Merging it into master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] incubator-zeppelin pull request: Add job operation API

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-163378097 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 fe

[GitHub] incubator-zeppelin pull request: ZEPPELIN-385 Read-only mode for z...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/389#issuecomment-163376715 @babokim Do you mind rebase or merge master to resolve the conflicts? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-484] A small utility to...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/513#issuecomment-163375121 Thanks all the feedbacks and alternative solutions. I have changed script name from `test.py` to `test_zeppelin_pr.py` and add more comment. A

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-493] Test fails VFSNote...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/522#issuecomment-163369613 I'm merging it into the 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 pro

[GitHub] incubator-zeppelin pull request: bump up org.seleniumhq.selenium t...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/504 --- 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] incubator-zeppelin pull request: Add job operation API

Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-163287793 @bzz I updated REST API docs, could you review? Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin pull request: ZEPPELIN-408 Fix configuration bu...

Github user chiwanpark commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/420#issuecomment-163228058 @jongyoul I triggered Travis again and got an error in `ZeppelinIT.testAngularDisplay:183`. Is this error relative to this changes? --- If your project i

[GitHub] incubator-zeppelin pull request: ZEPPELIN-485 ] Bug Fixed Paragrap...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/514 --- 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] incubator-zeppelin pull request: Edit title fix for Notebook and P...

Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/512#issuecomment-163179592 @corneadoug try merging master on #324 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163167311 @corneadoug similar to build status this badge is useful for checking the pr status --- If your project is set up for it, you can reply to this email and h

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163165199 @r-kamath what would be the used of that badge? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163164018 :+1: for bookmarklet --- 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 no

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163163806 @corneadoug can you also add PR status in readme ? `[](http://issuestats.c

[GitHub] incubator-zeppelin pull request: Interpreter properties editor upd...

GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/525 Interpreter properties editor updates - Add extra validations in add new interpreter property function to prevent 'undefined' - Instead of an empty table show a message when th

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

Github user swkimme commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-163154333 Cool! LGTM 2015년 12월 9일 (수) 오후 5:44, CORNEAU Damien 님이 작성: > Thanks, LGTM > > — > Reply to this email dire

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-163151792 Thanks, 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] incubator-zeppelin pull request: Add/pr teamplate

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163147825 1. I don't have a lot of Open Source projects example that I can post right now, but it is quite usual for projects to have their own Issue Template or PR

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-495] Enable running int...

GitHub user minahlee opened a pull request: https://github.com/apache/incubator-zeppelin/pull/524 [ZEPPELIN-495] Enable running interpreters with distribution package In distribution package, zeppelin-interpreter module is not added to classpath when Zeppelin starts interpreter proc

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163145070 This is a really good template. Two questions: 1. Do other open source projects find this level of formality necessary? I'm not saying its a