[GitHub] zeppelin issue #1461: [MINOR] Rearrange the contents in "About Zeppelin"

2016-10-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1461 @AhyoungRyu Rebasing from master should fix the CI 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

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1489 cool. will merge this if there are 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 well. If your project does not hav

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-10-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1441 Hi @cloverhearts. Thanks for the improvement! I just tested and got some small feedbacks :) 1. Is there any particular reason to have dropdown menu next to the filter bar? If so, s

[GitHub] zeppelin pull request #1484: [branch-0.6] Take care of comma/tab escape in c...

2016-10-05 Thread minahlee
Github user minahlee closed the pull request at: https://github.com/apache/zeppelin/pull/1484 --- 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 #1486: [ZEPPELIN-1518] Fix for: Lambda expressions are not wo...

2016-10-05 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1486 Tested, it fixes the bug. Btw do we need this patch only in branch-0.6 not to master? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-10-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1441 @corneadoug sorry, could you please review for 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 do

[GitHub] zeppelin issue #1461: [MINOR] Rearrange the contents in "About Zeppelin"

2016-10-05 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1461 @corneadoug Thanks! I retriggered CI just for sure. After checking CI then will merge this :) --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] zeppelin pull request #1491: Fix version of cachebust

2016-10-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1491 --- 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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

2016-10-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1461 @AhyoungRyu Tested, 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

[GitHub] zeppelin issue #1491: Fix version of cachebust

2016-10-05 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1491 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

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1489 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 #1488: [ZEPPELIN-1519] Fix notebook importer AM/PM omission p...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1488 Thanks. LGTM and merge if there're 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 well. If your project does not

[GitHub] zeppelin issue #1490: [ZEPPELIN-1480] Blocking message pending 10000 for BLO...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1490 @corneadoug i commented translation on the 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 t

[GitHub] zeppelin pull request #1490: [ZEPPELIN-1480] Blocking message pending 10000 ...

2016-10-05 Thread Leemoonsoo
GitHub user Leemoonsoo reopened a pull request: https://github.com/apache/zeppelin/pull/1490 [ZEPPELIN-1480] Blocking message pending 1 for BLOCKING ### What is this PR for? This patch try to address problem described in ZEPPELIN-1480 ### What type of PR is it?

[GitHub] zeppelin pull request #1490: [ZEPPELIN-1480] Blocking message pending 10000 ...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/1490 --- 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 #1491: Fix version of cachebust

2016-10-05 Thread corneadoug
GitHub user corneadoug reopened a pull request: https://github.com/apache/zeppelin/pull/1491 Fix version of cachebust ### What is this PR for? A dependency update of grunt-cache-bust is breaking the build. https://github.com/hollandben/grunt-cache-bust/issues/204 So fixin

[GitHub] zeppelin pull request #1491: Fix version of cachebust

2016-10-05 Thread corneadoug
Github user corneadoug closed the pull request at: https://github.com/apache/zeppelin/pull/1491 --- 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 #1490: [ZEPPELIN-1480] Blocking message pending 10000 for BLO...

2016-10-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1490 @Leemoonsoo That Jira issue is in Korean, could you update the PR description so that there is more details on what the problem is and what is the expected result of that test before and after

[GitHub] zeppelin issue #1491: Fix version of cachebust

2016-10-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1491 Merging this directly as a hotfix --- 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 enab

[GitHub] zeppelin pull request #1491: Fix version of cachebust

2016-10-05 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1491 Fix version of cachebust ### What is this PR for? A dependency update of grunt-cache-bust is breaking the build. So fixing the dependency version ### What type of PR is it

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

2016-10-05 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1363 @Peilin-Yang Yes, you will need to rebase first though --- 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 pull request #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-10-05 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/zeppelin/pull/1441 --- 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 featur

[GitHub] zeppelin pull request #1441: [ ZEPPELIN-967 ] Implement search bar in job ma...

2016-10-05 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/zeppelin/pull/1441 [ ZEPPELIN-967 ] Implement search bar in job manager ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the c

[GitHub] zeppelin pull request #1481: [ZEPPELIN-1466] Make %dep work for spark 2.0 wh...

2016-10-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1481 --- 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 #1484: [branch-0.6] Take care of comma/tab escape in csv/tsv ...

2016-10-05 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1484 Merging it into branch-0.6 --- 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 issue #1483: [ZEPPELIN-1509] Run certain SparkInterpreter tests onl...

2016-10-05 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1483 @jongyoul, what about merging this as it is, and create another jira issue for removing Spark 1.1, 1.2 test and see how community think about removing it? --- If your project is set up for it, y

[GitHub] zeppelin pull request #1490: [ZEPPELIN-1480] Blocking message pending 10000 ...

2016-10-05 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1490 [ZEPPELIN-1480] Blocking message pending 1 for BLOCKING ### What is this PR for? This patch try to address problem described in ZEPPELIN-1480 ### What type of PR is it?

[GitHub] zeppelin issue #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/133 @doanduyhai I see. Thanks for the explanation. Current master branch works with both scala-2.10 and scala-2.11 without rebuild, depending on which version of spark SPARK_HOME points. is

[GitHub] zeppelin issue #1471: Closes [ZEPPELIN-1505] Add Scio interpreter

