[I] [Improvement] Run integration test in rootless and non privileged mode [gravitino]

2025-02-18 Thread via GitHub
youngyjd opened a new issue, #6475: URL: https://github.com/apache/gravitino/issues/6475 ### What would you like to be improved? It seems that Gravitino integration tests require Docker to run in privileged mode. Is any there option we can run it in rootless and non privileged mode?

Re: [I] [Improvement] Run integration test in rootless and non privileged mode [gravitino]

2025-02-18 Thread via GitHub
jerqi commented on issue #6475: URL: https://github.com/apache/gravitino/issues/6475#issuecomment-2667423302 I will take a look at this issue. -- 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

Re: [PR] Sync a fork 11 [gravitino]

2025-02-18 Thread via GitHub
youngyjd closed pull request #6476: Sync a fork 11 URL: https://github.com/apache/gravitino/pull/6476 -- 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: co

Re: [PR] [#6459]improve(test): Use implicit wait for time-based condition [gravitino]

2025-02-18 Thread via GitHub
yuqi1129 closed pull request #6460: [#6459]improve(test): Use implicit wait for time-based condition URL: https://github.com/apache/gravitino/pull/6460 -- 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 t

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
Abyss-lord commented on PR #6461: URL: https://github.com/apache/gravitino/pull/6461#issuecomment-2667430845 @justinmclean should I and split this pr into several issues? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
Abyss-lord commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960902913 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/PlainFormat.java: ## @@ -21,71 +21,247 @@ import java.util.Arrays; import java.util.List; import j

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
Abyss-lord commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960903482 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/BaseOutputFormat.java: ## @@ -0,0 +1,99 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
Abyss-lord commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960904484 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/Constant.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
justinmclean commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960958689 ## clients/cli/src/test/java/org/apache/gravitino/cli/integration/test/TableFormatOutputIT.java: ## @@ -106,7 +106,7 @@ public void testMetalakeListCommand() {

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
justinmclean commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960952308 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandContext.java: ## @@ -32,6 +32,8 @@ public class CommandContext { private final boolean quiet;

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
justinmclean commented on PR #6461: URL: https://github.com/apache/gravitino/pull/6461#issuecomment-2667520339 Yep I think multiple PRs would be better as this one PR is trying to do too much -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
justinmclean commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960952844 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandContext.java: ## @@ -55,6 +57,7 @@ public CommandContext(CommandLine line) { ? line.get

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
justinmclean commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960954532 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListSchema.java: ## @@ -60,8 +67,10 @@ public void handle() { exitWithError(exp.getMessage(

Re: [PR] [#5746] improve(CLI): Support table format output for Audit command [gravitino]

2025-02-18 Thread via GitHub
justinmclean commented on code in PR #6461: URL: https://github.com/apache/gravitino/pull/6461#discussion_r1960954532 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListSchema.java: ## @@ -60,8 +67,10 @@ public void handle() { exitWithError(exp.getMessage(

Re: [I] [Improvement] Run integration test in rootless and non privileged mode [gravitino]

2025-02-18 Thread via GitHub
jerqi commented on issue #6475: URL: https://github.com/apache/gravitino/issues/6475#issuecomment-2667601939 > * TESTCONTAINERS_RYUK_DISABLED Let me try this environment variable `TESTCONTAINERS_RYUK_DISABLED` first. -- This is an automated message from the Apache Git Service. To re

[PR] [#6475] improvement: Disable the ryuk [gravitino]

2025-02-18 Thread via GitHub
jerqi opened a new pull request, #6480: URL: https://github.com/apache/gravitino/pull/6480 ### What changes were proposed in this pull request? Disable the ryuk ### Why are the changes needed? (Please clarify why the changes are needed. For instance, 1. If you propos

Re: [PR] [#128] Fix the failure of starting on the MacOs [gravitino-playground]

2025-02-18 Thread via GitHub
FANNG1 merged PR #130: URL: https://github.com/apache/gravitino-playground/pull/130 -- 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-playground) branch main updated: [#128] Fix the failure of starting on the MacOs (#130)

2025-02-18 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino-playground.git The following commit(s) were added to refs/heads/main by this push: new 6765e84 [#128] Fix the failure of star

Re: [I] [Bug report] Playground fails to start on Mac [gravitino-playground]

2025-02-18 Thread via GitHub
FANNG1 closed issue #128: [Bug report] Playground fails to start on Mac URL: https://github.com/apache/gravitino-playground/issues/128 -- 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 com

Re: [PR] [#6330] chore: add BSD license to missing files [gravitino]

2025-02-18 Thread via GitHub
jerryshao commented on PR #6355: URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2667472829 @justinmclean please help to review again? -- 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

Re: [PR] [#6382] feat(iceberg): Make Gravitino Iceberg REST server keep compatibility with Trino [gravitino]

2025-02-18 Thread via GitHub
FANNG1 commented on PR #6396: URL: https://github.com/apache/gravitino/pull/6396#issuecomment-2667529429 @jerryshao How about adding a flag to enable the compatibility action with Trino? cc @mchades -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [I] [Improvement] Run integration test in rootless and non privileged mode [gravitino]

2025-02-18 Thread via GitHub
xunliu commented on issue #6475: URL: https://github.com/apache/gravitino/issues/6475#issuecomment-2667550923 hi @jerqi I think maybe we can fix this problem in this way. - https://github.com/testcontainers/testcontainers-java/issues/7964#issuecomment-1859282053 - https://java.tes

[I] [Bug report] "Network request error, request method not allowed!" error occurs when registering Model Version [gravitino]

2025-02-18 Thread via GitHub
dingshun3016 opened a new issue, #6477: URL: https://github.com/apache/gravitino/issues/6477 ### Version main branch ### Describe what's wrong In the 0.8.0 release, when registering the Model Version, the error "Network request error, request method not allowed!" appears

Re: [PR] [#128] Fix the failure of starting on the MacOs [gravitino-playground]

2025-02-18 Thread via GitHub
FANNG1 commented on code in PR #130: URL: https://github.com/apache/gravitino-playground/pull/130#discussion_r1960999470 ## playground.sh: ## @@ -18,8 +18,6 @@ # under the License. # -set -e Review Comment: why remove this line? -- This is an automated message from

Re: [PR] [#6404] feat (gvfs-fuse): Add Readme document of gvfs-fuse [gravitino]

2025-02-18 Thread via GitHub
diqiu50 commented on code in PR #6405: URL: https://github.com/apache/gravitino/pull/6405#discussion_r1961004199 ## clients/filesystem-fuse/README.md: ## @@ -0,0 +1,132 @@ + + +# Gvfs-fuse + +Gvfs-fuse is the Fuse client for Gravitino fileset. It allows users to mount Gravitino

[I] [Bug report] web(UI): Version uri title mistake [gravitino]

2025-02-18 Thread via GitHub
LauraXia123 opened a new issue, #6478: URL: https://github.com/apache/gravitino/issues/6478 ### Version main branch ### Describe what's wrong The uri title of version show the 'Type' words ### Error message and/or stacktrace https://github.com/user-attachmen

Re: [PR] [#128] Fix the failure of starting on the MacOs [gravitino-playground]

2025-02-18 Thread via GitHub
FANNG1 commented on code in PR #130: URL: https://github.com/apache/gravitino-playground/pull/130#discussion_r1960999470 ## playground.sh: ## @@ -18,8 +18,6 @@ # under the License. # -set -e Review Comment: why remove this line? -- This is an automated message from

[PR] [#6478] web(ui): fix uri title of version detail [gravitino]

2025-02-18 Thread via GitHub
LauraXia123 opened a new pull request, #6479: URL: https://github.com/apache/gravitino/pull/6479 ### What changes were proposed in this pull request? fix uri title of version detail ### Why are the changes needed? N/A Fix: #6478 ### Does this PR introduce _a

Re: [I] [Bug report] "Network request error, request method not allowed!" error occurs when registering Model Version [gravitino]

2025-02-18 Thread via GitHub
LauraXia123 commented on issue #6477: URL: https://github.com/apache/gravitino/issues/6477#issuecomment-2667609132 https://github.com/user-attachments/assets/81bea324-43bb-410c-bc3b-f70a6c0b7dcd"; /> -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [I] [Bug report] "Network request error, request method not allowed!" error occurs when registering Model Version [gravitino]

2025-02-18 Thread via GitHub
dingshun3016 commented on issue #6477: URL: https://github.com/apache/gravitino/issues/6477#issuecomment-2667618369 This is what I see here. ![Image](https://github.com/user-attachments/assets/b23606c3-d995-494b-ab43-d46a153d5bc4) -- This is an automated message from the Apache Git

Re: [I] Playground fails to start with error dependency failed to start: container playground-ranger is unhealthy [gravitino-playground]

2025-02-18 Thread via GitHub
NICK-T-D commented on issue #112: URL: https://github.com/apache/gravitino-playground/issues/112#issuecomment-2667389271 I also encountered the same problem > [@tim108108](https://github.com/tim108108) It still doesn't work according to your operation -- This is an automated messa

Re: [PR] [#6404] feat (gvfs-fuse): Add Readme document of gvfs-fuse [gravitino]

2025-02-18 Thread via GitHub
diqiu50 commented on PR #6405: URL: https://github.com/apache/gravitino/pull/6405#issuecomment-2667387498 @FANNG1 Please take a review -- 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

Re: [I] Playground fails to start with error dependency failed to start: container playground-ranger is unhealthy [gravitino-playground]

2025-02-18 Thread via GitHub
jerqi commented on issue #112: URL: https://github.com/apache/gravitino-playground/issues/112#issuecomment-2667400799 > I also encountered the same problem > > > [@tim108108](https://github.com/tim108108) It still doesn't work according to your operation Could

[I] Playground fails to start with error dependency failed to start: container playground-ranger is unhealthy [gravitino-playground]

2025-02-18 Thread via GitHub
EmilyIsCoding opened a new issue, #112: URL: https://github.com/apache/gravitino-playground/issues/112 **Expected Behavior:** Following the instructions in the documentation, Gravitino Playground containers should be healthy **Current Behavior**: Starting playground with `./pla

Re: [I] Playground fails to start with error dependency failed to start: container playground-ranger is unhealthy [gravitino-playground]

2025-02-18 Thread via GitHub
NICK-T-D commented on issue #112: URL: https://github.com/apache/gravitino-playground/issues/112#issuecomment-2667399641 信件已收到  我将尽快回复 -- 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 specifi

Re: [PR] [Improvement] Enable setting of additional configuration parameters through environment variables [gravitino]

2025-02-18 Thread via GitHub
FANNG1 merged PR #6458: URL: https://github.com/apache/gravitino/pull/6458 -- 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

(gravitino) branch main updated: [Improvement] feat(iceberg): Enable setting of additional configuration parameters through environment variables (#6458)

2025-02-18 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/main by this push: new e2947e131a [Improvement] feat(iceberg): Enable set

Re: [PR] [Improvement] Enable setting of additional configuration parameters through environment variables [gravitino]

2025-02-18 Thread via GitHub
FANNG1 commented on code in PR #6458: URL: https://github.com/apache/gravitino/pull/6458#discussion_r1960832955 ## dev/docker/iceberg-rest-server/rewrite_config.py: ## @@ -21,13 +21,16 @@ env_map = { "GRAVITINO_IO_IMPL" : "io-impl", "GRAVITINO_URI" : "uri", + "GRAVITINO_

Re: [PR] [Improvement] Enable setting of additional configuration parameters through environment variables [gravitino]

2025-02-18 Thread via GitHub
FANNG1 commented on PR #6458: URL: https://github.com/apache/gravitino/pull/6458#issuecomment-2667321637 merged to main, thanks @this-user for the contribution and @tengqm for the review! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] [Improvement] Spark connector plugin add support for Hudi data sources [gravitino]

2025-02-18 Thread via GitHub
FANNG1 commented on issue #6454: URL: https://github.com/apache/gravitino/issues/6454#issuecomment-2667324900 Thanks @lifangyu for proposing this, do you like to work on this, or propose a feature request? -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] [#6076] improve(CLI): Support model pre event to Gravitino server [gravitino]

2025-02-18 Thread via GitHub
Abyss-lord commented on PR #6250: URL: https://github.com/apache/gravitino/pull/6250#issuecomment-2667441952 @FANNG1 should we add `ModelEventDispatcher` to the Context in this pr? Or wait until all the events are complete? -- This is an automated message from the Apache Git Service.

Re: [I] Playground fails to start with error dependency failed to start: container playground-ranger is unhealthy [gravitino-playground]

2025-02-18 Thread via GitHub
NICK-T-D commented on issue #112: URL: https://github.com/apache/gravitino-playground/issues/112#issuecomment-2667447729 > I also encountered the same problem > > > [@tim108108](https://github.com/tim108108) It still doesn't work according to your operation Context: Due to

Re: [PR] [#6404] feat (gvfs-fuse): Add Readme document of gvfs-fuse [gravitino]

2025-02-18 Thread via GitHub
tengqm commented on PR #6405: URL: https://github.com/apache/gravitino/pull/6405#issuecomment-2665619692 lgtm -- 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 unsubsc

Re: [PR] [Improvement] Enable setting of additional configuration parameters through environment variables [gravitino]

2025-02-18 Thread via GitHub
this-user commented on code in PR #6458: URL: https://github.com/apache/gravitino/pull/6458#discussion_r1959911849 ## dev/docker/iceberg-rest-server/rewrite_config.py: ## @@ -21,13 +21,16 @@ env_map = { "GRAVITINO_IO_IMPL" : "io-impl", "GRAVITINO_URI" : "uri", + "GRAVITI

(gravitino) branch branch-0.8 updated: [#6462] fix(filesystem-hadoop3): Fix GVFS client can't find `HDFSFileSystemProvider` problem (#6473)

2025-02-18 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a commit to branch branch-0.8 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/branch-0.8 by this push: new 51b4ba24e6 [#6462] fix(filesystem-hado

Re: [PR] [#6462] fix(filesystem-hadoop3): Fix GVFS client can't find `HDFSFileSystemProvider` problem [gravitino]

2025-02-18 Thread via GitHub
jerryshao merged PR #6473: URL: https://github.com/apache/gravitino/pull/6473 -- 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...@gravit

Re: [PR] [MINOR] docs: Remove outdated tips from `web-ui.md` [gravitino]

2025-02-18 Thread via GitHub
jerryshao commented on PR #6472: URL: https://github.com/apache/gravitino/pull/6472#issuecomment-2664990240 @LauraXia123 can you please help to review? -- 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 g

Re: [PR] [MINOR] docs: Remove outdated tips from `web-ui.md` [gravitino]

2025-02-18 Thread via GitHub
LauraXia123 commented on PR #6472: URL: https://github.com/apache/gravitino/pull/6472#issuecomment-2665070791 LGTM -- 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 unsubs

(gravitino) branch main updated: [MINOR] docs: Remove outdated tips from `web-ui.md` (#6472)

2025-02-18 Thread roryqi
This is an automated email from the ASF dual-hosted git repository. roryqi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/main by this push: new ca1c3c31f2 [MINOR] docs: Remove outdated tips fro

Re: [PR] [MINOR] docs: Remove outdated tips from `web-ui.md` [gravitino]

2025-02-18 Thread via GitHub
jerqi merged PR #6472: URL: https://github.com/apache/gravitino/pull/6472 -- 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.

(gravitino) branch cherry-pick-branch-0.8-07ae3f6c092edd3810765c61f38fe5c09431de32 deleted (was 4bc0fa6791)

2025-02-18 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch cherry-pick-branch-0.8-07ae3f6c092edd3810765c61f38fe5c09431de32 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 4bc0fa6791 [#6462] fix(filesystem-hadoop3): Fix GVFS clien

Re: [PR] [MINOR] docs: Remove outdated tips from `web-ui.md` [gravitino]

2025-02-18 Thread via GitHub
jerqi commented on PR #6472: URL: https://github.com/apache/gravitino/pull/6472#issuecomment-2665073529 Merged. Thanks @LauraXia123 @jerryshao for review! Thanks @LuciferYang for contribution. -- This is an automated message from the Apache Git Service. To respond to the message, please l