[HOTFIX]: Minor clean up regarding skipped artifacts in build files.

[pwendell]

There are two relevant 'skip' configurations in the build, the first
is for "mvn install" and the second is for "mvn deploy". As of 1.2,
we actually use "mvn install" to generate our deployed artifcts,
because we have some customization of the nexus upload due to having
to cross compile for Scala 2.10 and 2.11.

There is no reason to have differents settings for these values,
this patch simply cleans this up for the repl/ and yarn/
projects.

[SparkQA]

Test build #25685 has started for PR 4080 at commit e65581c.

• This patch merges cleanly.

[pwendell]

Sorry (@srowen, @vanzin) - made a mistake on the old PR and had to open a new one here.

[SparkQA]

Test build #25685 has finished for PR 4080 at commit e65581c.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • case class SparkListenerJobEnd(
  • class SparkILoop(
  • * @param id The id (variable name, method name, class name, etc) whose
  • * Retrieves the class representing the id (variable name, method name,
  • * @param id The id (variable name, method name, class name, etc) whose
  • * @return Some containing term name (id) class if exists, else None
  • * @param id The id (variable name, method name, class name, etc) whose
  • * @param id The id (variable name, method name, class name, etc) whose
  • * Retrieves the runtime class and type representing the id (variable name,
  • * @param id The id (variable name, method name, class name, etc) whose
  • * @param id The id (variable name, method name, class name, etc) whose

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25685/
Test FAILed.

[pwendell]

@srowen my comment on the old one was that, since companies probably still use the traditional deploy pattern internally (e.g. they aren't doing anything fancy to cross build), I think it's fine to keep both settings throughout.

[pwendell]

Jenkins, test this please.

[srowen]

OK I think you're more aware of the usage. It should basically mirror the install config right?

[SparkQA]

Test build #25686 has started for PR 4080 at commit e21b78b.

• This patch merges cleanly.

[vanzin]

Hi Patrick,

This looks ok to me, but could you look at my comment in #4078? I'm not familiar with the Spark deploy scripts and I'm afraid I may have broken something there inadvertently.

[pwendell]

@vanzin yeah I think it was fine that you removed the install skop. We wanted to do that anyways because some people requested that we continue to publish those artifacts. This patch simply brings the deploy in sync with install.

[pwendell]

Jenkins, test this please.

[SparkQA]

Test build #25688 has started for PR 4080 at commit e21b78b.

• This patch merges cleanly.

[SparkQA]

Test build #25686 has finished for PR 4080 at commit e21b78b.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25686/
Test PASSed.

[SparkQA]

Test build #25688 has finished for PR 4080 at commit e21b78b.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25688/
Test PASSed.