TungYuChiang commented on issue #5548:
URL: https://github.com/apache/gravitino/issues/5548#issuecomment-2491151562
Hi @FANNG1 ,
I’m sorry for the delay. I’ve been quite busy this week, but I will make
sure to complete it by this weekend. Thank you for your understanding!
--
This i
FANNG1 commented on PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#issuecomment-2491157851
suggest adding basic check for code style and common errors in ci
```
cargo fmt --all -- --check
cargo clippy --all-targets --all-features --workspace -- -D warn
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851867253
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -142,7 +144,7 @@ void addPolicyItem(RangerPol
FANNG1 commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851975498
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contrib
jerryshao commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851981721
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more cont
xloya commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851925369
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851936397
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -480,8 +505,8 @@ protected void updatePolicyO
FANNG1 commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851913594
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contrib
jerryshao merged PR #5580:
URL: https://github.com/apache/gravitino/pull/5580
--
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 #5556: [Improvement] integrate audit log framework for
Iceberg REST server
URL: https://github.com/apache/gravitino/issues/5556
--
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
This is an automated email from the ASF dual-hosted git repository.
liuxun 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 af259c3a6 [#4362] feat(web): Add GitHub stars and
jerryshao commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851957061
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more cont
FANNG1 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1852034056
##
clients/filesystem-fuse/check_rust_env.sh:
##
@@ -0,0 +1,43 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
diqiu50 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1853187380
##
clients/filesystem-fuse/check_rust_env.sh:
##
@@ -0,0 +1,45 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
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 67d316d04 [#5637] Add CLI unit tests to check if t
jerryshao closed issue #5637: [Improvement] Add more CLI command unit tests
URL: https://github.com/apache/gravitino/issues/5637
--
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.
jerryshao merged PR #5638:
URL: https://github.com/apache/gravitino/pull/5638
--
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 commented on code in PR #5638:
URL: https://github.com/apache/gravitino/pull/5638#discussion_r1853390470
##
clients/cli/src/main/java/org/apache/gravitino/cli/TestableCommandLine.java:
##
@@ -0,0 +1,423 @@
+/* Methods used for testing - basically a seam where we can se
jerryshao closed issue #5642: [Improvement] Remove TestFetchFileUtils test to
reduce test failures
URL: https://github.com/apache/gravitino/issues/5642
--
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 t
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 6641141c0 [#5642] test: disable TestFetchFileUtils
jerryshao closed issue #5642: [Improvement] Remove TestFetchFileUtils test to
reduce test failures
URL: https://github.com/apache/gravitino/issues/5642
--
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 t
jerryshao merged PR #5644:
URL: https://github.com/apache/gravitino/pull/5644
--
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 #5637: [Improvement] Add more CLI command unit tests
URL: https://github.com/apache/gravitino/issues/5637
--
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.
jerryshao commented on PR #5580:
URL: https://github.com/apache/gravitino/pull/5580#issuecomment-2490401329
@FANNG1 Please fix the conflicts.
--
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 s
justinmclean commented on issue #5279:
URL: https://github.com/apache/gravitino/issues/5279#issuecomment-2490405592
Thanks for offering to help. If you need any assistance or have any
questions, just ask.
--
This is an automated message from the Apache Git Service.
To respond to the messa
orenccl commented on PR #5636:
URL: https://github.com/apache/gravitino/pull/5636#issuecomment-2490340313
@LauraXia123
Thank you for reviewing this. Please approve it at your convenience. 🙇♂️
--
This is an automated message from the Apache Git Service.
To respond to the message,
jerryshao commented on PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#issuecomment-2490276898
@FANNG1 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 above to go to
jerryshao commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1851506565
##
clients/filesystem-fuse/check_rust_env.sh:
##
@@ -0,0 +1,45 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contribut
LauraXia123 commented on PR #5567:
URL: https://github.com/apache/gravitino/pull/5567#issuecomment-2490465749
Thank you very much for your contribution. LGTM
--
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
tengqm commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851716384
##
core/src/main/java/org/apache/gravitino/listener/api/event/PartitionPreEvent.java:
##
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
jerryshao commented on code in PR #5538:
URL: https://github.com/apache/gravitino/pull/5538#discussion_r1851690479
##
clients/client-python/tests/integration/test_gvfs_with_abs.py:
##
@@ -0,0 +1,373 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more cont
FANNG1 commented on code in PR #5580:
URL: https://github.com/apache/gravitino/pull/5580#discussion_r1851722175
##
core/src/main/java/org/apache/gravitino/audit/v2/CompatibilityUtils.java:
##
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
FANNG1 commented on code in PR #5580:
URL: https://github.com/apache/gravitino/pull/5580#discussion_r1851724509
##
core/src/main/java/org/apache/gravitino/listener/api/event/OperationStatus.java:
##
@@ -0,0 +1,28 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
jerryshao commented on code in PR #5580:
URL: https://github.com/apache/gravitino/pull/5580#discussion_r1851731973
##
core/src/main/java/org/apache/gravitino/audit/v2/CompatibilityUtils.java:
##
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
FANNG1 commented on issue #5625:
URL: https://github.com/apache/gravitino/issues/5625#issuecomment-2490496585
> > This feature is scheduled to be included in version 0.8, which is set to
be released by the end of December. Would this timeline work for you?
>
> OK，no problem , a month
danhuawang commented on code in PR #5632:
URL: https://github.com/apache/gravitino/pull/5632#discussion_r1851403170
##
trino-connector/integration-test/src/test/java/org/apache/gravitino/trino/connector/integration/test/TrinoQueryIT.java:
##
@@ -131,6 +132,23 @@ public void runO
xunliu merged PR #5567:
URL: https://github.com/apache/gravitino/pull/5567
--
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
xunliu closed issue #4362: [FEATURE] Add github logo, stars number and forks
number for UI
URL: https://github.com/apache/gravitino/issues/4362
--
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
jerryshao commented on code in PR #5580:
URL: https://github.com/apache/gravitino/pull/5580#discussion_r1851634885
##
core/src/main/java/org/apache/gravitino/audit/v2/CompatibilityUtils.java:
##
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under on
FANNG1 commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851596611
##
core/src/main/java/org/apache/gravitino/listener/api/event/PartitionPreEvent.java:
##
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
jerryshao commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1851503453
##
clients/filesystem-fuse/Cargo.toml:
##
@@ -0,0 +1,30 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.
jerryshao commented on PR #5558:
URL: https://github.com/apache/gravitino/pull/5558#issuecomment-2490343660
Do you need to update the PR based on #5630 ?
--
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
sunxiaojian commented on issue #5625:
URL: https://github.com/apache/gravitino/issues/5625#issuecomment-2490441876
> This feature is scheduled to be included in version 0.8, which is set to
be released by the end of December. Would this timeline work for you?
OK，no problem , a month s
tengqm commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1851496401
##
clients/gvfs-fuse/check_rust_env.sh:
##
@@ -0,0 +1,48 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor licens
jerryshao commented on PR #5638:
URL: https://github.com/apache/gravitino/pull/5638#issuecomment-2490270382
Yeah, I'm fine with the current direction, please add more tests to cover
all the options/flags as possible as we can.
--
This is an automated message from the Apache Git Service.
T
FANNG1 commented on issue #5625:
URL: https://github.com/apache/gravitino/issues/5625#issuecomment-2490351295
> @FANNG1 Can assign it to me
This feature is scheduled to be included in version 0.8, which is set to be
released by the end of December. Would this timeline work for you?
xunliu merged PR #5636:
URL: https://github.com/apache/gravitino/pull/5636
--
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
This is an automated email from the ASF dual-hosted git repository.
liuxun 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 85e9918ac [#5454] feat(web): different data table
FANNG1 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1852023763
##
clients/filesystem-fuse/Cargo.toml:
##
@@ -0,0 +1,34 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. Se
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851866974
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -252,8 +269,16 @@ protected boolean checkRang
FANNG1 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1852082386
##
clients/filesystem-fuse/Cargo.toml:
##
@@ -0,0 +1,34 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. Se
FANNG1 commented on code in PR #5580:
URL: https://github.com/apache/gravitino/pull/5580#discussion_r1851843599
##
core/src/main/java/org/apache/gravitino/audit/v2/CompatibilityUtils.java:
##
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
pithecuse527 commented on issue #5633:
URL: https://github.com/apache/gravitino/issues/5633#issuecomment-2490859856
I have a few questions as I go through the code.
1. Is this issue related to https://github.com/apache/gravitino/issues/2290?
(Is this why we need to remove the RocksD
FANNG1 commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851910770
##
core/src/main/java/org/apache/gravitino/listener/api/event/AddPartitionPreEvent.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
jerryshao commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851910781
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more cont
jerryshao commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851914627
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more cont
jerryshao commented on PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#issuecomment-2490812438
@xunliu @shaofengshi @FANNG1 @yuqi1129 @mchades @diqiu50 can you please also
help to review?
--
This is an automated message from the Apache Git Service.
To respond to the message,
FANNG1 commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851848929
##
core/src/main/java/org/apache/gravitino/listener/api/event/AddPartitionPreEvent.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
FANNG1 commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851848929
##
core/src/main/java/org/apache/gravitino/listener/api/event/AddPartitionPreEvent.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851849765
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -385,7 +411,6 @@ protected RangerPolicy
crea
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851849765
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -385,7 +411,6 @@ protected RangerPolicy
crea
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851960937
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -154,8 +156,8 @@ void addPolicyItem(RangerPol
diqiu50 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1851989715
##
clients/filesystem-fuse/check_rust_env.sh:
##
@@ -0,0 +1,45 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
tengqm commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851900160
##
core/src/main/java/org/apache/gravitino/listener/api/event/AddPartitionPreEvent.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
xloya commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851682137
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
jerryshao commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851984387
##
api/src/main/java/org/apache/gravitino/model/ModelCatalog.java:
##
@@ -0,0 +1,118 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or mo
tengqm commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1852073811
##
clients/filesystem-fuse/check_rust_env.sh:
##
@@ -0,0 +1,45 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
jerqi commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1848157026
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,171 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
tengqm commented on code in PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#discussion_r1851434652
##
core/src/main/java/org/apache/gravitino/listener/api/event/AddPartitionPreEvent.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
jerryshao commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1852029740
##
clients/filesystem-fuse/Cargo.toml:
##
@@ -0,0 +1,34 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.
waukin commented on code in PR #5606:
URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285
##
clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java:
##
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
waukin commented on code in PR #5606:
URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285
##
clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java:
##
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
LauraXia123 commented on PR #5635:
URL: https://github.com/apache/gravitino/pull/5635#issuecomment-2490653669
LGTM
--
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 unsubs
theoryxu commented on code in PR #5629:
URL: https://github.com/apache/gravitino/pull/5629#discussion_r1851936397
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -480,8 +505,8 @@ protected void updatePolicyO
FANNG1 commented on issue #5548:
URL: https://github.com/apache/gravitino/issues/5548#issuecomment-2491193539
@TungYuChiang this is just a notice ping, please take your time
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
FANNG1 commented on issue #5548:
URL: https://github.com/apache/gravitino/issues/5548#issuecomment-2491041241
@TungYuChiang is there any process?
--
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 t
jerryshao commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851957061
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more cont
FANNG1 commented on PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#issuecomment-2491037993
@LiuQhahah as #5580 is merged first, could you rebase the code and add more
work:
1. add operationType method to all specific partition event.
```
@Override
public Operat
xloya commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1852002594
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
xloya commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851996831
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
xloya commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851993208
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
FANNG1 commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851996670
##
api/src/main/java/org/apache/gravitino/model/ModelCatalog.java:
##
@@ -0,0 +1,118 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more
diqiu50 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1851487256
##
settings.gradle.kts:
##
@@ -47,7 +47,8 @@ include(
"clients:filesystem-hadoop3",
"clients:filesystem-hadoop3-runtime",
"clients:client-python",
- "client
xloya commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1851928380
##
api/src/main/java/org/apache/gravitino/model/Model.java:
##
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribu
waukin commented on code in PR #5606:
URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285
##
clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java:
##
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
tengqm commented on code in PR #5643:
URL: https://github.com/apache/gravitino/pull/5643#discussion_r1853143890
##
web/web/src/lib/utils/initial.js:
##
@@ -570,10 +566,64 @@ const relationalColumnTypeMap = {
]
}
-export const getRelationalColumnTypeMap = catalog => {
+expo
xunliu commented on issue #4309:
URL: https://github.com/apache/gravitino/issues/4309#issuecomment-2492791012
hi @amazingLychee Thank you for your interest in this issue.
I already assign it to you. :-)
--
This is an automated message from the Apache Git Service.
To respond to the messa
amazingLychee commented on issue #4309:
URL: https://github.com/apache/gravitino/issues/4309#issuecomment-2492788856
Hi @jerryshao, can I take this?
--
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 t
FANNG1 commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1853227496
##
clients/filesystem-fuse/Cargo.toml:
##
@@ -0,0 +1,34 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. Se
jerryshao commented on issue #4309:
URL: https://github.com/apache/gravitino/issues/4309#issuecomment-2492791717
> Hi @jerryshao, can I take this?
Please go ahead. Thanks @amazingLychee .
--
This is an automated message from the Apache Git Service.
To respond to the message, please
SophieTech88 opened a new pull request, #5646:
URL: https://github.com/apache/gravitino/pull/5646
### What changes were proposed in this pull request?
Implement expression in java, including:
- Expression.java
- FunctionExpression.java
- NamedReference.java
- UnparsedExpression
LiuQhahah commented on PR #5590:
URL: https://github.com/apache/gravitino/pull/5590#issuecomment-2492976776
> @LiuQhahah as #5580 is merged first, could you rebase the code and add
more work:
>
> 1. add operationType method to all specific partition event.
>
> ```
> @Overr
waukin commented on code in PR #5606:
URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285
##
clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java:
##
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
mchades commented on code in PR #5612:
URL: https://github.com/apache/gravitino/pull/5612#discussion_r1853238696
##
api/src/main/java/org/apache/gravitino/model/ModelCatalog.java:
##
@@ -0,0 +1,118 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more
justinmclean commented on code in PR #5638:
URL: https://github.com/apache/gravitino/pull/5638#discussion_r1853279611
##
clients/cli/src/main/java/org/apache/gravitino/cli/TestableCommandLine.java:
##
@@ -0,0 +1,423 @@
+/* Methods used for testing - basically a seam where we can
jerryshao commented on code in PR #5627:
URL: https://github.com/apache/gravitino/pull/5627#discussion_r1853260440
##
clients/filesystem-fuse/Cargo.toml:
##
@@ -0,0 +1,34 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.
justinmclean commented on code in PR #5638:
URL: https://github.com/apache/gravitino/pull/5638#discussion_r1853264072
##
clients/cli/src/main/java/org/apache/gravitino/cli/TestableCommandLine.java:
##
@@ -0,0 +1,423 @@
+/* Methods used for testing - basically a seam where we can
pithecuse527 opened a new pull request, #5645:
URL: https://github.com/apache/gravitino/pull/5645
### What changes were proposed in this pull request?
Remove Key-Value storage backend storage implementations
### Why are the changes needed?
Fix: #5633
### Does t
justinmclean commented on code in PR #5638:
URL: https://github.com/apache/gravitino/pull/5638#discussion_r1853264176
##
clients/cli/src/test/java/org/apache/gravitino/cli/TestCatalogCommands.java:
##
@@ -0,0 +1,293 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
LauraXia123 opened a new pull request, #37:
URL: https://github.com/apache/gravitino-site/pull/37
(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
1 - 100 of 127 matches
Mail list logo
