Skip to content

Commit b06389c

Browse files
committed
[SPARK-7805] [SQL] Move SQLTestUtils.scala and ParquetTest.scala to src/test
https://issues.apache.org/jira/browse/SPARK-7805 Because `sql/hive`'s tests depend on the test jar of `sql/core`, we do not need to store `SQLTestUtils` and `ParquetTest` in `src/main`. We should only add stuff that will be needed by `sql/console` or Python tests (for Python, we need it in `src/main`, right? davies). Author: Yin Huai <[email protected]> Closes #6334 from yhuai/SPARK-7805 and squashes the following commits: af6d0c9 [Yin Huai] mima b86746a [Yin Huai] Move SQLTestUtils.scala and ParquetTest.scala to src/test. (cherry picked from commit ed21476) Signed-off-by: Yin Huai <[email protected]>
1 parent 11d998e commit b06389c

File tree

4 files changed

+11
-1
lines changed

4 files changed

+11
-1
lines changed

project/MimaExcludes.scala

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,10 @@ object MimaExcludes {
133133
"org.apache.spark.sql.parquet.TestGroupWriteSupport"),
134134
ProblemFilters.exclude[MissingClassProblem]("org.apache.spark.sql.CachedData"),
135135
ProblemFilters.exclude[MissingClassProblem]("org.apache.spark.sql.CachedData$"),
136-
ProblemFilters.exclude[MissingClassProblem]("org.apache.spark.sql.CacheManager")
136+
ProblemFilters.exclude[MissingClassProblem]("org.apache.spark.sql.CacheManager"),
137+
// TODO: Remove the following rule once ParquetTest has been moved to src/test.
138+
ProblemFilters.exclude[MissingClassProblem](
139+
"org.apache.spark.sql.parquet.ParquetTest")
137140
) ++ Seq(
138141
// SPARK-7530 Added StreamingContext.getState()
139142
ProblemFilters.exclude[MissingMethodProblem](
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
README
2+
======
3+
4+
Please do not add any class in this place unless it is used by `sql/console` or Python tests.
5+
If you need to create any classes or traits that will be used by tests from both `sql/core` and
6+
`sql/hive`, you can add them in the `src/test` of `sql/core` (tests of `sql/hive`
7+
depend on the test jar of `sql/core`).

0 commit comments

Comments
 (0)