Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-05 Thread via GitHub
mchades commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2254398919 ## core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java: ## @@ -95,4 +120,31 @@ void insertRelation( Entity.EntityType dstType, boole

[PR] [#7745] fix(fs-bundles): avoid shade jdk class [gravitino]

2025-08-05 Thread via GitHub
mchades opened a new pull request, #7940: URL: https://github.com/apache/gravitino/pull/7940 ### What changes were proposed in this pull request? remove shade "com.sun" to avoid shade jdk class in bundle jar ### Why are the changes needed? Fix: #7745 ### Does this

Re: [PR] [#7793] improvement(fileset-catalog): Remove single file check in FilesetCatalogOperations [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on PR #7794: URL: https://github.com/apache/gravitino/pull/7794#issuecomment-3155150034 > Check the code of gvfs, it has some speicial logic to handle the single file logic, you should check the code. OK, I have checked and it seems that all single-file releated log

Re: [PR] [#7758] feat(server): Add the REST interface for job system - part2 [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on code in PR #7939: URL: https://github.com/apache/gravitino/pull/7939#discussion_r2254265966 ## server/src/main/java/org/apache/gravitino/server/web/rest/JobOperations.java: ## @@ -191,7 +199,111 @@ public Response deleteJobTemplate( } } - private

Re: [PR] [#7840] feat(core): Add job status puller and orphan file cleaner [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7936: URL: https://github.com/apache/gravitino/pull/7936#discussion_r2254246942 ## core/src/main/java/org/apache/gravitino/job/JobManager.java: ## @@ -103,6 +127,32 @@ public JobManager(Config config, EntityStore entityStore, IdGenerator idGen

Re: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3155081993 > [@yuqi1129](https://github.com/yuqi1129) yah, I want to use Hadoop Catalog to make easier to read Ceph RGW as Hadoop Compatible File System. Therefore, I try to set up some conf

Re: [PR] [#7840] feat(core): Add job status puller and orphan file cleaner [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7936: URL: https://github.com/apache/gravitino/pull/7936#discussion_r2254245883 ## core/src/main/java/org/apache/gravitino/job/JobManager.java: ## @@ -470,7 +620,7 @@ static String replacePlaceholder(String inputString, Map replace retu

Re: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-05 Thread via GitHub
nqvuong1998 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3155069824 @yuqi1129 yah, I want to use Hadoop Catalog to make easier to read Ceph RGW. I try to set up some conf follow docs, but I struggle with path-style-access Ceph RGW connection. D

Re: [PR] [#7840] feat(core): Add job status puller and orphan file cleaner [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on code in PR #7936: URL: https://github.com/apache/gravitino/pull/7936#discussion_r225346 ## core/src/main/java/org/apache/gravitino/job/JobManager.java: ## @@ -103,6 +127,32 @@ public JobManager(Config config, EntityStore entityStore, IdGenerator idGene

[PR] [#7758] feat(server): Add the REST interface for job system - part2 [gravitino]

2025-08-05 Thread via GitHub
jerryshao opened a new pull request, #7939: URL: https://github.com/apache/gravitino/pull/7939 ### What changes were proposed in this pull request? Add the job related rest interface for Gravitino. ### Why are the changes needed? Fix: #7758 ### Does this PR introd

Re: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3155057934 > I deploy a Iceberg REST Catalog with Ceph RGW and I want to create external s3 fileset on my warehouse I want to determine if you are using the iceberg rest catalog that s

Re: [PR] [#7758] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-05 Thread via GitHub
jerryshao merged PR #7923: URL: https://github.com/apache/gravitino/pull/7923 -- 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] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2254151087 ## core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java: ## @@ -95,4 +120,31 @@ void insertRelation( Entity.EntityType dstType, bool

Re: [PR] [#7758] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on PR #7923: URL: https://github.com/apache/gravitino/pull/7923#issuecomment-3154915835 @yuqi1129 @mchades can you 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 th

Re: [PR] [#7758] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-05 Thread via GitHub
mchades commented on code in PR #7923: URL: https://github.com/apache/gravitino/pull/7923#discussion_r2253614086 ## common/src/main/java/org/apache/gravitino/dto/job/ShellJobTemplateDTO.java: ## @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[I] [Improvement] metalake becomes orphan when its owner is deleted [gravitino]

2025-08-05 Thread via GitHub
mchades opened a new issue, #7938: URL: https://github.com/apache/gravitino/issues/7938 ### What would you like to be improved? When the owner of a Metalake deletes themselves from the Metalake, the Metalake becomes an orphan, and after that, no one can manage the Metalake anymore. W

Re: [PR] [#7440] feat(model): Support for altering model version with multiple URIs [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7881: URL: https://github.com/apache/gravitino/pull/7881#discussion_r2253542679 ## catalogs/catalog-model/src/main/java/org/apache/gravitino/catalog/model/ModelCatalogOperations.java: ## @@ -614,6 +625,18 @@ private void doRemoveProperty(

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerqi commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253785582 ## api/src/main/java/org/apache/gravitino/stats/PartitionDropStatistics.java: ## @@ -0,0 +1,68 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253561930 ## api/src/main/java/org/apache/gravitino/stats/PartitionComparator.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#3302][Sub-Task] StarRocks catalog web UI [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 merged PR #7798: URL: https://github.com/apache/gravitino/pull/7798 -- 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...@graviti

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerqi commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253783674 ## api/src/main/java/org/apache/gravitino/stats/PartitionUpdateStatistics.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#7922] feat (server): Client-related releases jars compatible with JDK 8 [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on PR #7877: URL: https://github.com/apache/gravitino/pull/7877#issuecomment-3154171741 This configuration can be a built-in configuration, we don't have to let the user know or set it explicitly. -- This is an automated message from the Apache Git Service. To respond

Re: [I] [Subtask] Add the dispatcher logic for the job system [gravitino]

2025-08-05 Thread via GitHub
jerryshao closed issue #7757: [Subtask] Add the dispatcher logic for the job system URL: https://github.com/apache/gravitino/issues/7757 -- 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

[I] [Subtask] Add the integration test for the job system [gravitino]

2025-08-05 Thread via GitHub
jerryshao opened a new issue, #7937: URL: https://github.com/apache/gravitino/issues/7937 ### Describe the subtask This issue tracks the work of adding the integration test for the job system. ### Parent issue #7154 -- This is an automated message from the Apache Git

Re: [I] [Subtask] Add the job execution framework in Gravitino [gravitino]

2025-08-05 Thread via GitHub
jerryshao closed issue #7506: [Subtask] Add the job execution framework in Gravitino URL: https://github.com/apache/gravitino/issues/7506 -- 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: [PR] [#7440] feat(model): Support for altering model version with multiple URIs [gravitino]

2025-08-05 Thread via GitHub
xiaozcy commented on code in PR #7881: URL: https://github.com/apache/gravitino/pull/7881#discussion_r2253693810 ## catalogs/catalog-model/src/main/java/org/apache/gravitino/catalog/model/ModelCatalogOperations.java: ## @@ -614,6 +625,18 @@ private void doRemoveProperty( en

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerqi commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253692745 ## api/src/main/java/org/apache/gravitino/stats/PartitionComparator.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253582327 ## api/src/main/java/org/apache/gravitino/stats/PartitionRange.java: ## @@ -0,0 +1,87 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] [#7758] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7923: URL: https://github.com/apache/gravitino/pull/7923#discussion_r2253642029 ## common/src/main/java/org/apache/gravitino/dto/job/ShellJobTemplateDTO.java: ## @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] [#7477] feat(web): Implement FilesetView component for file browser interface [gravitino]

2025-08-05 Thread via GitHub
LauraXia123 commented on PR #7846: URL: https://github.com/apache/gravitino/pull/7846#issuecomment-3154150386 https://github.com/apache/gravitino/blob/0f956f6065630eb347b41990fae4ae5d003e7ca2/web/integration-test/src/test/java/org/apache/gravitino/integration/test/web/ui/CatalogsPageTest.jav

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253573275 ## api/src/main/java/org/apache/gravitino/stats/PartitionDropStatistics.java: ## @@ -0,0 +1,68 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253586741 ## api/src/main/java/org/apache/gravitino/stats/PartitionUpdateStatistics.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253564007 ## api/src/main/java/org/apache/gravitino/stats/PartitionComparator.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253571341 ## api/src/main/java/org/apache/gravitino/stats/PartitionComparator.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#6719][#6337][#7413] feat(iceberg): upgrade Iceberg version to 1.9 [gravitino]

2025-08-05 Thread via GitHub
FANNG1 commented on code in PR #7917: URL: https://github.com/apache/gravitino/pull/7917#discussion_r2253567842 ## gradle/libs.versions.toml: ## @@ -55,8 +55,8 @@ commons-collections3 = "3.2.2" commons-configuration1 = "1.6" commons-dbcp2 = "2.11.0" caffeine = "2.9.3" -iceber

Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-05 Thread via GitHub
mchades commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2253556397 ## core/src/main/java/org/apache/gravitino/meta/DummyEntity.java: ## @@ -0,0 +1,154 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] [#6719][#6337][#7413] feat(iceberg): upgrade Iceberg version to 1.9 [gravitino]

2025-08-05 Thread via GitHub
FANNG1 commented on code in PR #7917: URL: https://github.com/apache/gravitino/pull/7917#discussion_r2253556804 ## iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/service/TestIcebergCatalogWrapperManagerForREST.java: ## @@ -55,7 +55,7 @@ public void testVa

Re: [PR] [#7440] feat(model): Support for altering model version with multiple URIs [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7881: URL: https://github.com/apache/gravitino/pull/7881#discussion_r2253530242 ## catalogs/catalog-model/src/main/java/org/apache/gravitino/catalog/model/ModelCatalogOperations.java: ## @@ -614,6 +625,18 @@ private void doRemoveProperty(

Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2253521381 ## core/src/main/java/org/apache/gravitino/policy/PolicyManager.java: ## @@ -222,17 +229,71 @@ public boolean deletePolicy(String metalake, String policyName) {

Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2253518932 ## core/src/main/java/org/apache/gravitino/meta/DummyEntity.java: ## @@ -0,0 +1,154 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mor

Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2253513453 ## core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java: ## @@ -76,6 +77,30 @@ List listEntitiesByRelation( Type relType, NameIdentifier

Re: [PR] [#6719][#6337][#7413] feat(iceberg): upgrade Iceberg version to 1.9 [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7917: URL: https://github.com/apache/gravitino/pull/7917#discussion_r2253498836 ## iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/service/TestIcebergCatalogWrapperManagerForREST.java: ## @@ -55,7 +55,7 @@ public void tes

Re: [PR] [#6719][#6337][#7413] feat(iceberg): upgrade Iceberg version to 1.9 [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on code in PR #7917: URL: https://github.com/apache/gravitino/pull/7917#discussion_r2253495906 ## gradle/libs.versions.toml: ## @@ -55,8 +55,8 @@ commons-collections3 = "3.2.2" commons-configuration1 = "1.6" commons-dbcp2 = "2.11.0" caffeine = "2.9.3" -ice

Re: [PR] [#7758] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on PR #7923: URL: https://github.com/apache/gravitino/pull/7923#issuecomment-3154014418 I have no further comments. @mchades Would you like to look again? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] [#7840] feat(core): Add job status puller and orphan file cleaner [gravitino]

2025-08-05 Thread via GitHub
jerryshao opened a new pull request, #7936: URL: https://github.com/apache/gravitino/pull/7936 ### What changes were proposed in this pull request? Add the job status pulling thread and orphan file cleaner thread for job manager. ### Why are the changes needed? Fix:

Re: [PR] [#7758] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on PR #7923: URL: https://github.com/apache/gravitino/pull/7923#issuecomment-3153950465 > The issue number in the title and description appears to be incorrect. Fixed. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [#7793] improvement(fileset-catalog): Remove single file check in FilesetCatalogOperations [gravitino]

2025-08-05 Thread via GitHub
jerryshao commented on PR #7794: URL: https://github.com/apache/gravitino/pull/7794#issuecomment-3153935722 Check the code of gvfs, it has some speicial logic to handle the single file logic, you should check the code. -- This is an automated message from the Apache Git Service. To respon

Re: [PR] [#7793] improvement(fileset-catalog): Remove single file check in FilesetCatalogOperations [gravitino]

2025-08-05 Thread via GitHub
yuqi1129 commented on PR #7794: URL: https://github.com/apache/gravitino/pull/7794#issuecomment-3153886653 > you should also fix the gvfs code I have added code to disable creating a fileset with a single file. I have no idea what changes should be made to the GVFS and why, regarding

[I] [Bug report] bin/spark-sql java.lang.RuntimeException: Failed to load the real sparkTable: [gravitino]

2025-08-05 Thread via GitHub
dyrnq opened a new issue, #7935: URL: https://github.com/apache/gravitino/issues/7935 ### Version main branch ### Describe what's wrong 1. create two gravitino-iceberg-rest named `foo_rest_catalog` and `bar_rest_catalog` 2. create db and tables on `foo_rest_catalog`

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-04 Thread via GitHub
mchades commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253312497 ## api/src/main/java/org/apache/gravitino/stats/PartitionRange.java: ## @@ -0,0 +1,87 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mor

[I] [Subtask] add user doc for policy management [gravitino]

2025-08-04 Thread via GitHub
mchades opened a new issue, #7933: URL: https://github.com/apache/gravitino/issues/7933 ### Describe the subtask add user doc for policy management ### Parent issue #7139 -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] [#7922] feat (server): Client-related releases jars compatible with JDK 8 [gravitino]

2025-08-04 Thread via GitHub
FANNG1 commented on PR #7877: URL: https://github.com/apache/gravitino/pull/7877#issuecomment-3153716098 `clientCompatibleWithJDK8` seems complicated, others LGTM, @jerryshao do you have time to take another look? -- This is an automated message from the Apache Git Service. To respond to

[PR] [#7933] feat(doc): add user doc for policy usage [gravitino]

2025-08-04 Thread via GitHub
mchades opened a new pull request, #7934: URL: https://github.com/apache/gravitino/pull/7934 ### What changes were proposed in this pull request? add user doc for policy usage ### Why are the changes needed? Fix: #7933 ### Does this PR introduce _any_ user-facing

Re: [PR] [#7609] feat(api): Add partition statistics interfaces [gravitino]

2025-08-04 Thread via GitHub
jerqi commented on code in PR #7610: URL: https://github.com/apache/gravitino/pull/7610#discussion_r2253255208 ## api/src/main/java/org/apache/gravitino/stats/PartitionRange.java: ## @@ -0,0 +1,87 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] [#7440] feat(model): Support for altering model version with multiple URIs [gravitino]

2025-08-04 Thread via GitHub
xiaozcy commented on code in PR #7881: URL: https://github.com/apache/gravitino/pull/7881#discussion_r2253070830 ## catalogs/catalog-model/src/main/java/org/apache/gravitino/catalog/model/ModelCatalogOperations.java: ## @@ -614,6 +625,18 @@ private void doRemoveProperty( en

Re: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-04 Thread via GitHub
nqvuong1998 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3153215206 @yuqi1129 , I only want to show you that Ceph RGW AssumeRole (STS) work! Currently, I deploy a Iceberg REST Catalog with Ceph RGW and I want to create external s3 fileset on my

Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-04 Thread via GitHub
mchades commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2253056743 ## core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java: ## @@ -76,6 +77,30 @@ List listEntitiesByRelation( Type relType, NameIdentifier n

Re: [PR] [#7624] feat(policy): Add core logic implementation for policy management (part-2) [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 commented on code in PR #7796: URL: https://github.com/apache/gravitino/pull/7796#discussion_r2253014536 ## core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java: ## @@ -76,6 +77,30 @@ List listEntitiesByRelation( Type relType, NameIdentifier

Re: [PR] [#7440] feat(model): Support for altering model version with multiple URIs [gravitino]

2025-08-04 Thread via GitHub
jerryshao commented on code in PR #7881: URL: https://github.com/apache/gravitino/pull/7881#discussion_r2253018279 ## catalogs/catalog-model/src/main/java/org/apache/gravitino/catalog/model/ModelCatalogOperations.java: ## @@ -614,6 +625,18 @@ private void doRemoveProperty(

Re: [PR] [#7906] docs(trino-connector): Document the usage constraints for trino.bypass.* properties [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 merged PR #7931: URL: https://github.com/apache/gravitino/pull/7931 -- 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...@graviti

Re: [I] [Improvement] Optimize fileset catalog server side operations [gravitino]

2025-08-04 Thread via GitHub
mchades closed issue #7771: [Improvement] Optimize fileset catalog server side operations URL: https://github.com/apache/gravitino/issues/7771 -- 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 spe

Re: [PR] [#7771] improvement(fileset-catalog): Introduce the cache system for filesystem in FilesetCatalogOperations [gravitino]

2025-08-04 Thread via GitHub
mchades merged PR #7782: URL: https://github.com/apache/gravitino/pull/7782 -- 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...@gravitin

Re: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3153092606 > Yes, Ceph RGW supports Credentials Vending through STS token, I am using Ceph RGW with Iceberg REST Catalog I'm quite puzzled. What on earth do you use? Is the Iceberg

[PR] [#7906] docs(trino-connector): Document the usage constraints for trino.bypass.* properties [gravitino]

2025-08-04 Thread via GitHub
hqbhoho opened a new pull request, #7931: URL: https://github.com/apache/gravitino/pull/7931 ### What changes were proposed in this pull request? Document the usage constraints for trino.bypass.* properties ### Why are the changes needed? Fix: #7906 ### Does this PR introd

Re: [PR] [#7925] feat(oauth): Implement OIDC authentication flow for web UI with automatic token management [gravitino]

2025-08-04 Thread via GitHub
bharos commented on PR #7930: URL: https://github.com/apache/gravitino/pull/7930#issuecomment-3153064395 > @LauraXia123 Could you help me review this code? I need to still cleanup this code. Will ping on the PR again. Meanwhile feel free to take a look at the high level approach --

Re: [PR] [#7925] feat(oauth): Implement OIDC authentication flow for web UI with automatic token management [gravitino]

2025-08-04 Thread via GitHub
jerqi commented on code in PR #7930: URL: https://github.com/apache/gravitino/pull/7930#discussion_r2252953053 ## web/web/package.json: ## @@ -34,6 +34,7 @@ "lodash-es": "^4.17.21", "next": "14.2.21", "nprogress": "^0.2.0", +"oidc-client-ts": "^3.3.0", Review

Re: [PR] [#7925] feat(oauth): Implement OIDC authentication flow for web UI with automatic token management [gravitino]

2025-08-04 Thread via GitHub
jerqi commented on PR #7930: URL: https://github.com/apache/gravitino/pull/7930#issuecomment-3153049904 @LauraXia123 Could you help me review this code? -- 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

Re: [PR] [#7906] improvement(trino-connector): Increase the priority of configuration which is defined by Gravitino [gravitino]

2025-08-04 Thread via GitHub
diqiu50 merged PR #7907: URL: https://github.com/apache/gravitino/pull/7907 -- 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...@gravitin

Re: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-04 Thread via GitHub
nqvuong1998 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3153043558 Hi @yuqi1129 1. It shows `Caused by: com.amazonaws.SdkClientException: Unable to execute HTTP request: nprod-dev-spark-artifacts.lbrgw.ceph-nonprod-pdc.hcnet.vn: Name or se

Re: [I] [Improvement] ​​Increase the priority of configuration which is defined by Gravitino in Trino [gravitino]

2025-08-04 Thread via GitHub
diqiu50 closed issue #7906: [Improvement] ​​Increase the priority of configuration which is defined by Gravitino in Trino URL: https://github.com/apache/gravitino/issues/7906 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] [#7900] fix(catalogs): Fix jdbc catalogs DateTimeType column load with default value [gravitino]

2025-08-04 Thread via GitHub
mchades merged PR #7901: URL: https://github.com/apache/gravitino/pull/7901 -- 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...@gravitin

Re: [I] [Bug report] MySQL table with time column with default value can't load [gravitino]

2025-08-04 Thread via GitHub
mchades closed issue #7900: [Bug report] MySQL table with time column with default value can't load URL: https://github.com/apache/gravitino/issues/7900 -- 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: [I] [Bug report] Can not create fileset with Ceph RGW [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 commented on issue #7919: URL: https://github.com/apache/gravitino/issues/7919#issuecomment-3153012447 1. Can you provide the full stack of error messages here? 2. It seems that you are trying to use `s3-token`, does `Ceph RGW` support s3-style token, and whether the protocol co

Re: [PR] [#7926] feat(auth): Add OAuth support for JWKS URLs [gravitino]

2025-08-04 Thread via GitHub
jerqi commented on PR #7883: URL: https://github.com/apache/gravitino/pull/7883#issuecomment-3153006013 Thanks, merged. -- 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 u

Re: [PR] [#7926] feat(auth): Add OAuth support for JWKS URLs [gravitino]

2025-08-04 Thread via GitHub
jerqi merged PR #7883: URL: https://github.com/apache/gravitino/pull/7883 -- 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] [#7906] improvement(trino-connector): Increase the priority of configuration which is defined by Gravitino [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 commented on code in PR #7907: URL: https://github.com/apache/gravitino/pull/7907#discussion_r2252923944 ## docs/trino-connector/supported-catalog.md: ## @@ -137,6 +137,7 @@ call gravitino.system.create_catalog( ``` A prefix with `trino.bypass.` in the configuration

Re: [I] [Subtask] Add OAuth support for JWKS URLs [gravitino]

2025-08-04 Thread via GitHub
jerqi closed issue #7926: [Subtask] Add OAuth support for JWKS URLs URL: https://github.com/apache/gravitino/issues/7926 -- 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 uns

Re: [PR] [#7906] improvement(trino-connector): Increase the priority of configuration which is defined by Gravitino [gravitino]

2025-08-04 Thread via GitHub
hqbhoho commented on code in PR #7907: URL: https://github.com/apache/gravitino/pull/7907#discussion_r2252920817 ## docs/trino-connector/supported-catalog.md: ## @@ -137,6 +137,7 @@ call gravitino.system.create_catalog( ``` A prefix with `trino.bypass.` in the configuration

Re: [PR] [#7906] improvement(trino-connector): Increase the priority of configuration which is defined by Gravitino [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 commented on code in PR #7907: URL: https://github.com/apache/gravitino/pull/7907#discussion_r2252897902 ## docs/trino-connector/supported-catalog.md: ## @@ -137,6 +137,7 @@ call gravitino.system.create_catalog( ``` A prefix with `trino.bypass.` in the configuration

Re: [PR] [#7440] feat(model): Support for altering model version with multiple URIs [gravitino]

2025-08-04 Thread via GitHub
jerryshao commented on code in PR #7881: URL: https://github.com/apache/gravitino/pull/7881#discussion_r2252895392 ## api/src/main/java/org/apache/gravitino/model/ModelVersionChange.java: ## @@ -302,6 +335,18 @@ final class UpdateUri implements ModelVersionChange { * @para

Re: [I] [Bug report] role: catalog's fullName in securableObjects didn't update immediately after the catalog renamed [gravitino]

2025-08-04 Thread via GitHub
LindaSummer commented on issue #7797: URL: https://github.com/apache/gravitino/issues/7797#issuecomment-3152958821 Hi @xunliu , I am interested in this topic.😊 Could this issue be assigned to me? Best Regards, Edward -- This is an automated message from the Apache Git S

Re: [PR] [#7751] feat(server): Add REST API for job system (part-1) [gravitino]

2025-08-04 Thread via GitHub
jerryshao commented on code in PR #7923: URL: https://github.com/apache/gravitino/pull/7923#discussion_r2252892333 ## server/src/main/java/org/apache/gravitino/server/web/rest/JobOperations.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [#7771] improvement(fileset-catalog): Introduce the cache system for filesystem in FilesetCatalogOperations [gravitino]

2025-08-04 Thread via GitHub
yuqi1129 commented on code in PR #7782: URL: https://github.com/apache/gravitino/pull/7782#discussion_r2252891688 ## catalogs/catalog-fileset/src/main/java/org/apache/gravitino/catalog/fileset/FilesetCatalogOperations.java: ## @@ -129,8 +130,76 @@ public class FilesetCatalogOper

Re: [PR] [#6719][#6337][#7413] feat(iceberg): upgrade Iceberg version to 1.9 [gravitino]

2025-08-04 Thread via GitHub
FANNG1 commented on PR #7917: URL: https://github.com/apache/gravitino/pull/7917#issuecomment-3152953947 @jerryshao @jerqi PTAL, thx -- 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

Re: [PR] [#5199] refactor(client-python): refactor of serdes utils [gravitino]

2025-08-04 Thread via GitHub
Copilot commented on code in PR #7904: URL: https://github.com/apache/gravitino/pull/7904#discussion_r2252852269 ## clients/client-python/gravitino/utils/serdes.py: ## @@ -0,0 +1,104 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

[PR] #7925 feat(oauth): implement OIDC authentication flow with automatic token management [gravitino]

2025-08-04 Thread via GitHub
bharos opened a new pull request, #7930: URL: https://github.com/apache/gravitino/pull/7930 #7925 feat(oauth): implement OIDC authentication flow with automatic token management - Add OIDC login/logout components and event handlers - Store JWT id_token for backend validation if ac

Re: [PR] build(deps-dev): bump @next/bundle-analyzer from 14.2.24 to 14.2.30 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] commented on PR #7409: URL: https://github.com/apache/gravitino/pull/7409#issuecomment-3152216901 Superseded by #7929. -- 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: [PR] build(deps): bump next from 14.2.21 to 14.2.30 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] closed pull request #7411: build(deps): bump next from 14.2.21 to 14.2.30 in /web/web URL: https://github.com/apache/gravitino/pull/7411 -- 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

[PR] build(deps-dev): bump @next/bundle-analyzer from 14.2.24 to 14.2.31 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] opened a new pull request, #7929: URL: https://github.com/apache/gravitino/pull/7929 Bumps [@next/bundle-analyzer](https://github.com/vercel/next.js/tree/HEAD/packages/next-bundle-analyzer) from 14.2.24 to 14.2.31. Release notes Sourced from https://github.com/verc

Re: [PR] build(deps-dev): bump @next/bundle-analyzer from 14.2.24 to 14.2.30 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] closed pull request #7409: build(deps-dev): bump @next/bundle-analyzer from 14.2.24 to 14.2.30 in /web/web URL: https://github.com/apache/gravitino/pull/7409 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] build(deps): bump next from 14.2.21 to 14.2.30 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] commented on PR #7411: URL: https://github.com/apache/gravitino/pull/7411#issuecomment-3152216704 Superseded by #7928. -- 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

[PR] build(deps): bump next from 14.2.21 to 14.2.31 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] opened a new pull request, #7928: URL: https://github.com/apache/gravitino/pull/7928 Bumps [next](https://github.com/vercel/next.js) from 14.2.21 to 14.2.31. Release notes Sourced from https://github.com/vercel/next.js/releases";>next's releases. v14.2.31

Re: [PR] build(deps): bump react-hook-form from 7.54.2 to 7.60.0 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] closed pull request #7606: build(deps): bump react-hook-form from 7.54.2 to 7.60.0 in /web/web URL: https://github.com/apache/gravitino/pull/7606 -- 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

[PR] build(deps): bump react-hook-form from 7.54.2 to 7.62.0 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] opened a new pull request, #7927: URL: https://github.com/apache/gravitino/pull/7927 Bumps [react-hook-form](https://github.com/react-hook-form/react-hook-form) from 7.54.2 to 7.62.0. Release notes Sourced from https://github.com/react-hook-form/react-hook-form/rele

Re: [PR] build(deps): bump react-hook-form from 7.54.2 to 7.60.0 in /web/web [gravitino]

2025-08-04 Thread via GitHub
dependabot[bot] commented on PR #7606: URL: https://github.com/apache/gravitino/pull/7606#issuecomment-3152112838 Superseded by #7927. -- 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

[I] [Subtask] Add OIDC support for Gravitino Web UI Login [gravitino]

2025-08-04 Thread via GitHub
bharos opened a new issue, #7925: URL: https://github.com/apache/gravitino/issues/7925 ### Describe the subtask Current UI login for OAuth only supports explicitly providing client ID, secret and scope, which is 1. Not safe or feasible for user logins and 2. Doesn't work with provide

Re: [PR] [#7926] feat(auth): Add OAuth support for JWKS URLs [gravitino]

2025-08-04 Thread via GitHub
bharos commented on PR #7883: URL: https://github.com/apache/gravitino/pull/7883#issuecomment-3151740402 > Could you create some subtasks issues? We shouldn't reuse an issue. Thanks for the review! Added subtask #7926 , updated the PR description -- This is an automated message f

[I] [Subtask] Add OAuth support for JWKS URLs [gravitino]

2025-08-04 Thread via GitHub
bharos opened a new issue, #7926: URL: https://github.com/apache/gravitino/issues/7926 ### Describe the subtask Currently Gravitino server only has capability to validate OAuth tokens using defaultSignKey. But most of the OAuth providers need support of validation through JWKS URI

Re: [PR] [#7779] refactor(client-python): Use uv as the package manager for client-python [gravitino]

2025-08-04 Thread via GitHub
tsungchih commented on PR #7811: URL: https://github.com/apache/gravitino/pull/7811#issuecomment-3151387787 > > > I got another error, when I run > > > ``` > > > ./gradlew :clients:client-python:build > > > ``` > > > > > > > > > > > > > > > > > >

Re: [PR] Feature/marquez integration br 02 [gravitino]

2025-08-04 Thread via GitHub
vishnu-chalil commented on PR #7765: URL: https://github.com/apache/gravitino/pull/7765#issuecomment-3151383021 > @vishnu-chalil the code seems not updated, could you recheck it? @FANNG1 could check this again -- This is an automated message from the Apache Git Service. To respond t

  1   2   3   4   5   6   7   8   9   10   >