Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-04 Thread via GitHub
xunliu merged PR #5699: URL: https://github.com/apache/gravitino/pull/5699 -- 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: commits-unsubscr...@gravitino

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-02 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1867207977 ## docs/jdbc-mysql-catalog.md: ## @@ -170,12 +170,24 @@ Index[] indexes = new Index[] { Although MySQL itself does not support table properties, Gravitino offers

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-02 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1867207977 ## docs/jdbc-mysql-catalog.md: ## @@ -170,12 +170,24 @@ Index[] indexes = new Index[] { Although MySQL itself does not support table properties, Gravitino offers

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-02 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1867188168 ## docs/jdbc-mysql-catalog.md: ## @@ -170,12 +170,24 @@ Index[] indexes = new Index[] { Although MySQL itself does not support table properties, Gravitino offers

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-02 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1867167050 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,27 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-01 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1865268161 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,27 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-01 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1865198156 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,29 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-01 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1865192568 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,29 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-01 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1865164636 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,31 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-12-01 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1864868816 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,31 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-30 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1864185583 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,25 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-29 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1864096099 ## docs/apache-hive-catalog.md: ## @@ -140,25 +140,25 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-29 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1864093600 ## docs/apache-hive-catalog.md: ## @@ -140,15 +140,7 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1862915209 ## docs/apache-hive-catalog.md: ## @@ -140,15 +140,7 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1862915209 ## docs/apache-hive-catalog.md: ## @@ -140,15 +140,7 @@ Since 0.6.0-incubating, the data types other than listed above are mapped to Gra Table properties supply or

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1862898691 ## docs/apache-hive-catalog.md: ## @@ -160,6 +152,21 @@ Hive automatically adds and manages some reserved properties. Users aren't allow | `EXTERNAL` |

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
mchades commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1862883301 ## docs/jdbc-mysql-catalog.md: ## @@ -170,12 +170,18 @@ Index[] indexes = new Index[] { Although MySQL itself does not support table properties, Gravitino offers

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
jerryshao commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1862841467 ## docs/apache-hive-catalog.md: ## @@ -160,6 +152,21 @@ Hive automatically adds and manages some reserved properties. Users aren't allow | `EXTERNAL`

Re: [PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
orenccl commented on code in PR #5699: URL: https://github.com/apache/gravitino/pull/5699#discussion_r1861728911 ## docs/apache-hive-catalog.md: ## @@ -160,6 +152,21 @@ Hive automatically adds and manages some reserved properties. Users aren't allow | `EXTERNAL` |

[PR] [minor] improvement(docs): adjust table properties docs [gravitino]

2024-11-28 Thread via GitHub
orenccl opened a new pull request, #5699: URL: https://github.com/apache/gravitino/pull/5699 ### What changes were proposed in this pull request? Adjust table properties documentation. ### Why are the changes needed? Add missing information about certain immutable fields