You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Kevin McCarthy (Code Review)" <ge...@cloudera.org> on 2020/08/03 21:45:41 UTC

[kudu-CR] [KUDU-3177] Added kudu.snapshotTimestampMicros to kudu spark readOptions as optional property

Hello Kudu Jenkins, Grant Henke, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/16276

to look at the new patch set (#3).

Change subject: [KUDU-3177] Added kudu.snapshotTimestampMicros to kudu spark readOptions as optional property
......................................................................

[KUDU-3177] Added kudu.snapshotTimestampMicros to kudu spark readOptions as optional property

Added property snapshotTimestampMs to spark read options which will allow consistant scans
when timestamp is set before the first dataFrame read.

Change-Id: I00862c0e174a964efc6cab0b8141b1ac5a1bebc0
---
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/KuduRDD.scala
M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduReadOptions.scala
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
5 files changed, 100 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/16276/3
-- 
To view, visit http://gerrit.cloudera.org:8080/16276
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I00862c0e174a964efc6cab0b8141b1ac5a1bebc0
Gerrit-Change-Number: 16276
Gerrit-PatchSet: 3
Gerrit-Owner: Kevin McCarthy <00...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kevin McCarthy <00...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)