Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18211 )

Change subject: [java] KUDU-3350 add the support for deleteIgnoreRows
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/18211/4/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:

http://gerrit.cloudera.org:8080/#/c/18211/4/java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/KuduContext.scala@347
PS4, Line 347: ${numDeletes.value.get(tableName)}
> The numDeletes Accumulator will increase after the KuduContext's writeParti
Let me clarify: I'm not sure I understand how it's possible to report report 
the actual number of deleted rows when using DELETE_IGNORE.  As I understand, 
the actual number of deleted rows is known only to the server side since it's 
not reported to the client.  Do I miss anything?



--
To view, visit http://gerrit.cloudera.org:8080/18211
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6f89ced9ffa4a79f46661873f01c38aefb1d78d5
Gerrit-Change-Number: 18211
Gerrit-PatchSet: 4
Gerrit-Owner: Hongjiang Zhang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Hongjiang Zhang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 15 Feb 2022 02:35:19 +0000
Gerrit-HasComments: Yes

Reply via email to