2016-10-05 Thread ravwojdyla
Github user ravwojdyla commented on the issue: https://github.com/apache/zeppelin/pull/1471 @AhyoungRyu sorry I missed it. Will go ahead and add all the licenses. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] zeppelin issue #1479: Notebook: HDFS as a backend storage (Read & Write Mode...

2016-10-05 Thread hayssams
Github user hayssams commented on the issue: https://github.com/apache/zeppelin/pull/1479 @Leemoonsoo Just pushed the changes to make hadoop lib provided. --- 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 d

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

2016-10-05 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1363 @corneadoug merged your PR. Do you think now it is the good time to merge this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] zeppelin issue #1487: [ZEPPELIN-1520] Test with the local mode of Apache Spa...

2016-10-05 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1487 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, o

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1489 @Leemoonsoo 8f9006a --- 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 s

[GitHub] zeppelin issue #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-10-05 Thread doanduyhai
Github user doanduyhai commented on the issue: https://github.com/apache/zeppelin/pull/133 @Leemoonsoo Yes we do need this patch because `DisplayUtils` has been disabled temporarily because of the issue it causes. This patch fixes the issue and re-enable `DisplayUtils`

[GitHub] zeppelin issue #133: [ZEPPELIN-127] Fix issue raised by ZEPPELIN-127 related...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/133 master branch runs ``` val wbIPR=Array(8.40, 8.09, 7.56, 6.83, 5.88, 4.72, 3.36, 1.78, 0.0) val pressureIPR=Array(0.00,1000.00,2000.0

[GitHub] zeppelin issue #1479: Notebook: HDFS as a backend storage (Read & Write Mode...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1479 Thanks for the contribution. How about let user set not only `HADOOP_CONF_DIR` but also `HADOOP_HOME` in conf/zeppelin-env.sh to use HDFS notebook storage? Then hadoop dependen

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1489 Tested on local, 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

[GitHub] zeppelin issue #1483: [ZEPPELIN-1509] Run certain SparkInterpreter tests onl...

2016-10-05 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1483 nice ! 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 #1483: [ZEPPELIN-1509] Run certain SparkInterpreter tests onl...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1483 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 #1484: [branch-0.6] Take care of comma/tab escape in csv/tsv ...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1484 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 #1486: [ZEPPELIN-1518] Fix for: Lambda expressions are not wo...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1486 Thanks @mvitaly for the contribution! 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

[GitHub] zeppelin issue #1483: [ZEPPELIN-1509] Run certain SparkInterpreter tests onl...

2016-10-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1483 @minahlee Thank you for your effort!! :+1: --- 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 fea

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1489 @Leemoonsoo sure. will update. --- 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 a

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1489 Shell we have an entry for this change in [upgrading section](https://github.com/apache/zeppelin/blob/master/docs/install/upgrade.md#upgrading-from-zeppelin-06-to-07)? --- If your project is s

[GitHub] zeppelin issue #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

2016-10-05 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/1489 Travis failure is unrelated and only on one of the build profile --- 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 d

[jira] [Created] (ZEPPELIN-1522) Not correct (change) display of the height of diagram in IE

2016-10-05 Thread Ivan Savin (JIRA)
Ivan Savin created ZEPPELIN-1522: Summary: Not correct (change) display of the height of diagram in IE Key: ZEPPELIN-1522 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1522 Project: Zeppelin

[jira] [Created] (ZEPPELIN-1521) Not correct display of the lower axe in Chrome

2016-10-05 Thread Ivan Savin (JIRA)
Ivan Savin created ZEPPELIN-1521: Summary: Not correct display of the lower axe in Chrome Key: ZEPPELIN-1521 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1521 Project: Zeppelin Issue T

[GitHub] zeppelin pull request #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port shou...

2016-10-05 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/zeppelin/pull/1489 ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be managed seperately ### What is this PR for? Use a different variable for ssl port number - add new property for ssl port ###

[GitHub] zeppelin pull request #1488: [ZEPPELIN-1519] Fix notebook importer AM/PM omi...

2016-10-05 Thread sixmen
GitHub user sixmen opened a pull request: https://github.com/apache/zeppelin/pull/1488 [ZEPPELIN-1519] Fix notebook importer AM/PM omission problem ### What is this PR for? When loading note from storage (or importing note), AM/PM informations are gone. So dates are always in AM

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

2016-10-05 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1441 code rebase --- 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

Re: Refactor time !

2016-10-05 Thread Jungtaek Lim
Hi Jongyoul, I strongly agree that changing requirement of JDK version is definitely breaking change. May be better to treat it to separate issue. I just think it's good food for thought, not meant to do it now. If Zeppelin project plans to have major version (1.0.0 finally), that would be a chanc

Re: Refactor time !

2016-10-05 Thread Jongyoul Lee
Hello Jungtaek, I agree that it's relatively easy to run Zeppelin with JDK 8 because Zeppelin is not linked with another project directly. But as you told, it bothers admin or system engineer, sometimes it's impossible for some reasons. Thus personally, we should keep backward compatibility. On W

Re: Refactor time !

2016-10-05 Thread Jungtaek Lim
I also think there's also many places to use old version of JDK in production. But unlike library projects, Zeppelin doesn't become dependency of other projects, so no need to consider other projects and also users' own projects, which greatly lowers the barrier. Installing JDK/JRE 8 and using it f