wuchong commented on code in PR #2537:
URL: https://github.com/apache/fluss/pull/2537#discussion_r2753361925
##########
fluss-lake/fluss-lake-lance/src/test/java/org/apache/fluss/lake/lance/tiering/LanceTieringITCase.java:
##########
@@ -144,4 +196,61 @@ private void checkDataInLanceAppendOnlyTable(LanceConfig
config, List<InternalRo
assertThat(flussRowIterator.hasNext()).isFalse();
}
}
+
+ private void checkDataInLanceWithAllArrayTypes(
Review Comment:
The data assertion code appears quite verbose and hard to maintain. Could we
simplify it by using `readerRoot.contentToTSVString()`—similar to how we assert
Flink `Row` results? This would make the test more concise and easier to read.
Additionally, it seems you haven’t merged this test into `testTiering()`
yet. Is there a specific reason this can’t be done? If so, could you please
clarify?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
