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

Change subject: KUDU-1261 [Java] Add PartialRow.getArrayData()
......................................................................


Patch Set 5: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23563/5/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java
File java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java:

http://gerrit.cloudera.org:8080/#/c/23563/5/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java@309
PS5, Line 309:     assertEquals("foo", view.getString(0));
nit: isn't this a duplicate of line 307?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I55e7ecfc6876c8aba5f15232d570617ad008f772
Gerrit-Change-Number: 23563
Gerrit-PatchSet: 5
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 22 Oct 2025 03:40:25 +0000
Gerrit-HasComments: Yes

Reply via email to