baiyangtx commented on code in PR #4116:
URL: https://github.com/apache/amoro/pull/4116#discussion_r2922015907
##########
amoro-common/src/main/java/org/apache/amoro/TableRuntime.java:
##########
@@ -59,6 +59,13 @@ public interface TableRuntime {
*/
ServerTableIdentifier getTableIdentifier();
+ /**
+ * Load the current table instance for this runtime.
+ *
+ * <p>This method is mainly intended for in-AMS processes.
+ */
+ AmoroTable<?> loadTable();
Review Comment:
When processFactory try to create a Process for table, it need to visit the
original datalake table.
Adding a `loadTable` to tableRuntime make ProcessFactory implement simpler.
The goal of this PR is to shift the extension of format and process from
TableRuntime to the ProcessFactory interface, so that in the future, users do
not need to extend a TableRuntime implementation, And also no longer need to
worry about how to provide the TableLoader.
--
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]
