Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14491 )
Change subject: [spark] Separate out DefaultSourceTests ...................................................................... [spark] Separate out DefaultSourceTests We have seen flaky test failures due to timeouts of DefaultSourceTest. This is primarily due to the sheer number of tests in that class. This patch break out the SQL based tests, ones using `sqlContext.sql(…)` into their own class. There is no change in test methods or coverage. The result is 22 DefaultSourceTests and 22 SparkSQLTests. Change-Id: I54aa0327ffb5254c03fcfe8a0a08dba230360a40 Reviewed-on: http://gerrit.cloudera.org:8080/14491 Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Hao Hao <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala A java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/SparkSQLTest.scala 3 files changed, 548 insertions(+), 497 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14491 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I54aa0327ffb5254c03fcfe8a0a08dba230360a40 Gerrit-Change-Number: 14491 Gerrit-PatchSet: 5 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
