-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[iceberg] introduce procedure for migrating iceberg table to paimon #4664
Conversation
# Conflicts: # paimon-core/src/main/java/org/apache/paimon/iceberg/metadata/IcebergDataField.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job! It is an important way for iceberg user migrate to paimon, left some comments @LsomeYeah
|
||
import static org.apache.paimon.utils.ThreadPoolUtils.createCachedThreadPool; | ||
|
||
/** migrate hive table to paimon table. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
migrate iceberg table to paimon table?
|
||
import org.apache.flink.table.annotation.ArgumentHint; | ||
import org.apache.flink.table.annotation.DataTypeHint; | ||
import org.apache.flink.table.annotation.ProcedureHint; | ||
import org.apache.flink.table.procedure.ProcedureContext; | ||
import org.slf4j.Logger; | ||
import org.slf4j.LoggerFactory; | ||
|
||
/** Migrate procedure to migrate hive table to paimon table. */ | ||
public class MigrateTableProcedure extends ProcedureBase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also implement in spark side?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will add it later.
Purpose
Linked issue: close #xxx
In pr #4639, we had introduced the feature to migrating iceberg table to paimon.
This pr added corresponding procedures and IT cases.
Tests
org.apache.paimon.hive.procedure.MigrateIcebergTableProcedureITCase
org.apache.paimon.procedure.MigrateIcebergTableProcedure23ITCase
org.apache.paimon.procedure.MigrateIcebergTableProcedure31ITCase
API and Format
Documentation