LadyForest commented on a change in pull request #40:
URL: https://github.com/apache/flink-table-store/pull/40#discussion_r823428099



##########
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:
       Correct me if I'm wrong. I'm a little confused that`innerTestOnce` 
relies on `TestDataReadWrite`. But the latter hardcoded the key type, value 
type, and accumulator type to the pk situation. How does it work to test the 
value count(no pk) condition?




-- 
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


Reply via email to