FANNG1 commented on code in PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#discussion_r1945889037
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java:
##
@@ -215,6 +216,11 @@ public LoadViewResponse updateView(
diqiu50 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1945887279
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
FANNG1 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1945884458
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
FANNG1 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1945882932
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
jerryshao commented on code in PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#discussion_r1945862159
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java:
##
@@ -215,6 +216,11 @@ public LoadViewResponse updateVi
danhuawang commented on issue #6406:
URL: https://github.com/apache/gravitino/issues/6406#issuecomment-2641744697
@jerqi @xunliu Can you help check 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 ab
danhuawang opened a new issue, #6406:
URL: https://github.com/apache/gravitino/issues/6406
### Version
main branch
### Describe what's wrong
Cron Integration Test execute failed:
https://github.com/apache/gravitino/actions/runs/13164696252/job/36741893632
Error
diqiu50 opened a new pull request, #6405:
URL: https://github.com/apache/gravitino/pull/6405
### What changes were proposed in this pull request?
Add Readme document of gvfs-fuse
### Why are the changes needed?
Fix: #6404
### Does this PR introduce _any_ user-
FANNG1 commented on PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#issuecomment-2641521645
> @FANNG1, thanks for reviewing. I just fixed it, could you help me review
it again? Thank you!
one little comment.
--
This is an automated message from the Apache Git Service.
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1945721571
##
core/src/main/java/org/apache/gravitino/listener/api/event/AlterTagPreEvent.java:
##
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
justinmclean commented on code in PR #6343:
URL: https://github.com/apache/gravitino/pull/6343#discussion_r1945581200
##
clients/cli/src/main/java/org/apache/gravitino/cli/MetalakeCommandHandler.java:
##
@@ -40,15 +39,18 @@ public class MetalakeCommandHandler extends CommandHand
gesaleh opened a new issue, #6403:
URL: https://github.com/apache/gravitino/issues/6403
### Version
main branch
### Describe what's wrong
according to the documents update fro mysql connector is supported if we
give constant values
```
trino> UPDATE catalog_mysql.
metalshanked opened a new issue, #6402:
URL: https://github.com/apache/gravitino/issues/6402
### Version
main branch
### Describe what's wrong
Hi, is there a simple way to change the default client id and secret for
gravitino example mentioned here
https://gravitino.
FANNG1 commented on code in PR #6374:
URL: https://github.com/apache/gravitino/pull/6374#discussion_r1944974354
##
iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/integration/test/IcebergRESTGCSIT.java:
##
@@ -92,10 +93,10 @@ private void copyGCSBundleJar(
FANNG1 commented on code in PR #6374:
URL: https://github.com/apache/gravitino/pull/6374#discussion_r1944975577
##
dev/docker/iceberg-rest-server/iceberg-rest-server-dependency.sh:
##
@@ -49,19 +49,19 @@ cp
${gravitino_home}/bundles/aws/build/libs/gravitino-aws-*.jar bundles/
amazingLychee commented on PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#issuecomment-2640059201
@FANNG1, thanks for reviewing. I just fixed it, could you help me review it
again? Thank you!
--
This is an automated message from the Apache Git Service.
To respond to the mess
jerryshao commented on code in PR #6374:
URL: https://github.com/apache/gravitino/pull/6374#discussion_r1944817175
##
iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/integration/test/IcebergRESTGCSIT.java:
##
@@ -92,10 +93,10 @@ private void copyGCSBundleJ
jerryshao commented on code in PR #6374:
URL: https://github.com/apache/gravitino/pull/6374#discussion_r1944815380
##
dev/docker/iceberg-rest-server/iceberg-rest-server-dependency.sh:
##
@@ -49,19 +49,19 @@ cp
${gravitino_home}/bundles/aws/build/libs/gravitino-aws-*.jar bundles
FANNG1 commented on code in PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#discussion_r1944594847
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java:
##
@@ -228,10 +234,18 @@ public void renameView(RenameTable
Abyss-lord commented on PR #6359:
URL: https://github.com/apache/gravitino/pull/6359#issuecomment-2639666848
> I agree with Justin that, anonymous user is a system reserved user, should
not be deleted, instead of reminding user. @Abyss-lord could you please update
the logic here?
@sh
FANNG1 commented on code in PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#discussion_r1944590990
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java:
##
@@ -215,6 +216,11 @@ public LoadViewResponse updateView(
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 52d250edd7 [#6375] improvement(catalog
jerryshao merged PR #6401:
URL: https://github.com/apache/gravitino/pull/6401
--
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
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.8-45349852629d00eb38eff531af1a0f2bd1dab852
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 64bb67d8df [#6375] improvment(catalog-hadoop): Remove `pro
mchades commented on PR #6400:
URL: https://github.com/apache/gravitino/pull/6400#issuecomment-2639420677
plz make the CI pass. see:
https://github.com/apache/gravitino/actions/runs/13173847662/job/36768887540?pr=6400
--
This is an automated message from the Apache Git Service.
To respond
shaofengshi commented on code in PR #6343:
URL: https://github.com/apache/gravitino/pull/6343#discussion_r1944389104
##
clients/cli/src/main/java/org/apache/gravitino/cli/MetalakeCommandHandler.java:
##
@@ -40,15 +39,18 @@ public class MetalakeCommandHandler extends CommandHandl
jerryshao commented on code in PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#discussion_r1944430131
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java:
##
@@ -228,10 +234,18 @@ public void renameView(RenameTa
jerryshao commented on code in PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#discussion_r1944425607
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java:
##
@@ -215,6 +216,11 @@ public LoadViewResponse updateVi
jerryshao closed pull request #6401: [#6375] improvment(catalog-hadoop): Remove
`protobuf-java` to avoid conflict with authorization module
URL: https://github.com/apache/gravitino/pull/6401
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
jerryshao closed issue #6375: [Improvement] protobuf-java in module
`catalog-hadoop` is useless and will conflict with a new version in
authorization module
URL: https://github.com/apache/gravitino/issues/6375
--
This is an automated message from the Apache Git Service.
To respond to the mes
jerryshao closed issue #6375: [Improvement] protobuf-java in module
`catalog-hadoop` is useless and will conflict with a new version in
authorization module
URL: https://github.com/apache/gravitino/issues/6375
--
This is an automated message from the Apache Git Service.
To respond to the mes
github-actions[bot] opened a new pull request, #6401:
URL: https://github.com/apache/gravitino/pull/6401
### What changes were proposed in this pull request?
Remove jar `protobuf-java.jar` from the distribution package to avoid
conflicts
### Why are the changes needed?
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.8-45349852629d00eb38eff531af1a0f2bd1dab852
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 64bb67d8df [#6375] improvment(catalog-hadoop): Remove
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 4534985262 [#6375] improvment(catalog-hadoop): Rem
jerryshao merged PR #6376:
URL: https://github.com/apache/gravitino/pull/6376
--
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
FANNG1 commented on issue #6392:
URL: https://github.com/apache/gravitino/issues/6392#issuecomment-2639170721
ok, seems this IT failed in some specific environment
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
FANNG1 commented on PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#issuecomment-2639160962
sorry for the delay, LGTM except minor comments, could you fix it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944308748
##
core/src/main/java/org/apache/gravitino/listener/api/event/AssociateTagsForMetadataObjectPreEvent.java:
##
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944307961
##
core/src/main/java/org/apache/gravitino/listener/api/event/AssociateTagsForMetadataObjectPreEvent.java:
##
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944308748
##
core/src/main/java/org/apache/gravitino/listener/api/event/AssociateTagsForMetadataObjectPreEvent.java:
##
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944296316
##
core/src/main/java/org/apache/gravitino/listener/api/event/TagPreEvent.java:
##
@@ -0,0 +1,35 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944295964
##
core/src/main/java/org/apache/gravitino/listener/api/event/ListTagsInfoPreEvent.java:
##
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944293015
##
core/src/main/java/org/apache/gravitino/listener/api/event/GetTagForMetadataObjectPreEvent.java:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Foundat
FANNG1 commented on code in PR #5980:
URL: https://github.com/apache/gravitino/pull/5980#discussion_r1944293241
##
core/src/main/java/org/apache/gravitino/listener/api/event/GetTagForMetadataObjectPreEvent.java:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Foundat
FANNG1 commented on code in PR #6374:
URL: https://github.com/apache/gravitino/pull/6374#discussion_r1944265030
##
catalogs/catalog-common/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergConstants.java:
##
@@ -84,4 +84,6 @@ public class IcebergConstants {
45 matches
Mail list logo
