Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-04 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1979413598 ## flink-connector/flink/build.gradle.kts: ## @@ -97,7 +97,7 @@ dependencies { testImplementation(libs.testcontainers.junit.jupiter) testImplementation(libs.tes

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-04 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1979416299 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/jdbc/GravitinoJdbcCatalog.java: ## @@ -0,0 +1,86 @@ +/* + * Licensed to the Apache Softwar

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-04 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1979418248 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/catalog/BaseCatalog.java: ## @@ -101,7 +101,7 @@ protected BaseCatalog( this.partitio

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-04 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1979417241 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/jdbc/GravitinoJdbcCatalog.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Softwar

Re: [PR] [#6600] Docs: restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on PR #6596: URL: https://github.com/apache/gravitino/pull/6596#issuecomment-2697508044 > @tengqm We use Github issue to track the works, so before submitting a PR, we should create a corresponding issue to describe what we should do, follow our PR title and description tem

[I] [FEATURE] add demo about model management [gravitino-playground]

2025-03-04 Thread via GitHub
zhoukangcn opened a new issue, #133: URL: https://github.com/apache/gravitino-playground/issues/133 ### Describe the feature Add a demonstration section on model management. This should include examples of how to create, update, and version models, as well as best practices for maint

[PR] [##6594] Provide gravitino chart for deploying different storage backend on K8S [gravitino]

2025-03-04 Thread via GitHub
danhuawang opened a new pull request, #6599: URL: https://github.com/apache/gravitino/pull/6599 ### What changes were proposed in this pull request? 1. Provide gravitino chart for deploying on K8S 2. Provide CI to check chart ### Why are the changes needed? Fix: #6594

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
diqiu50 commented on PR #6597: URL: https://github.com/apache/gravitino/pull/6597#issuecomment-2697451904 > Will this only skip cascading test, or it will skip all the Trino query tests? Only skip the Trino cascading query test. -- This is an automated message from the Apache Git S

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-04 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1979412957 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/jdbc/GravitinoJdbcCatalogFactory.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
mchades commented on code in PR #6597: URL: https://github.com/apache/gravitino/pull/6597#discussion_r1979384657 ## .github/workflows/trino-integration-test.yml: ## @@ -87,7 +87,6 @@ jobs: run: | ./gradlew -PskipTests -PtestMode=embedded -PjdkVersion=${{ mat

[I] [EPIC] Improvement to docs [gravitino]

2025-03-04 Thread via GitHub
tengqm opened a new issue, #6600: URL: https://github.com/apache/gravitino/issues/6600 ### Describe the proposal The current docs for Gravitino has a lot of repeated and redundant materials. Navigating through the docs is not a pleasant experience, especially for new users. We need t

Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-04 Thread via GitHub
mchades commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1979544302 ## core/src/main/java/org/apache/gravitino/metalake/MetalakeManager.java: ## @@ -331,6 +375,7 @@ public void enableMetalake(NameIdentifier ident) throws NoSuchMetala

[I] [Bug report] The CLI.sh can't find the Jar [gravitino]

2025-03-04 Thread via GitHub
Abyss-lord opened a new issue, #6603: URL: https://github.com/apache/gravitino/issues/6603 ### Version main branch ### Describe what's wrong I ran the script in the root directory and couldn't find the Jar ![Image](https://github.com/user-attachments/assets/7f984722-5

Re: [I] [Bug report] The CLI.sh can't find the Jar [gravitino]

2025-03-04 Thread via GitHub
Abyss-lord commented on issue #6603: URL: https://github.com/apache/gravitino/issues/6603#issuecomment-2699967506 @justinmclean Justin, plz check this idea when you have time. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[I] [Bug report] Model version alias should be optional [gravitino]

2025-03-04 Thread via GitHub
LauraXia123 opened a new issue, #6605: URL: https://github.com/apache/gravitino/issues/6605 ### Version main branch ### Describe what's wrong Model version alias should be optional, but now it is required ### Error message and/or stacktrace https://github.co

Re: [I] [Bug report] CI failed while running the Trino cascading query integration test. [gravitino]

2025-03-04 Thread via GitHub
jerqi closed issue #6595: [Bug report] CI failed while running the Trino cascading query integration test. URL: https://github.com/apache/gravitino/issues/6595 -- 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

(gravitino) branch main updated: [#6595] fix (trino-connector): Disable the Trino cascading query integration test (#6597)

2025-03-04 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 9ec7f2d2f8 [#6595] fix (trino-connector): Disable

Re: [I] [Bug report] CI failed while running the Trino cascading query integration test. [gravitino]

2025-03-04 Thread via GitHub
jerqi closed issue #6595: [Bug report] CI failed while running the Trino cascading query integration test. URL: https://github.com/apache/gravitino/issues/6595 -- 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

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-04 Thread via GitHub
xunliu commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1980439669 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerHiveE2EIT.java: ## @@ -196,6 +196,32 @@ public vo

Re: [I] [Improvement] IcebergRESTServer should not use Gravitino iceberg catalog with REST backend [gravitino]

2025-03-04 Thread via GitHub
FANNG1 commented on issue #5153: URL: https://github.com/apache/gravitino/issues/5153#issuecomment-277534 Hi, @lsyulong, Would you still like to work on this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

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

2025-03-04 Thread via GitHub
Abyss-lord commented on code in PR #6250: URL: https://github.com/apache/gravitino/pull/6250#discussion_r1980784037 ## core/src/main/java/org/apache/gravitino/listener/ModelEventDispatcher.java: ## @@ -0,0 +1,268 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

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

2025-03-04 Thread via GitHub
Abyss-lord commented on PR #6250: URL: https://github.com/apache/gravitino/pull/6250#issuecomment-2700037499 @FANNG1 @jerryshao I’ve finished updating the code(add alias and version to GetModelVersionPreEvent ). Please take a look at the PR again when you have time. -- This is an automat

[PR] [#6605] fix(web): fix model version alias to be optional [gravitino]

2025-03-04 Thread via GitHub
LauraXia123 opened a new pull request, #6607: URL: https://github.com/apache/gravitino/pull/6607 ### What changes were proposed in this pull request? click submit button to trigger verification of version form, and no required info for alias form item. https://github.com/user-a

[I] [flaky test] catalogs:catalog-lakehouse-iceberg:testCatalogIcebergKerberosHiveIT.testIcebergWithKerberosAndUserImpersonation() [gravitino]

2025-03-04 Thread via GitHub
zhengkezhou1 opened a new issue, #6606: URL: https://github.com/apache/gravitino/issues/6606 ### Version main branch ### Describe what's wrong flaky test: https://github.com/apache/gravitino/actions/runs/13667966296/job/38212834258?pr=6540 ![Image](https://git

Re: [PR] build(deps-dev): bump tailwindcss from 3.4.17 to 4.0.9 in /web/web [gravitino]

2025-03-04 Thread via GitHub
LauraXia123 commented on PR #6588: URL: https://github.com/apache/gravitino/pull/6588#issuecomment-2700123968 upgrade not recommended. Tailwind 4.0 optimizes the build process and may affect existing build configurations. -- This is an automated message from the Apache Git Service. To

Re: [I] [flaky test] catalogs:catalog-lakehouse-iceberg:testCatalogIcebergKerberosHiveIT.testIcebergWithKerberosAndUserImpersonation() [gravitino]

2025-03-04 Thread via GitHub
zhengkezhou1 closed issue #6606: [flaky test] catalogs:catalog-lakehouse-iceberg:testCatalogIcebergKerberosHiveIT.testIcebergWithKerberosAndUserImpersonation() URL: https://github.com/apache/gravitino/issues/6606 -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] [#6563] improvement(core) Optimize the configuration iterator in FileSystem provider [gravitino]

2025-03-04 Thread via GitHub
sunxiaojian commented on PR #6576: URL: https://github.com/apache/gravitino/pull/6576#issuecomment-2697055633 > @sunxiaojian There is something wrong with Trino-related tests currently. ok , I'm resolving it. -- This is an automated message from the Apache Git Service. To re

[PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
diqiu50 opened a new pull request, #6597: URL: https://github.com/apache/gravitino/pull/6597 ### What changes were proposed in this pull request? Disable the Trino cascading query integration test ### Why are the changes needed? Fix: #6595 ### Does this PR introduc

Re: [PR] [#6570] improvement(core): Optimize fetching entity parent id logic [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on PR #6574: URL: https://github.com/apache/gravitino/pull/6574#issuecomment-2696782619 This is nice. -- 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. T

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

2025-03-04 Thread via GitHub
tengqm commented on code in PR #6250: URL: https://github.com/apache/gravitino/pull/6250#discussion_r1978996580 ## core/src/main/java/org/apache/gravitino/listener/ModelEventDispatcher.java: ## @@ -0,0 +1,386 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] Restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on PR #6596: URL: https://github.com/apache/gravitino/pull/6596#issuecomment-2696761638 /assign @jerryshao -- 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.

Re: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on code in PR #6583: URL: https://github.com/apache/gravitino/pull/6583#discussion_r1978989070 ## .github/ISSUE_TEMPLATE/improvement.yml: ## @@ -1,23 +1,23 @@ -# See https://gh-community.github.io/issue-template-feedback/structured/ +# See https://github.com/or

Re: [PR] Restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on PR #6596: URL: https://github.com/apache/gravitino/pull/6596#issuecomment-2697128787 Thanks @tengqm for the work, I will review this PR soon. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-04 Thread via GitHub
jerryshao closed pull request #6583: [6582] fix: Broken Link in Issue Template Documentation URL: https://github.com/apache/gravitino/pull/6583 -- 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 sp

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on PR #6597: URL: https://github.com/apache/gravitino/pull/6597#issuecomment-2697148985 @yuqi1129 @mchades 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 abov

Re: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on PR #6583: URL: https://github.com/apache/gravitino/pull/6583#issuecomment-2697143237 @Brijeshthummar02 would you please update your PR description to remove all the template words? -- This is an automated message from the Apache Git Service. To respond to the messag

[I] [Bug report] CI failed while running the Trino cascading query integration test. [gravitino]

2025-03-04 Thread via GitHub
diqiu50 opened a new issue, #6595: URL: https://github.com/apache/gravitino/issues/6595 ### Version main branch ### Describe what's wrong ``` 0 00 00 0 0 0 --:--:-- --:--:-- --:--:-- 0 100 9 100 90 0 50 0

[I] [Improvement] Gravitino helm chart for deployment on k8s [gravitino]

2025-03-04 Thread via GitHub
danhuawang opened a new issue, #6594: URL: https://github.com/apache/gravitino/issues/6594 ### What would you like to be improved? 1. Gravitino helm chart support use h2 as storage backend 2. Gravitino helm chart support launch mysql as storage backend 3. Gravitino helm chart sup

[PR] Restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
tengqm opened a new pull request, #6596: URL: https://github.com/apache/gravitino/pull/6596 This PR restructures the getting-started tutorial. Changes include: - push the getting started tutorial into a separate directory; - remove duplicated instructions on install and configuratio

Re: [PR] Restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on PR #6596: URL: https://github.com/apache/gravitino/pull/6596#issuecomment-2697177719 @tengqm We use Github issue to track the works, so before submitting a PR, we should create a corresponding issue to describe what we should do, follow our PR title and description te

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on PR #6597: URL: https://github.com/apache/gravitino/pull/6597#issuecomment-2697203315 Will this only skip cascading test, or it will skip all the Trino query tests? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] [#6563] improvement(core) Optimize the configuration iterator in FileSystem provider [gravitino]

2025-03-04 Thread via GitHub
yuqi1129 commented on PR #6576: URL: https://github.com/apache/gravitino/pull/6576#issuecomment-2696568590 @sunxiaojian There is something wrong with Trino-related tests currently. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[I] [Improvement] Inconsistencies Between Python and Java APIs [gravitino]

2025-03-04 Thread via GitHub
AndreVale69 opened a new issue, #6598: URL: https://github.com/apache/gravitino/issues/6598 ### What would you like to be improved? While working on issue #6577, I noticed (thanks to @yuqi1129, PR #6581) that there are some inconsistencies between APIs written on Python and Java. I th

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-04 Thread via GitHub
FANNG1 commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1980530375 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/jdbc/GravitinoJdbcCatalogFactory.java: ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache S

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
diqiu50 commented on code in PR #6597: URL: https://github.com/apache/gravitino/pull/6597#discussion_r1980549555 ## .github/workflows/trino-integration-test.yml: ## @@ -87,7 +87,6 @@ jobs: run: | ./gradlew -PskipTests -PtestMode=embedded -PjdkVersion=${{ mat

(gravitino) branch dependabot/github_actions/docker/setup-buildx-action-3 deleted (was e2f59a3172)

2025-03-04 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/docker/setup-buildx-action-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git was e2f59a3172 build(deps): bump docker/setup-buildx-action from 2

Re: [PR] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-03-04 Thread via GitHub
zhengkezhou1 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2699668213 I tested the failing part in CI locally https://github.com/apache/gravitino/actions/runs/13622804486/job/38080243805?pr=6540#:~:text=Run%20./gradlew%20%2DPtestMode,bash%20%2De%20

Re: [PR] [#6600] Docs: restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on code in PR #6596: URL: https://github.com/apache/gravitino/pull/6596#discussion_r1980625448 ## docs/getting-started/index.md: ## @@ -0,0 +1,247 @@ +--- +title: "Getting started with Apache Gravitino" +slug: /getting-started +license: "This software is lice

(gravitino) branch main updated: build(deps): bump docker/login-action from 2 to 3 (#6586)

2025-03-04 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 0ed25153c1 build(deps): bump docker/login-action f

(gravitino) branch dependabot/github_actions/docker/login-action-3 deleted (was 4a11eb3dde)

2025-03-04 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/docker/login-action-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 4a11eb3dde build(deps): bump docker/login-action from 2 to 3 The rev

Re: [PR] build(deps): bump docker/login-action from 2 to 3 [gravitino]

2025-03-04 Thread via GitHub
jerryshao merged PR #6586: URL: https://github.com/apache/gravitino/pull/6586 -- 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] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-03-04 Thread via GitHub
yuqi1129 closed pull request #6540: [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation URL: https://github.com/apache/gravitino/pull/6540 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] [#6600] Docs: restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on code in PR #6596: URL: https://github.com/apache/gravitino/pull/6596#discussion_r1980622563 ## docs/index.md: ## @@ -34,19 +35,22 @@ and [How to use the playground](./how-to-use-the-playground.md). ## Getting started -To get started with Gravitino, see

Re: [PR] [#6600] Docs: restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on code in PR #6596: URL: https://github.com/apache/gravitino/pull/6596#discussion_r1980634267 ## docs/getting-started/index.md: ## @@ -0,0 +1,247 @@ +--- +title: "Getting started with Apache Gravitino" +slug: /getting-started +license: "This software is license

Re: [PR] [#6600] Docs: restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
tengqm commented on code in PR #6596: URL: https://github.com/apache/gravitino/pull/6596#discussion_r1980631718 ## docs/getting-started/index.md: ## @@ -0,0 +1,247 @@ +--- +title: "Getting started with Apache Gravitino" +slug: /getting-started +license: "This software is license

Re: [PR] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-03-04 Thread via GitHub
yuqi1129 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2699706033 > I tested the failing part in CI locally [apache/gravitino/actions/runs/13622804486/job/38080243805?pr=6540#:~:text=Run%20./gradlew%20%2DPtestMode,bash%20%2De%20%7B0%7D](https://github

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-04 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1980637628 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerHiveE2EIT.java: ## @@ -196,6 +196,32 @@ public voi

Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-04 Thread via GitHub
mchades commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1980648311 ## core/src/main/java/org/apache/gravitino/metalake/MetalakeManager.java: ## @@ -365,6 +412,7 @@ public void disableMetalake(NameIdentifier ident) throws NoSuchMetal

Re: [PR] [#6570] improvement(core): Optimize fetching entity parent id logic [gravitino]

2025-03-04 Thread via GitHub
yuqi1129 closed pull request #6574: [#6570] improvement(core): Optimize fetching entity parent id logic URL: https://github.com/apache/gravitino/pull/6574 -- 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

(gravitino) branch main updated: build(deps): bump docker/setup-buildx-action from 2 to 3 (#6585)

2025-03-04 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 71e76bec80 build(deps): bump docker/setup-buildx-a

Re: [PR] build(deps): bump docker/setup-buildx-action from 2 to 3 [gravitino]

2025-03-04 Thread via GitHub
jerryshao merged PR #6585: URL: https://github.com/apache/gravitino/pull/6585 -- 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] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
diqiu50 commented on code in PR #6597: URL: https://github.com/apache/gravitino/pull/6597#discussion_r1980565869 ## .github/workflows/trino-integration-test.yml: ## @@ -87,7 +87,6 @@ jobs: run: | ./gradlew -PskipTests -PtestMode=embedded -PjdkVersion=${{ mat

[I] [Improvement] Found out the bottleneck of `Configuration.get` and optimize it [gravitino]

2025-03-04 Thread via GitHub
yuqi1129 opened a new issue, #6602: URL: https://github.com/apache/gravitino/issues/6602 ### What would you like to be improved? https://github.com/user-attachments/assets/9429fc93-76f8-4dda-9e11-9ac032b94bbc"; /> When we try to call `Filesystem.get()`, most of the CPU times wi

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
diqiu50 commented on code in PR #6597: URL: https://github.com/apache/gravitino/pull/6597#discussion_r1980549555 ## .github/workflows/trino-integration-test.yml: ## @@ -87,7 +87,6 @@ jobs: run: | ./gradlew -PskipTests -PtestMode=embedded -PjdkVersion=${{ mat

Re: [PR] [#6595] fix (trino-connector): Disable the Trino cascading query integration test [gravitino]

2025-03-04 Thread via GitHub
mchades commented on code in PR #6597: URL: https://github.com/apache/gravitino/pull/6597#discussion_r1980552414 ## .github/workflows/trino-integration-test.yml: ## @@ -87,7 +87,6 @@ jobs: run: | ./gradlew -PskipTests -PtestMode=embedded -PjdkVersion=${{ mat

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-04 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1980581378 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -935,6 +980,17 @@ protected void r

[PR] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-04 Thread via GitHub
unknowntpo opened a new pull request, #6601: URL: https://github.com/apache/gravitino/pull/6601 ### What changes were proposed in this pull request? Batch retrieving securable objects for `metalake`, `catalog`, `schema`, `table`, `column`, `model`, `topic`. ### Why are

Re: [PR] [#6600] Docs: restructure the getting started docs [gravitino]

2025-03-04 Thread via GitHub
jerryshao commented on code in PR #6596: URL: https://github.com/apache/gravitino/pull/6596#discussion_r1980601388 ## docs/index.md: ## @@ -34,19 +35,22 @@ and [How to use the playground](./how-to-use-the-playground.md). ## Getting started -To get started with Gravitino, s