wzhero1 commented on code in PR #6933:
URL: https://github.com/apache/paimon/pull/6933#discussion_r2680985828
##########
paimon-flink/paimon-flink-common/src/test/java/org/apache/paimon/flink/BatchFileStoreITCase.java:
##########
@@ -1053,6 +1053,108 @@ public void testBinlogTableWithProjection() {
.containsExactly(Row.of("+I", new String[] {"A"}));
}
+ @Test
+ public void testAuditLogTableWithSequenceNumberEnabled() {
+ // Create primary key table with
changelog-read.sequence-number.enabled option
+ sql(
+ "CREATE TABLE test_table_seq (a int PRIMARY KEY NOT ENFORCED,
b int, c AS a + b) "
+ + "WITH
('changelog-read.sequence-number.enabled'='true');");
Review Comment:
This new parameter cannot be applied to SQL hints, because the `rowType`
passed by the operator is determined before obtaining dynamic options.
* Before processing SQL hint parameters, the Flink job first constructs the
`AuditLogTable` to get the `rowtype` for building upstream/downstream `rowtype`
transmission information.
* Later, when reading `AuditLogTable`, it copies the `dynamicOptions`.
* At this point, the operator's `rowtype` (without `SEQUENCE_NUM` before
obtaining dynamic parameters) does not match the task's processing `rowtype`
(with `SEQUENCE_NUM`).
--
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]
