mchades commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1751374566
##
.github/workflows/spark-integration-test.yml:
##
@@ -108,3 +107,64 @@ jobs:
spark-connector/v3.5/spark/build/spark-3.5-integration-test.log
FANNG1 commented on code in PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#discussion_r1751381097
##
trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/util/GeneralDataTypeTransformer.java:
##
@@ -103,7 +104,11 @@ public Type
getTri
FANNG1 commented on code in PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#discussion_r1751387253
##
trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java:
##
@@ -20,6 +20,9 @@
package or
FANNG1 commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1751404237
##
.github/workflows/spark-integration-test.yml:
##
@@ -108,3 +107,64 @@ jobs:
spark-connector/v3.5/spark/build/spark-3.5-integration-test.log
FANNG1 commented on code in PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#discussion_r1751416440
##
core/src/main/java/org/apache/gravitino/audit/AbstractAuditLogWriter.java:
##
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
FANNG1 opened a new pull request, #4900:
URL: https://github.com/apache/gravitino/pull/4900
### What changes were proposed in this pull request?
(Please outline the changes and how this PR fixes the issue.)
### Why are the changes needed?
(Please clarify why the c
danhuawang commented on PR #56:
URL:
https://github.com/apache/gravitino-playground/pull/56#issuecomment-2340330075
> > Can you help add some steps in README.md like the following examples?
Then these commands are align with above Docker Container CLI , so user can use
Trino CLI or spark s
xunliu commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1751803934
##
.github/workflows/access-control-integration-test.yml:
##
@@ -83,7 +87,8 @@ jobs:
- name: Authorization Integration Test (JDK${{ matrix.java-version
}}-${{
lw-yang commented on PR #4901:
URL: https://github.com/apache/gravitino/pull/4901#issuecomment-2340495762
yes, i forget it, i will update it
--
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
mchades commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1751829089
##
.github/workflows/access-control-integration-test.yml:
##
@@ -83,7 +87,8 @@ jobs:
- name: Authorization Integration Test (JDK${{ matrix.java-version
}}-${{
jerqi commented on PR #4901:
URL: https://github.com/apache/gravitino/pull/4901#issuecomment-2340646761
> @jerqi I haven't found any doc about role cache
OK, we can remove it directly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
jerryshao merged PR #4899:
URL: https://github.com/apache/gravitino/pull/4899
--
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
github-actions[bot] opened a new pull request, #4904:
URL: https://github.com/apache/gravitino/pull/4904
### What changes were proposed in this pull request?
polish document for spark connector and Iceberg REST server
### Why are the changes needed?
Fix: # (issue)
###
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 f48e9ad52 [MINOR] docs: polish document for spark
jerryshao closed pull request #4904: [MINOR] docs: polish document for spark
connector and Iceberg REST server
URL: https://github.com/apache/gravitino/pull/4904
--
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
jerryshao commented on code in PR #4896:
URL: https://github.com/apache/gravitino/pull/4896#discussion_r1752404369
##
docs/iceberg-rest-service.md:
##
@@ -138,6 +135,20 @@ For other Iceberg OSS properties not managed by Gravitino
like `client.security-
Please set the `gravitin
jerryshao commented on code in PR #4744:
URL: https://github.com/apache/gravitino/pull/4744#discussion_r1752391728
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/AuthorizationConfig.java:
##
@@ -0,0 +1,88 @@
+/*
+ * Licensed to the
jerryshao commented on PR #4898:
URL: https://github.com/apache/gravitino/pull/4898#issuecomment-2341716465
@mchades can you please take another look?
--
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
jerryshao closed pull request #71: fix(init/spark): fix wrong maven repo id
URL: https://github.com/apache/gravitino-playground/pull/71
--
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 co
jerryshao commented on PR #52:
URL:
https://github.com/apache/gravitino-playground/pull/52#issuecomment-2342140464
Have you already updated this @jerqi ?
--
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 t
FANNG1 commented on issue #4887:
URL: https://github.com/apache/gravitino/issues/4887#issuecomment-2342379902
@hanwxx , could you create other issues like creating a json formater,
FileWriter, etc.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
FANNG1 commented on code in PR #4548:
URL: https://github.com/apache/gravitino/pull/4548#discussion_r1752963679
##
iceberg/iceberg-common/build.gradle.kts:
##
@@ -41,17 +48,22 @@ dependencies {
exclude("com.github.spotbugs")
exclude("com.sun.jersey")
exclude("java
danhuawang commented on PR #56:
URL:
https://github.com/apache/gravitino-playground/pull/56#issuecomment-2342412516
It looks good to me.
@jerqi Would you like to take a look at this PR? Any more comments?
--
This is an automated message from the Apache Git Service.
To respond to the me
FANNG1 commented on PR #4548:
URL: https://github.com/apache/gravitino/pull/4548#issuecomment-2342423479
> Can you please list all the jars for the Iceberg catalog and Iceberg rest
service after you excluded jars?
>
> Besides, I think #4696 is also addressed here in this PR, can you p
mchades commented on code in PR #4898:
URL: https://github.com/apache/gravitino/pull/4898#discussion_r1752986266
##
core/src/test/java/org/apache/gravitino/storage/TestEntityStorage.java:
##
@@ -151,14 +183,16 @@ private void destroy(String type) {
} catch (Exception e) {
FANNG1 commented on PR #4548:
URL: https://github.com/apache/gravitino/pull/4548#issuecomment-2342425140
```
du -sh distribution/package/iceberg-rest-server/libs/* | sort -h
4.0K
distribution/package/iceberg-rest-server/libs/listenablefuture-.0-empty-to-avoid-conflict-with-guava.ja
yuqi1129 commented on code in PR #4898:
URL: https://github.com/apache/gravitino/pull/4898#discussion_r1752997625
##
core/src/test/java/org/apache/gravitino/storage/TestEntityStorage.java:
##
@@ -151,14 +183,16 @@ private void destroy(String type) {
} catch (Exception e)
xloya commented on code in PR #4891:
URL: https://github.com/apache/gravitino/pull/4891#discussion_r1753008121
##
core/src/main/java/org/apache/gravitino/listener/api/event/GetFileLocationEvent.java:
##
@@ -0,0 +1,58 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) un
yangyuxia opened a new issue, #4905:
URL: https://github.com/apache/gravitino/issues/4905
### Describe the feature
Is there any plan to support Apache HBase? If HBase namespace and table are
supported, is it recommended to use Phoenix JDBC or HBase client Java API?
### Motivati
mchades commented on PR #4898:
URL: https://github.com/apache/gravitino/pull/4898#issuecomment-2342452876
Does this PR need to be backported to branch-0.6?
--
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
mchades closed issue #4882: [Improvement] Add some UT for JDBC storage backend
such as MySQL and PostgreSQL.
URL: https://github.com/apache/gravitino/issues/4882
--
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
mchades closed issue #4882: [Improvement] Add some UT for JDBC storage backend
such as MySQL and PostgreSQL.
URL: https://github.com/apache/gravitino/issues/4882
--
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
mchades commented on PR #4883:
URL: https://github.com/apache/gravitino/pull/4883#issuecomment-2342459194
@jerryshao Could you please review the new API definition when you have
time? Thank you.
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
xloya commented on code in PR #4891:
URL: https://github.com/apache/gravitino/pull/4891#discussion_r1753008239
##
core/src/main/java/org/apache/gravitino/listener/api/event/GetFileLocationFailureEvent.java:
##
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache Software Foundation (
yuqi1129 commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1753012656
##
.github/workflows/access-control-integration-test.yml:
##
@@ -83,7 +87,8 @@ jobs:
- name: Authorization Integration Test (JDK${{ matrix.java-version
}}-${
jerryshao commented on PR #4898:
URL: https://github.com/apache/gravitino/pull/4898#issuecomment-2342471872
We don't have PG support in 0.6.
--
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
This is an automated email from the ASF dual-hosted git repository.
mchades 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 228a1f59d [#4882] improvement(core): Add some ut
mchades merged PR #4898:
URL: https://github.com/apache/gravitino/pull/4898
--
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
mchades commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1753022176
##
.github/workflows/access-control-integration-test.yml:
##
@@ -22,21 +22,32 @@ jobs:
with:
filters: |
source_changes:
+
diqiu50 commented on code in PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#discussion_r1753039299
##
trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java:
##
@@ -20,6 +20,9 @@
package o
jerqi commented on PR #74:
URL:
https://github.com/apache/gravitino-playground/pull/74#issuecomment-2342515079
@yuqi1129 @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
mchades commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1753025963
##
.github/workflows/access-control-integration-test.yml:
##
@@ -83,7 +87,8 @@ jobs:
- name: Authorization Integration Test (JDK${{ matrix.java-version
}}-${{
mchades commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1753029446
##
.github/workflows/spark-integration-test.yml:
##
@@ -108,3 +107,64 @@ jobs:
spark-connector/v3.5/spark/build/spark-3.5-integration-test.log
jerqi commented on PR #52:
URL:
https://github.com/apache/gravitino-playground/pull/52#issuecomment-2342493614
> Have you already updated this @jerqi ?
This pull request is not needed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
diqiu50 commented on PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#issuecomment-2342500732
> Does this need to be backported to branch-0.6?
Yes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
diqiu50 commented on code in PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#discussion_r1753036266
##
trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/util/GeneralDataTypeTransformer.java:
##
@@ -103,7 +104,11 @@ public Type
getTr
diqiu50 commented on code in PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#discussion_r1753039299
##
trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java:
##
@@ -20,6 +20,9 @@
package o
jerqi opened a new pull request, #74:
URL: https://github.com/apache/gravitino-playground/pull/74
(no comment)
--
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 unsubscrib
LauraXia123 opened a new pull request, #4908:
URL: https://github.com/apache/gravitino/pull/4908
### What changes were proposed in this pull request?
fix md links issue
### Why are the changes needed?
N/A
Fix: # (issue)
### Does this PR introduce _any_ user-
yaoderek opened a new pull request, #4909:
URL: https://github.com/apache/gravitino/pull/4909
### What changes were proposed in this pull request?
Adding license and notice to iceberg-rest-server and gravitino server jars
### Why are the changes needed?
Fix: #4661 The rel
danhuawang commented on PR #56:
URL:
https://github.com/apache/gravitino-playground/pull/56#issuecomment-2342613385
@unknowntpo Thanks for your work!
I install this chart on Docker Desktop , it works. It seems Docker Desktop
(enable kubenetes) is more easy to use than minikube and it's
jerryshao merged PR #17:
URL: https://github.com/apache/gravitino-site/pull/17
--
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...@gravi
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-site.git
The following commit(s) were added to refs/heads/main by this push:
new 1e40e2b Add release notes for 0.6.0 (#20)
1e
jerryshao merged PR #20:
URL: https://github.com/apache/gravitino-site/pull/20
--
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...@gravi
FANNG1 commented on code in PR #4893:
URL: https://github.com/apache/gravitino/pull/4893#discussion_r1753131642
##
trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java:
##
@@ -20,6 +20,9 @@
package or
mchades commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1753149517
##
.github/workflows/spark-integration-test.yml:
##
@@ -108,3 +107,64 @@ jobs:
spark-connector/v3.5/spark/build/spark-3.5-integration-test.log
github-actions[bot] opened a new pull request, #4910:
URL: https://github.com/apache/gravitino/pull/4910
### What changes were proposed in this pull request?
fix md links issue
### Why are the changes needed?
N/A
Fix: # (issue)
### Does this PR introduce _an
jerryshao merged PR #4908:
URL: https://github.com/apache/gravitino/pull/4908
--
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 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 9a7ec10e2 [MINOR] docs: fix md links issue (#4908)
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.6-9a7ec10e2fa5d99972cb9a2f4a58b4f4bc008277
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 9502d72c4 [MINOR] docs: fix md links issue (#4908)
N
jerryshao closed pull request #4910: [MINOR] docs: fix md links issue
URL: https://github.com/apache/gravitino/pull/4910
--
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 unsu
jerryshao closed issue #4888: [Subtask] Add the implementations for
`getFileLocation` interface in Core
URL: https://github.com/apache/gravitino/issues/4888
--
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
jerryshao merged PR #4891:
URL: https://github.com/apache/gravitino/pull/4891
--
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
jerryshao closed issue #4888: [Subtask] Add the implementations for
`getFileLocation` interface in Core
URL: https://github.com/apache/gravitino/issues/4888
--
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
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 192e59234 [#4888] improvement(core): Add the imple
jerryshao merged PR #4910:
URL: https://github.com/apache/gravitino/pull/4910
--
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.6-9a7ec10e2fa5d99972cb9a2f4a58b4f4bc008277
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 9502d72c4 [MINOR] docs: fix md links issue (#4908)
The re
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.6 by this push:
new 2d1da6be3 [MINOR] docs: fix md links i
yuanoOo opened a new pull request, #4911:
URL: https://github.com/apache/gravitino/pull/4911
### What changes were proposed in this pull request?
- add catalog-jdbc-oceanbase module
- add full code to support non-partitioned tables
### Why are the changes needed?
yuanoOo commented on PR #4911:
URL: https://github.com/apache/gravitino/pull/4911#issuecomment-2342704507
@mchades Please take a look in your free time.
--
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
mchades commented on PR #4911:
URL: https://github.com/apache/gravitino/pull/4911#issuecomment-2342708915
Could you please submit a code skeleton without the implementations? The PR
is too large to review.
--
This is an automated message from the Apache Git Service.
To respond to the
yuanoOo commented on PR #4911:
URL: https://github.com/apache/gravitino/pull/4911#issuecomment-2342714459
> Could you please submit a code skeleton without the implementations? The
PR is too large to review.
OK, I'll try.
--
This is an automated message from the Apache Git Service.
yuqi1129 commented on code in PR #4827:
URL: https://github.com/apache/gravitino/pull/4827#discussion_r1753186586
##
.github/workflows/access-control-integration-test.yml:
##
@@ -22,21 +22,32 @@ jobs:
with:
filters: |
source_changes:
+
xloya opened a new pull request, #4912:
URL: https://github.com/apache/gravitino/pull/4912
### What changes were proposed in this pull request?
Add the implementations for getFileLocation interface in Catalog Hadoop.
### Why are the changes needed?
Fix: #4889
###
diqiu50 opened a new issue, #4913:
URL: https://github.com/apache/gravitino/issues/4913
### Version
main branch
### Describe what's wrong
The bucket expression like `bucket(x, 3)`, call the function arguments in
the BucketTransform throw an ArrayStoreException
##
75 matches
Mail list logo
