Skip to content

Commit cc63767

Browse files
buvbwuchong
authored andcommitted
[lake/paimon] Fix timestamp field assertion in test
- Use getTimestamp() instead of getLong() for timestamp system column - Addresses review comment: 'should assert getTimestamp because this is a timestamp type' This completes all review feedback from wuchong in PR apache#2189
1 parent b0357bb commit cc63767

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

fluss-lake/fluss-lake-paimon/src/test/java/org/apache/fluss/lake/paimon/tiering/FlussRecordAsPaimonRowTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -635,7 +635,8 @@ void testPaimonSchemaWiderThanFlussRecord() {
635635
assertThat(flussRecordAsPaimonRow.isNullAt(1)).isTrue();
636636
assertThat(flussRecordAsPaimonRow.getInt(2)).isEqualTo(tableBucket);
637637
assertThat(flussRecordAsPaimonRow.getLong(3)).isEqualTo(logOffset);
638-
assertThat(flussRecordAsPaimonRow.getLong(4)).isEqualTo(timeStamp);
638+
assertThat(flussRecordAsPaimonRow.getTimestamp(4, 3))
639+
.isEqualTo(Timestamp.fromEpochMillis(timeStamp));
639640
}
640641

641642
@Test

0 commit comments

Comments
 (0)