JonasJ-ap commented on code in PR #6600:
URL: https://github.com/apache/iceberg/pull/6600#discussion_r1145539246


##########
docs/table-migration.md:
##########
@@ -0,0 +1,129 @@
+---
+title: "Table Migration"
+url: table-migration
+weight: 400
+menu: main
+---
+<!--
+ - Licensed to the Apache Software Foundation (ASF) under one or more
+ - contributor license agreements.  See the NOTICE file distributed with
+ - this work for additional information regarding copyright ownership.
+ - The ASF licenses this file to You under the Apache License, Version 2.0
+ - (the "License"); you may not use this file except in compliance with
+ - the License.  You may obtain a copy of the License at
+ -
+ -   http://www.apache.org/licenses/LICENSE-2.0
+ -
+ - Unless required by applicable law or agreed to in writing, software
+ - distributed under the License is distributed on an "AS IS" BASIS,
+ - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ - See the License for the specific language governing permissions and
+ - limitations under the License.
+ -->
+# Hive and Spark Table Migration
+Iceberg supports converting Hive and Spark tables to Iceberg tables through 
Spark Procedures. For specific instructions on how to use the procedures, 
please refer to the [Table Migration via Spark 
Procedures](../spark-procedures/#table-migration) page.

Review Comment:
   Currently, the [Spark 
Procedures](https://iceberg.apache.org/docs/latest/spark-procedures/#table-migration)
 contains a comprehensive explanation on migrating from Hive or Spark to 
Iceberg. So I made a pointer to that page here. I think another option would be 
to move everything to this newly added page and leave a pointer at [Spark 
Procedures](https://iceberg.apache.org/docs/latest/spark-procedures/#table-migration)
 which point to the new migraiton page.
   
   More thoughts are welcomed on the design. Please feel free to share your 
opinions on this. Thank you in advance.



##########
docs/table-migration.md:
##########
@@ -0,0 +1,129 @@
+---
+title: "Table Migration"
+url: table-migration
+weight: 400
+menu: main

Review Comment:
   Since this is a new page, I would like to put a preview of the new page 
organization here:
   ![Screen Shot 2023-03-22 at 20 08 
00](https://user-images.githubusercontent.com/55990951/227065989-003e075a-9ca6-46ec-913e-b62034cef355.png)
   
   Currently I made the `Table Migration` in the main menu.
   The preview is built via instructions in 
https://github.com/apache/iceberg-docs



-- 
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]

Reply via email to