Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11537 )
Change subject: [spark] Add KuduReadOptions to encapsulate the read parameters ...................................................................... [spark] Add KuduReadOptions to encapsulate the read parameters This patch adds KuduReadOptions and refactors the exisiting private APIs to leverage it. Additionally some refactoring was done to KuduWriteOptions to ensure the interaction with both classes was unified. This patch is a preliminary step to adding keep alive support to the Spark integration. Change-Id: Iaee3d09471cad2cddd816ec77eafaae457faf1c2 Reviewed-on: http://gerrit.cloudera.org:8080/11537 Tested-by: Kudu Jenkins Reviewed-by: Hao Hao <[email protected]> --- M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduContext.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduRDD.scala A java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduReadOptions.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduWriteOptions.scala M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/SparkUtil.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala 7 files changed, 170 insertions(+), 133 deletions(-) Approvals: Kudu Jenkins: Verified Hao Hao: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11537 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iaee3d09471cad2cddd816ec77eafaae457faf1c2 Gerrit-Change-Number: 11537 Gerrit-PatchSet: 6 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]>
