szehon-ho commented on code in PR #12115:
URL: https://github.com/apache/iceberg/pull/12115#discussion_r1957066734


##########
docs/docs/spark-procedures.md:
##########
@@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 
'my_table', snapshot_id =>
 Collect statistics of the snapshot with id `snap1` of table `my_table` for 
columns `col1` and `col2`
 ```sql
 CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id 
=> 'snap1', columns => array('col1', 'col2'));
-```
\ No newline at end of file
+```
+
+## Table Replication
+
+The `rewrite-table-path` procedure prepares an Iceberg table for moving or 
copying to another location.
+
+### `rewrite-table-path`
+
+Stages a copy of the Iceberg table's metadata files where every absolute path 
source prefix is replaced by the specified target prefix.  
+This can be the starting point to fully or incrementally move or copy an 
Iceberg table to a new location.
+
+!!! info
+    This procedure only stages rewritten metadata files and prepares a list of 
files to copy. The actual file copy is not part of this procedure.
+
+
+| Argument Name      | Required? | default                                     
   | Type   | Description                                                       
      |
+|--------------------|-----------|------------------------------------------------|--------|-------------------------------------------------------------------------|
+| `table`            | ✔️        |                                             
   | string | Name of the table                                                 
      |
+| `source_prefix`    | ✔️        |                                             
   | string | The existing prefix to be replaced                                
      |
+| `target_prefix`    | ✔️        |                                             
   | string | The replacement prefix for `source_prefix`                        
      |
+| `start_version`    |           | first metadata.json in table's metadata log 
   | string | The name or path to the chronologically first metadata.json to 
rewrite  |
+| `end_version`      |           | latest metadata.json                        
   | string | The name or path to the chronologically last metadata.json to 
rewrite   |
+| `staging_location` |           | new directory under table's metadata 
directory | string | The output location for newly modified metadata files      
             |
+
+
+#### Modes of operation:
+
+- Full Rewrite:
+
+By default, the procedure operates in full rewrite mode, rewriting all 
reachable metadata files. This includes metadata.json, manifest lists, 
manifests, and position delete files.

Review Comment:
   Nit: how about adding a sentence about the copy plan:
   
   ``` By default, the procedure operates in full rewrite mode, rewriting all 
reachable metadata files (this includes metadata.json, manifest lists, 
manifests, and position delete files), and returning all reachable files in the 
result copy plan.```



##########
docs/docs/spark-procedures.md:
##########
@@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 
'my_table', snapshot_id =>
 Collect statistics of the snapshot with id `snap1` of table `my_table` for 
columns `col1` and `col2`
 ```sql
 CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id 
=> 'snap1', columns => array('col1', 'col2'));
-```
\ No newline at end of file
+```
+
+## Table Replication
+
+The `rewrite-table-path` procedure prepares an Iceberg table for moving or 
copying to another location.

Review Comment:
   Nit: recommend remove `moving` here as well.  Also the title is 
'replication' and not 'migration'



##########
docs/docs/spark-procedures.md:
##########
@@ -981,12 +981,10 @@ The `rewrite-table-path` procedure prepares an Iceberg 
table for moving or copyi
 ### `rewrite-table-path`
 
 Stages a copy of the Iceberg table's metadata files where every absolute path 
source prefix is replaced by the specified target prefix.  
-This can be the starting point to fully or incrementally copy an Iceberg table 
under a 
-source prefix to another under the target prefix.
+This can be the starting point to fully or incrementally move or copy an 
Iceberg table to a new location.

Review Comment:
   Nit: let's remove `move` to keep it brief ?  (based on previous discussion)



##########
docs/docs/spark-procedures.md:
##########
@@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 
'my_table', snapshot_id =>
 Collect statistics of the snapshot with id `snap1` of table `my_table` for 
