You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2019/10/18 00:25:08 UTC

[kudu-CR] [spark] separate out DefaultSourceTests

Alexey Serbin has uploaded a new patch set (#3) to the change originally created by Grant Henke. ( 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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/14491/3
-- 
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: newpatchset
Gerrit-Change-Id: I54aa0327ffb5254c03fcfe8a0a08dba230360a40
Gerrit-Change-Number: 14491
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)