Re: [I] [Improvement] Remove all tags from an entity in the Gravitino CLI. [gravitino]

2024-11-30 Thread via GitHub
justinmclean commented on issue #5529: URL: https://github.com/apache/gravitino/issues/5529#issuecomment-2509622038 There would be some similarities, but the command would go something different. In the command, you would need to get the list of tags and then remove them. Look at the other

Re: [I] [Improvement] Remove all tags from an entity in the Gravitino CLI. [gravitino]

2024-11-30 Thread via GitHub
suvaidkhan commented on issue #5529: URL: https://github.com/apache/gravitino/issues/5529#issuecomment-2509610669 @justinmclean The changes in this PR seems to be kinda similar to what I'm supposed to do - https://github.com/apache/gravitino/pull/5509 Am I on the right track with this

Re: [I] [FEATURE]spark-connector plugin Add support for Paimon and Hudi [gravitino]

2024-11-30 Thread via GitHub
caican00 commented on issue #5664: URL: https://github.com/apache/gravitino/issues/5664#issuecomment-2509595271 > Paimon for Spark is tracked in #2404, it's planned to support basic operations in the next release. cc @caican00 yes, I am developing paimon-spark-conenctor related featur

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-30 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1864725205 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

[PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-30 Thread via GitHub
waukin opened a new pull request, #5714: URL: https://github.com/apache/gravitino/pull/5714 ### What changes were proposed in this pull request? Enhance the Gravitino CLI with formatted output. https://github.com/user-attachments/assets/9655c081-1575-44c3-bcce-49ca2b49a2

Re: [PR] [#3783] test: add test for the 'list-all-tables' hive catalog pr [gravitino]

2024-11-30 Thread via GitHub
chenyuan99 commented on code in PR #5710: URL: https://github.com/apache/gravitino/pull/5710#discussion_r1864563414 ## catalogs/catalog-hive/src/test/java/org/apache/gravitino/catalog/hive/integration/test/CatalogHiveIT.java: ## @@ -629,6 +630,9 @@ public void testListTables() {

Re: [I] [EPIC] Implement GVFS fuse tools to access Gravitino file sets in the POSIX Protocol [gravitino]

2024-11-30 Thread via GitHub
alan910127 commented on issue #5504: URL: https://github.com/apache/gravitino/issues/5504#issuecomment-2509073835 Hi, I have a few questions regarding the design. Could it be adjusted? 1. Python typically follows the `snake_case` convention for variable and function names. 2. Most CLI

[PR] [minor] fix(dev): fix ValueError when parsing config file with multiple '=' in one line [gravitino]

2024-11-30 Thread via GitHub
orenccl opened a new pull request, #5713: URL: https://github.com/apache/gravitino/pull/5713 ### What changes were proposed in this pull request? Fix potential parsing error in the `rewrite_config.parse_config_file` function. ### Why are the changes needed? If a configur

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