[SPARK-13050] [Build] Scalatest tags fail build with the addition of the sketch module

[ajbozarth]

A dependency on the spark test tags was left out of the sketch module pom file causing builds to fail when test tags were used. This dependency is found in the pom file for every other module in spark.

[SparkQA]

Test build #50221 has finished for PR 10954 at commit 6ab0ec9.

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

[holdenk]

This looks reasonable to me, maybe @liancheng who added the sketch build file can take a look as well?

[ajbozarth]

Also looping in @rxin and @JoshRosen since they were in on #10851 which caused this

[holdenk]

Would it maybe be a good idea to add a test for this to the jenkins scripts?

[liancheng]

One of the requirements of the spark-sketch module is to have zero external runtime dependencies. Is it possible to make it a test dependency? It seems quite reasonable since those tags are only used in test code.

[liancheng]

Hm, but I do find that spark-test-tags is listed as default compile dependency in other modules. Is that necessary? @JoshRosen

[vanzin]

spark-test-tags is declared with a test scope in the root pom, and the scope is inherited. You can check these things with mvn dependency:tree.

[liancheng]

@vanzin I see, thanks for the explanation! Then this LGTM.

[liancheng]

I'm merging this. Thanks for fixing and reviewing this!