nooneuse commented on code in PR #66307:
URL: https://github.com/apache/doris/pull/66307#discussion_r3869840799
##########
fe/fe-connector/fe-connector-spi/src/main/java/org/apache/doris/connector/spi/event/MetastoreChangeDescriptor.java:
##########
@@ -104,16 +109,24 @@ private MetastoreChangeDescriptor(Op op, String dbName,
String tableName, String
/** A database-level change ({@code
REGISTER_/UNREGISTER_/RENAME_DATABASE}). */
public static MetastoreChangeDescriptor forDatabase(Op op, String dbName,
String dbNameAfter,
long eventId, long updateTime) {
- return new MetastoreChangeDescriptor(op, dbName, null, dbNameAfter,
null, null, updateTime, eventId);
+ return new MetastoreChangeDescriptor(
+ op, dbName, null, dbNameAfter, null, null, null, updateTime,
eventId);
}
/** A table-level change ({@code REGISTER_/UNREGISTER_/RENAME_TABLE},
{@code REFRESH_TABLE}). */
public static MetastoreChangeDescriptor forTable(Op op, String dbName,
String tableName,
String tableNameAfter, long eventId, long updateTime) {
- return new MetastoreChangeDescriptor(op, dbName, tableName, null,
tableNameAfter, null,
+ return new MetastoreChangeDescriptor(op, dbName, tableName, null,
tableNameAfter, null, null,
updateTime, eventId);
}
+ /** An in-place table refresh caused by ALTER TABLE, including columns
removed by the schema change. */
+ public static MetastoreChangeDescriptor forTableRefresh(String dbName,
String tableName,
Review Comment:
Alright. I will make version to 7.0
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
