Hao Hao has posted comments on this change. Change subject: spark: add support for fault tolerant scanner ......................................................................
Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/6782/6/java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduContext.scala File java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduContext.scala: Line 78: > That's true, and putting it last with a default value isn't something we'd Done http://gerrit.cloudera.org:8080/#/c/6782/8/java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala File java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala: Line 27: test("Test collect rows with non fault tolerant scanner") { > This file no longer needs to be changed, right? Yeah, DefaultSourceTest should have the same coverage. Thought it does not hurt to test it here as well. But I am ok with removing it. -- To view, visit http://gerrit.cloudera.org:8080/6782 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f3192025ca5d74197600480fd3d040d70b4bbc2 Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
