Wang Xixu has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19949 )

Change subject: [KUDU-3483] Fixbug of auto flush data when table schema changed
......................................................................


Patch Set 12:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/19949/11/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
File 
java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java:

http://gerrit.cloudera.org:8080/#/c/19949/11/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java@370
PS11, Line 370: belong
> nit: belong
Done


http://gerrit.cloudera.org:8080/#/c/19949/11/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java@371
PS11, Line 371: schema
> nit: schemas
Done


http://gerrit.cloudera.org:8080/#/c/19949/11/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java@418
PS11, Line 418: // Compare with the lastest sch
> The batches are put into the list in the same order of operations, so regul
Done


http://gerrit.cloudera.org:8080/#/c/19949/11/java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java
File java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java:

http://gerrit.cloudera.org:8080/#/c/19949/11/java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java@139
PS11, Line 139: e
> typo?
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie6501962b32814d121f180b2942999c402d927db
Gerrit-Change-Number: 19949
Gerrit-PatchSet: 12
Gerrit-Owner: Wang Xixu <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Yuqi Du <[email protected]>
Gerrit-Comment-Date: Mon, 19 Jun 2023 02:51:01 +0000
Gerrit-HasComments: Yes

Reply via email to