[#4956] fix(docs): Correct array and union and struct mapping documen…

…t for Gravitino (#5041) ### What changes were proposed in this pull request? Fix the data type mapping in some catalog doc ### Why are the changes needed? Datatype in some catalog doc should be correct Fix: #4956 Co-authored-by: lsyulong <[email protected]>
apache · Sep 29, 2024 · a0dee14 · a0dee14
1 parent 554e410
commit a0dee14
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 26 deletions.
diff --git a/docs/apache-hive-catalog.md b/docs/apache-hive-catalog.md
@@ -119,10 +119,10 @@ The following table lists the data types mapped from the Hive catalog to Graviti
 | `interval_year_month`       | `interval_year`     | 0.2.0         |
 | `interval_day_time`         | `interval_day`      | 0.2.0         |
 | `binary`                    | `binary`            | 0.2.0         |
-| `array`                     | `array`             | 0.2.0         |
+| `array`                     | `list`              | 0.2.0         |
 | `map`                       | `map`               | 0.2.0         |
 | `struct`                    | `struct`            | 0.2.0         |
-| `uniontype`                 | `uniontype`         | 0.2.0         |
+| `uniontype`                 | `union`             | 0.2.0         |
 
 :::info
 Since 0.6.0, the data types other than listed above are mapped to Gravitino **[External Type](./manage-relational-metadata-using-gravitino.md#external-type)** that represents an unresolvable data type from the Hive catalog.

diff --git a/docs/lakehouse-iceberg-catalog.md b/docs/lakehouse-iceberg-catalog.md
@@ -224,7 +224,7 @@ Apache Iceberg doesn't support Gravitino `EvenDistribution` type.
 |-----------------------------|-----------------------------|
 | `Struct`                    | `Struct`                    |
 | `Map`                       | `Map`                       |
-| `Array`                     | `Array`                     |
+| `List`                      | `Array`                     |
 | `Boolean`                   | `Boolean`                   |
 | `Integer`                   | `Integer`                   |
 | `Long`                      | `Long`                      |
@@ -237,7 +237,7 @@ Apache Iceberg doesn't support Gravitino `EvenDistribution` type.
 | `TimestampType withoutZone` | `TimestampType withoutZone` |
 | `Decimal`                   | `Decimal`                   |
 | `Fixed`                     | `Fixed`                     |
-| `BinaryType`                | `Binary`                    |
+| `Binary`                    | `Binary`                    |
 | `UUID`                      | `UUID`                      |
 
 :::info

diff --git a/docs/lakehouse-paimon-catalog.md b/docs/lakehouse-paimon-catalog.md
@@ -127,28 +127,28 @@ Paimon Table primary key constraint should not be same with partition fields, th
 
 ### Table column types
 
-| Gravitino Type                | Apache Paimon Type             |
-|-------------------------------|--------------------------------|
-| `Sturct`                      | `Row`                          |
-| `Map`                         | `Map`                          |
-| `Array`                       | `Array`                        |
-| `Boolean`                     | `Boolean`                      |
-| `Byte`                        | `TinyInt`                      |
-| `Short`                       | `SmallInt`                     |
-| `Integer`                     | `Int`                          |
-| `Long`                        | `BigInt`                       |
-| `Float`                       | `Float`                        |
-| `Double`                      | `Double`                       |
-| `Decimal`                     | `Decimal`                      |
-| `String`                      | `VarChar(Integer.MAX_VALUE)`   |
-| `VarChar`                     | `VarChar`                      |
-| `FixedChar`                   | `Char`                         |
-| `Date`                        | `Date`                         |
-| `Time`                        | `Time`                         |
-| `TimestampType withZone`      | `LocalZonedTimestamp`          |
-| `TimestampType withoutZone`   | `Timestamp`                    |
-| `Binary`                      | `Binary`                       |
-| `Fixed`                       | `VarBinary`                    |
+| Gravitino Type              | Apache Paimon Type             |
+|-----------------------------|--------------------------------|
+| `Struct`                    | `Row`                          |
+| `Map`                       | `Map`                          |
+| `List`                      | `Array`                        |
+| `Boolean`                   | `Boolean`                      |
+| `Byte`                      | `TinyInt`                      |
+| `Short`                     | `SmallInt`                     |
+| `Integer`                   | `Int`                          |
+| `Long`                      | `BigInt`                       |
+| `Float`                     | `Float`                        |
+| `Double`                    | `Double`                       |
+| `Decimal`                   | `Decimal`                      |
+| `String`                    | `VarChar(Integer.MAX_VALUE)`   |
+| `VarChar`                   | `VarChar`                      |
+| `FixedChar`                 | `Char`                         |
+| `Date`                      | `Date`                         |
+| `Time`                      | `Time`                         |
+| `TimestampType withZone`    | `LocalZonedTimestamp`          |
+| `TimestampType withoutZone` | `Timestamp`                    |
+| `Binary`                    | `Binary`                       |
+| `Fixed`                     | `VarBinary`                    |
 
 :::info
 Gravitino doesn't support Paimon `MultisetType` type.