columns `col1` and `col2`
 ```sql
 CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id 
=> 'snap1', columns => array('col1', 'col2'));
-```
\ No newline at end of file
+```
+
+## Table Replication
+
+The `rewrite-table-path` procedure prepares an Iceberg table for moving or 
copying to another location.
+
+### `rewrite-table-path`
+
+Stages a copy of the Iceberg table's metadata files where every absolute path 
source prefix is replaced by the specified target prefix.  
+This can be the starting point to fully or incrementally move or copy an 
Iceberg table to a new location.
+
+!!! info
+    This procedure only stages rewritten metadata files and prepares a list of 
files to copy. The actual file copy is not part of this procedure.
+
+
+| Argument Name      | Required? | default                                     
   | Type   | Description                                                       
      |
+|--------------------|-----------|------------------------------------------------|--------|-------------------------------------------------------------------------|
+| `table`            | ✔️        |                                             
   | string | Name of the table                                                 
      |
+| `source_prefix`    | ✔️        |                                             
   | string | The existing prefix to be replaced                                
      |
+| `target_prefix`    | ✔️        |                                             
   | string | The replacement prefix for `source_prefix`                        
      |
+| `start_version`    |           | first metadata.json in table's metadata log 
   | string | The name or path to the chronologically first metadata.json to 
rewrite  |
+| `end_version`      |           | latest metadata.json                        
   | string | The name or path to the chronologically last metadata.json to 
rewrite   |
+| `staging_location` |           | new directory under table's metadata 
directory | string | The output location for newly modified metadata files      
             |
+
+
+#### Modes of operation:
+
+- Full Rewrite:
+
+By default, the procedure operates in full rewrite mode, rewriting all 
reachable metadata files. This includes metadata.json, manifest lists, 
manifests, and position delete files.
+
+- Incremental Rewrite:
+
+If `start_version` is provided, the procedure will only rewrite metadata files 
created between `start_version` and `end_version`. `end_version` defaults to 
the latest metadata.json of the table. 
+
+#### Output
+
+| Output Name          | Type   | Description                                  
                     |
+|----------------------|--------|-------------------------------------------------------------------|
+| `latest_version`     | string | Name of the latest metadata file rewritten 
by this procedure      |
+| `file_list_location` | string | Path to a csv file containing a mapping of 
source to target paths |
+
+##### File List Copy Plan
+The file list contains the copy plan for all files added to the table between 
`startVersion` and `endVersion`. 

Review Comment:
   NIt:  should we put a colon in the end of the sentence, to indicate that a 
list is coming up?
   
   Also does this render as a list?



##########
docs/docs/spark-procedures.md:
##########
@@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 
'my_table', snapshot_id =>
 Collect statistics of the snapshot with id `snap1` of table `my_table` for 
columns `col1` and `col2`
 ```sql
 CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id 
=> 'snap1', columns => array('col1', 'col2'));
-```
\ No newline at end of file
+```
+
+## Table Replication
+
+The `rewrite-table-path` procedure prepares an Iceberg table for moving or 
copying to another location.
+
+### `rewrite-table-path`
+
+Stages a copy of the Iceberg table's metadata files where every absolute path 
source prefix is replaced by the specified target prefix.  
+This can be the starting point to fully or incrementally move or copy an 
Iceberg table to a new location.
+
+!!! info
+    This procedure only stages rewritten metadata files and prepares a list of 
files to copy. The actual file copy is not part of this procedure.
+
+
+| Argument Name      | Required? | default                                     
   | Type   | Description                                                       
      |
+|--------------------|-----------|------------------------------------------------|--------|-------------------------------------------------------------------------|
+| `table`            | ✔️        |                                             
   | string | Name of the table                                                 
      |
+| `source_prefix`    | ✔️        |                                             
   | string | The existing prefix to be replaced                                
      |
+| `target_prefix`    | ✔️        |                                             
   | string | The replacement prefix for `source_prefix`                        
      |
+| `start_version`    |           | first metadata.json in table's metadata log 
   | string | The name or path to the chronologically first metadata.json to 
rewrite  |
+| `end_version`      |           | latest metadata.json                        
   | string | The name or path to the chronologically last metadata.json to 
rewrite   |
+| `staging_location` |           | new directory under table's metadata 
directory | string | The output location for newly modified metadata files      
             |
+
+
+#### Modes of operation:
+
+- Full Rewrite:
+
+By default, the procedure operates in full rewrite mode, rewriting all 
reachable metadata files. This includes metadata.json, manifest lists, 
manifests, and position delete files.
+
+- Incremental Rewrite:
+
+If `start_version` is provided, the procedure will only rewrite metadata files 
created between `start_version` and `end_version`. `end_version` defaults to 
the latest metadata.json of the table. 

Review Comment:
   Hm i just noticed you only mention 'end_version' default, and not 
'start_version'.
   
   How about:  
   ```
   The 'start_version' and/or 'end_version' arguments may be provided to limit 
the scope to prepare an incremental copy.  Only metadata files added between 
'start_version' and 'end_version' will be rewritten, and only files added in 
this range are returned in the copy plan.
   ```



