Re: [I] [FEATURE] Support table format output for Audit command [gravitino]

2025-02-14 Thread via GitHub
Abyss-lord commented on issue #5746: URL: https://github.com/apache/gravitino/issues/5746#issuecomment-2658563392 > Hi [@Abyss-lord](https://github.com/Abyss-lord), I cannot see the image you attached (https://files.slack.com/files-pri/T05DWGA9CBT-F08D9SNQVPU/image.png). I think I don't ha

Re: [I] [FEATURE] Support table format output for Audit command [gravitino]

2025-02-14 Thread via GitHub
waukin commented on issue #5746: URL: https://github.com/apache/gravitino/issues/5746#issuecomment-2658714591 Hi @Abyss-lord, That sounds like a great idea! Feel free to modify the original output structure as needed. By the way, I have a question—how did you create the diagram above? It

[PR] [#6450] fix(web): fix authentication.type issue [gravitino]

2025-02-14 Thread via GitHub
github-actions[bot] opened a new pull request, #6456: URL: https://github.com/apache/gravitino/pull/6456 ### What changes were proposed in this pull request? authentication.type of catalog property should send correct request to server ### Why are the changes needed? N/A

Re: [PR] [#6450] fix(web): fix authentication.type issue [gravitino]

2025-02-14 Thread via GitHub
jerryshao merged PR #6452: URL: https://github.com/apache/gravitino/pull/6452 -- 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] [#6450] fix(web): fix authentication.type issue [gravitino]

2025-02-14 Thread via GitHub
jerryshao closed pull request #6456: [#6450] fix(web): fix authentication.type issue URL: https://github.com/apache/gravitino/pull/6456 -- 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 c

(gravitino) branch main updated: [#6450] fix(web): fix authentication.type issue (#6452)

2025-02-14 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao 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 74435c9b83 [#6450] fix(web): fix authentication.ty

Re: [I] [Bug report] Web page can't send correct request to Gravitino server when creating Hadoop catalogs with properties `authentication.type` [gravitino]

2025-02-14 Thread via GitHub
jerryshao closed issue #6450: [Bug report] Web page can't send correct request to Gravitino server when creating Hadoop catalogs with properties `authentication.type` URL: https://github.com/apache/gravitino/issues/6450 -- This is an automated message from the Apache Git Service. To respond

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

2025-02-14 Thread via GitHub
zzzk1 commented on issue #128: URL: https://github.com/apache/gravitino-playground/issues/128#issuecomment-2659608028 Same on ubuntu 22.04, promble code is : `openPort=$(sudo lsof -i :${port} -sTCP:LISTEN)` ![Image](https://github.com/user-attachments/assets/0abd45d8-82ef-4020-9827-

[PR] [Improvement] Enable setting JDBC user and password through environment variables [gravitino]

2025-02-14 Thread via GitHub
this-user opened a new pull request, #6458: URL: https://github.com/apache/gravitino/pull/6458 ### What changes were proposed in this pull request? Add two environment variables `GRAVITINO_JDBC_USER` and `GRAVITINO_JDBC_PASSWORD` to the `rewrite_config.py`, so database user an

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

2025-02-14 Thread via GitHub
zzzk1 opened a new pull request, #6460: URL: https://github.com/apache/gravitino/pull/6460 ### What changes were proposed in this pull request? This improvement follows our [E2E testing guide](https://gravitino.apache.org/docs/0.8.0-incubating/webui#e2e-test) by utilizing the wait method

Re: [PR] [#6450] fix(web): fix authentication.type issue [gravitino]

2025-02-14 Thread via GitHub
yuqi1129 commented on PR #6452: URL: https://github.com/apache/gravitino/pull/6452#issuecomment-2658651287 I have tested that the problem has been fix, however, I found another problem. https://github.com/user-attachments/assets/c67e2257-7fb3-43a8-a5a7-55a3a84d7a22"; /> @Laura

Re: [PR] [#6450] fix(web): fix authentication.type issue [gravitino]

2025-02-14 Thread via GitHub
LauraXia123 commented on PR #6452: URL: https://github.com/apache/gravitino/pull/6452#issuecomment-2658673644 > I have tested that the problem has been fix, however, I found another problem. > > https://private-user-images.githubusercontent.com/15794564/413234006-c67e2257-7fb3-43a8-a5

Re: [PR] [#6450] fix(web): fix authentication.type issue [gravitino]

2025-02-14 Thread via GitHub
yuqi1129 commented on PR #6452: URL: https://github.com/apache/gravitino/pull/6452#issuecomment-2658676811 > > I have tested that the problem has been fix, however, I found another problem. > > https://private-user-images.githubusercontent.com/15794564/413234006-c67e2257-7fb3-43a8-a5a7-55

(gravitino) branch cherry-pick-branch-0.8-74435c9b83b382daf0dc4753a2af9fe7af1c89a7 created (now 036959d9f3)

2025-02-14 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch cherry-pick-branch-0.8-74435c9b83b382daf0dc4753a2af9fe7af1c89a7 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 036959d9f3 [#6450] fix(web): fix authentication.type

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

2025-02-14 Thread via GitHub
zzzk1 commented on PR #6355: URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2659058693 Thanks! btw should we include all of them in the file header? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

(gravitino) branch branch-0.8 updated: [#6450] fix(web): fix authentication.type issue (#6456)

2025-02-14 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 44d0518808 [#6450] fix(web): fix authe

Re: [PR] [#6450] fix(web): fix authentication.type issue [gravitino]

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

(gravitino) branch cherry-pick-branch-0.8-74435c9b83b382daf0dc4753a2af9fe7af1c89a7 deleted (was 036959d9f3)

2025-02-14 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-74435c9b83b382daf0dc4753a2af9fe7af1c89a7 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 036959d9f3 [#6450] fix(web): fix authentication.type issue

Re: [PR] [#6392] fix(test): SparkJdbcMysqlCatalogIT33 failed in some enviroment [gravitino]

2025-02-14 Thread via GitHub
jerryshao commented on code in PR #6432: URL: https://github.com/apache/gravitino/pull/6432#discussion_r1956015194 ## integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/MySQLContainer.java: ## @@ -119,6 +119,14 @@ public void createDatabase(Tes

Re: [I] [Bug report] Update table using trino no supported [gravitino]

2025-02-14 Thread via GitHub
gesaleh commented on issue #6403: URL: https://github.com/apache/gravitino/issues/6403#issuecomment-2659216534 according to trino document it is support to set one value https://trino.io/docs/current/connector/mysql.html#mysql-update -- This is an automated message from the Apache G

Re: [I] [Improvement] Improve the current SQL of JDBCBackend [gravitino]

2025-02-14 Thread via GitHub
yuqi1129 commented on issue #4024: URL: https://github.com/apache/gravitino/issues/4024#issuecomment-2659233617 > Hi [https://avatars.githubusercontent.com/u/850797?u=fabb5216d8d77522b02ff6ac72ede0fdbd95b7cf&v=4&size=80";>@jerryshao](https://github.com/jerryshao?rgh-link-date=2025-02-14T12%3A

Re: [I] [Improvement] Improve the current SQL of JDBCBackend [gravitino]

2025-02-14 Thread via GitHub
zzzk1 commented on issue #4024: URL: https://github.com/apache/gravitino/issues/4024#issuecomment-2659226826 Hi @jerryshao Can I work on: using JOIN to reduce the number of queries? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] [#6238] improvement(storage): Improve get role performance when roles is bound to many metadata. [gravitino]

2025-02-14 Thread via GitHub
jerqi commented on code in PR #6455: URL: https://github.com/apache/gravitino/pull/6455#discussion_r1955867635 ## core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/base/SchemaMetaBaseSQLProvider.java: ## @@ -35,6 +36,24 @@ public String listSchemaPOsByCa

Re: [I] [FEATURE] Support table format output for Audit command [gravitino]

2025-02-14 Thread via GitHub
waukin commented on issue #5746: URL: https://github.com/apache/gravitino/issues/5746#issuecomment-2658548816 Hi @Abyss-lord, I cannot see the image you attached (https://files.slack.com/files-pri/T05DWGA9CBT-F08D9SNQVPU/image.png). I think I don't have permission to view the channel at Dat

Re: [PR] [#6238] improvement(storage): Improve get role performance when roles is bound to many metadata. [gravitino]

2025-02-14 Thread via GitHub
jerqi commented on code in PR #6455: URL: https://github.com/apache/gravitino/pull/6455#discussion_r1955953007 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -394,4 +443,50 @@ private static MetadataObject.Type getType(String ty

[I] [Bug Report] Iceberg REST Server Docker Image Fails to Start [gravitino]

2025-02-14 Thread via GitHub
this-user opened a new issue, #6457: URL: https://github.com/apache/gravitino/issues/6457 ### Version main branch ### Describe what's wrong Trying to start the Iceberg REST server image published on Docker Hub results in an error, and the server fails to start. This happ