Hello Mike Percy, Grant Henke,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/11916
to review the following change.
Change subject: KUDU-2599: fix flaky
DefaultSourceTest.testSocketReadTimeoutPropagation
......................................................................
KUDU-2599: fix flaky DefaultSourceTest.testSocketReadTimeoutPropagation
This test is just about checking the propagation of an option from the
DefaultSource to the KuduRelation. However, building a DataFrame involves a
connection to the cluster, and if we set socketReadTimeoutMs to 1, it's
going to be very difficult to connect, especially if we're testing
TSAN-instrumented binaries whose thread creation is inherently slower.
The fix? Use a magic number larger than '1'. It's that simple.
Change-Id: I81a20f776c115caf3bd720bc532bbeb5fc69bc42
---
M
java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala
1 file changed, 8 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/16/11916/1
--
To view, visit http://gerrit.cloudera.org:8080/11916
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I81a20f776c115caf3bd720bc532bbeb5fc69bc42
Gerrit-Change-Number: 11916
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
