LadyForest commented on a change in pull request #40: URL: https://github.com/apache/flink-table-store/pull/40#discussion_r823462520
########## File path: flink-table-store-connector/src/test/java/org/apache/flink/table/store/connector/source/FileStoreSourceSplitReaderTest.java ########## @@ -64,32 +67,92 @@ public static void after() { } @Test - public void testKeyAsRecord() throws Exception { - innerTestOnce(true); + public void testPrimaryKey() throws Exception { + innerTestOnce(false, 0); + } + + @Test + public void testValueCount() throws Exception { + innerTestOnce(true, 0); Review comment: > value is a bigint. > > * has pk, value is the record > * value count, value is the record count Got that. when the record has no pk, the kV pairs stands for <record, occurrence>, and the sign stands for add/delete -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org