Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8823 )

Change subject: [WIP]KUDU-1704: add c++ client support for READ_YOUR_WRITES mode
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/8823/1/src/kudu/client/client-test.cc
File src/kudu/client/client-test.cc:

http://gerrit.cloudera.org:8080/#/c/8823/1/src/kudu/client/client-test.cc@1194
PS1, Line 1194:
> rename this test?
Done


http://gerrit.cloudera.org:8080/#/c/8823/1/src/kudu/client/client-test.cc@1199
PS1, Line 1199:   NO_FATALS(InsertTestRows(client_table_.get(), 
FLAGS_test_scan_num_rows));
              :   KuduScanner scanner(client_table_.get());
              :   ASSERT_OK(scanner.SetProjectedColumns({ "key" }));
              :   ASSERT_OK(scanner.Open());
              :
              :   ASSERT_TRUE(scanner.HasMoreRows());
              :   KuduScanBatch batch;
              :   while (scanner.HasMoreRows()) {
              :     ASSERT_OK(scanner.NextBatch(&batch));
              :     for (const KuduScanBatch::RowPtr& row : batch) {
              :       ASSERT_NE("(int32 key=<redacted>)", row.ToString());
              :     }
              :   }
> this doesn't strike me as a very good test, particularly if we make this mo
Added more unit tests for scan with different replica selection, and multi 
tablets scan. I am planning to add some integration tests.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I34214245a78aed172a28fbdb395ff5bccd0fc0e1
Gerrit-Change-Number: 8823
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Fri, 12 Jan 2018 03:58:18 +0000
Gerrit-HasComments: Yes

Reply via email to