Skip to content

Commit e433e6b

Browse files
committed
[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 #2189
1 parent 99e1d5b commit e433e6b

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
@@ -575,7 +575,8 @@ void testPaimonSchemaWiderThanFlussRecord() {
575575
assertThat(flussRecordAsPaimonRow.isNullAt(1)).isTrue();
576576
assertThat(flussRecordAsPaimonRow.getInt(2)).isEqualTo(tableBucket);
577577
assertThat(flussRecordAsPaimonRow.getLong(3)).isEqualTo(logOffset);
578-
assertThat(flussRecordAsPaimonRow.getLong(4)).isEqualTo(timeStamp);
578+
assertThat(flussRecordAsPaimonRow.getTimestamp(4, 3))
579+
.isEqualTo(Timestamp.fromEpochMillis(timeStamp));
579580
}
580581

581582
@Test

0 commit comments

Comments
 (0)