##########
docs/docs/spark-procedures.md:
##########
@@ -972,4 +972,98 @@ CALL catalog_name.system.compute_table_stats(table => 
'my_table', snapshot_id =>
 Collect statistics of the snapshot with id `snap1` of table `my_table` for 
columns `col1` and `col2`
 ```sql
 CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id 
=> 'snap1', columns => array('col1', 'col2'));
-```
\ No newline at end of file
+```
+
+## Table Replication
+
+The `rewrite-table-path` procedure prepares an Iceberg table for moving or 
copying to another location.
+
+### `rewrite-table-path`
+
+Stages a copy of the Iceberg table's metadata files where every absolute path 
source prefix is replaced by the specified target prefix.  
+This can be the starting point to fully or incrementally move or copy an 
Iceberg table to a new location.
+
+!!! info
+    This procedure only stages rewritten metadata files and prepares a list of 
files to copy. The actual file copy is not part of this procedure.
+
+
+| Argument Name      | Required? | default                                     
   | Type   | Description                                                       
      |
+|--------------------|-----------|------------------------------------------------|--------|-------------------------------------------------------------------------|
+| `table`            | ✔️        |                                             
   | string | Name of the table                                                 
      |
+| `source_prefix`    | ✔️        |                                             
   | string | The existing prefix to be replaced                                
      |
+| `target_prefix`    | ✔️        |                                             
   | string | The replacement prefix for `source_prefix`                        
      |
+| `start_version`    |           | first metadata.json in table's metadata log 
   | string | The name or path to the chronologically first metadata.json to 
rewrite  |
+| `end_version`      |           | latest metadata.json                        
   | string | The name or path to the chronologically last metadata.json to 
rewrite   |
+| `staging_location` |           | new directory under table's metadata 
directory | string | The output location for newly modified metadata files      
             |
+
+
+#### Modes of operation:
+
+- Full Rewrite:
+
+By default, the procedure operates in full rewrite mode, rewriting all 
reachable metadata files. This includes metadata.json, manifest lists, 
manifests, and position delete files.
+
+- Incremental Rewrite:
+
+If `start_version` is provided, the procedure will only rewrite metadata files 
created between `start_version` and `end_version`. `end_version` defaults to 
the latest metadata.json of the table. 
+
+#### Output
+
+| Output Name          | Type   | Description                                  
                     |
+|----------------------|--------|-------------------------------------------------------------------|
+| `latest_version`     | string | Name of the latest metadata file rewritten 
by this procedure      |
+| `file_list_location` | string | Path to a csv file containing a mapping of 
source to target paths |
+
+##### File List Copy Plan
+The file list contains the copy plan for all files added to the table between 
`startVersion` and `endVersion`. 
+For each file, it specifies
+
+- Source Path:
+The original file path in the table, or the staging location if the file has 
been rewritten.
+
+- Target Path:
+The path with the replacement prefix.
+
+The following example shows a copy plan for three files:
+
+```csv
+sourcepath/datafile1.parquet,targetpath/datafile1.parquet
+sourcepath/datafile2.parquet,targetpath/datafile2.parquet
+stagingpath/manifest.avro,targetpath/manifest.avro
+```
+
+#### Examples
+
+Full rewrite of a table's path from source location in HDFS to a target 
location in S3 bucket of table `my_table`.
+This produces a new set of metadata using the s3a prefix in the default 
staging location under table's metadata directory.
+
+```sql
+CALL catalog_name.system.rewrite_table_path(
+    table => 'db.my_table', 
+    source_prefix => "hdfs://nn:8020/path/to/source_table",
+    target_prefix => "s3a://bucket/prefix/db.db/my_table"
+);
+```
+
+Incremental rewrite of a table's path from a source location to a target 
location between metadata versions
+`v2.metadata.json` and `v20.metadata.json`, with files written to an explicit 
staging location.
+
+```sql
+CALL catalog_name.system.rewrite_table_path(
+    table => 'db.my_table', 
+    source_prefix => "s3a://bucketOne/prefix/db.db/my_table",
+    target_prefix => "s3a://bucketTwo/prefix/db.db/my_table",
+    start_version => "v2.metadata.json",
+    end_version => "v20.metadata.json",
+    staging_location => "s3a://bucketStaging/my_table"  
+);
+```
+
+Once the rewrite is completed, third-party tools (
+eg. 
[Distcp](https://hadoop.apache.org/docs/current/hadoop-distcp/DistCp.html)) can 
be used to copy the newly created
+metadata files and data files to the target location.
+
+Lastly, [register_table](#register_table) procedure can be used to register 
the copied table in the target location with a catalog.

Review Comment:
   Nit: put `the` before `register_table procedure`



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to