[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-25 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 @bzz Yes, what you mentioned is related to #1339 which will make a change to build custom Spark. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-25 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1301 Quick question: just to double-check - this should not affect users following published build instructions like https://www.mapr.com/blog/building-apache-zeppelin-mapr-using-spark-under-yarn , right?

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 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 feature e

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-19 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 @felixcheung What I want to remove is duplicated from spark-dependencies/pom.xml. This deletion never hurt current features. It just removes duplicated codes. --- If your project is set up for

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-19 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1301 I thought we do need `cassandra-spark-*` @doanduyhai ? --- 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 #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 And about docs, it never change some behaviour. I think another issue linked above is a better point to update docs. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-18 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 @zjffdu @bzz I've already removed yarn profile. Have I missed something? --- 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 issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-16 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1301 @jongyoul do you want to take of docs update that @zjffdu mention in this PR 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. I

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-12 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1301 Definitely should remove profile yarn. I got classpath issue when enabling yarn profile. And we need to update the docs accordingly. --- If your project is set up for it, you can reply t

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-11 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1301 Thank you for explanation! Makes sense. Looks great to me. May be it's worth to `git rebase master` as CI seems to be recently fixed. --- If your project is set up for it, you can r

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 @bzz Basically, Spark module doesn't have to need hadoop version because we only use Spark repl, core and so on not including specific hadoop version. spark-dependencies only needs specific hadoo

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-09 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1301 Looks like candidate for `flaky-test` labeled JIRA issue in zeppelin-server: ``` Failed tests: ZeppelinSparkClusterTest.zRunTest:204 expected: but was: Tests run: 65, Failures: 1

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 https://travis-ci.org/apache/zeppelin/builds/150645554 --- 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 #1301: [MINOR] Removed unused profiles from spark/pom.xml

2016-08-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 https://travis-ci.org/apache/zeppelin/builds/150548730 --- 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