Pranaykarvi commented on PR #6400:
URL: https://github.com/apache/gravitino/pull/6400#issuecomment-2639087532
> Thanks for your PR!
>
> Could you plz follow the format like other PR titles?
done, please could you check now?
--
This is an automated message from the Apache
mchades commented on PR #6400:
URL: https://github.com/apache/gravitino/pull/6400#issuecomment-2639037180
Thanks for your PR!
Could you plz follow the format like other PR titles?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Pranaykarvi opened a new pull request, #6400:
URL: https://github.com/apache/gravitino/pull/6400
### Issue #6399: UI Should Allow Hyphen in Fileset Name
Issue Description
The UI was incorrectly rejecting file names containing hyphens when creating
a fileset. Although hyphe
FANNG1 commented on PR #6396:
URL: https://github.com/apache/gravitino/pull/6396#issuecomment-2638984657
@jerryshao @theoryxu @diqiu50 PTAL.
--
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 ca20c94b0a [MINOR] fix(gvfs): expose the nested
mchades merged PR #6398:
URL: https://github.com/apache/gravitino/pull/6398
--
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 issue #6260:
URL: https://github.com/apache/gravitino/issues/6260#issuecomment-2638868695
In actuality, the original intention behind the design of the
`testConnection` interface was to validate the usability of the parameters in
the `createCatalog` interface, plz see #
jerryshao commented on issue #6260:
URL: https://github.com/apache/gravitino/issues/6260#issuecomment-2638800036
@mchades can you please take a look at this requirement?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
justinmclean commented on code in PR #6163:
URL: https://github.com/apache/gravitino/pull/6163#discussion_r1944073998
##
clients/cli/src/test/java/org/apache/gravitino/cli/TestCatalogCommands.java:
##
@@ -525,12 +525,19 @@ void testCatalogWithDisableAndEnableOptions() {
justinmclean commented on PR #6343:
URL: https://github.com/apache/gravitino/pull/6343#issuecomment-2638779365
@shaofengshi can you please review this when you have time - you may need to
look at the conversation in other related PRs.
--
This is an automated message from the Apache Git Se
justinmclean closed issue #6356: [Improvement] Add tag support for model in
Gravitino CLI
URL: https://github.com/apache/gravitino/issues/6356
--
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
justinmclean merged PR #6360:
URL: https://github.com/apache/gravitino/pull/6360
--
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...@gra
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 f4a0ebb85c [#6356] improve(CLI): Add tag support
justinmclean closed issue #6356: [Improvement] Add tag support for model in
Gravitino CLI
URL: https://github.com/apache/gravitino/issues/6356
--
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
justinmclean commented on PR #6355:
URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2638767083
Note also that is a 2 clause BSD license not an MIT license.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
mchades commented on issue #6399:
URL: https://github.com/apache/gravitino/issues/6399#issuecomment-2638766872
@LauraXia123 cc
--
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
mchades opened a new issue, #6399:
URL: https://github.com/apache/gravitino/issues/6399
### Version
main branch
### Describe what's wrong
hyphen is allowed in the name spec:
https://github.com/apache/gravitino/blob/d74ce369701ce6a950ef4be485395c814ae34baf/core/src/mai
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 9a1a8baa34 [#6380] fix(postgres-sql):
jerryshao merged PR #6397:
URL: https://github.com/apache/gravitino/pull/6397
--
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-d74ce369701ce6a950ef4be485395c814ae34baf
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 355a070e6b [#6380] fix(postgres-sql): Fix errors for PG ba
justinmclean commented on PR #6355:
URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2638764665
The license can be obtain from their repo and includes the copyright line.
See https://salsa.debian.org/debian/krb5/-/blob/master/NOTICE
Copyright (C) 1985-2024 by the Massac
jerryshao commented on issue #6390:
URL: https://github.com/apache/gravitino/issues/6390#issuecomment-2638764104
I guess this might not be an JDK version problem, it might be the
environment problem, @youngyjd can you try using a clean environment to run the
test again?
--
This is an aut
jerryshao commented on issue #6392:
URL: https://github.com/apache/gravitino/issues/6392#issuecomment-2638759096
I saw this issue for several times, can you please spend time on this?
@FANNG1
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
jerryshao opened a new pull request, #6398:
URL: https://github.com/apache/gravitino/pull/6398
### What changes were proposed in this pull request?
Improve the gvfs to expose the nested exception for better understanding.
### Why are the changes needed?
The exception mess
shaofengshi commented on PR #6359:
URL: https://github.com/apache/gravitino/pull/6359#issuecomment-2638737454
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?
--
This
youngyjd commented on issue #6390:
URL: https://github.com/apache/gravitino/issues/6390#issuecomment-2638738714
let's not worry about it now. It seems the error only happens with my local
environment. I tried it in our CI and on a remote machine, they both went
through.
--
This is an aut
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.8-09d4e64fac9fc2a0d5fdf9304d2e4a0b1f8fdb6f
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was e00fe1e0f6 [6378] improvement: bump log4j dependency from
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.8-316efc26184dba7d8b2ad6f8818c820747277a24
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 89e108912c [#6384] fix: Gravitino default JVM config misma
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.8-342a8b46aa5bcbf1d80299ae71023c703424
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 0223cd4e76 Merge branch 'branch-0.8' into
cherry-pick-bra
yuqi1129 commented on code in PR #6376:
URL: https://github.com/apache/gravitino/pull/6376#discussion_r1944024074
##
catalogs/catalog-hadoop/build.gradle.kts:
##
@@ -52,6 +52,8 @@ dependencies {
exclude("org.eclipse.jetty", "*")
exclude("io.netty")
exclude("org.fu
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.8-d74ce369701ce6a950ef4be485395c814ae34baf
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 355a070e6b [#6380] fix(postgres-sql): Fix errors for
jerryshao closed pull request #6397: [#6380] fix(postgres-sql): Fix errors for
PG backend about `delete...limit..` clause.
URL: https://github.com/apache/gravitino/pull/6397
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
github-actions[bot] opened a new pull request, #6397:
URL: https://github.com/apache/gravitino/pull/6397
### What changes were proposed in this pull request?
PostgreSQL does not support SQL sentences like `DELETE FROM _table where
limit 10` , Cluase `limit xxx` is not allowed
jerryshao closed issue #6380: [Bug report] Purge meta use LIMIT in the delete
statement for POSTGRESQL
URL: https://github.com/apache/gravitino/issues/6380
--
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 #6393:
URL: https://github.com/apache/gravitino/pull/6393
--
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 #6380: [Bug report] Purge meta use LIMIT in the delete
statement for POSTGRESQL
URL: https://github.com/apache/gravitino/issues/6380
--
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 d74ce36970 [#6380] fix(postgres-sql): Fix errors f
shaofengshi commented on issue #6392:
URL: https://github.com/apache/gravitino/issues/6392#issuecomment-2638686518
The command "./gradlew :spark-connector:spark-3.3:test -PtestMode=deploy" at
my side has no error.
--
This is an automated message from the Apache Git Service.
To respond to
jerryshao commented on code in PR #6374:
URL: https://github.com/apache/gravitino/pull/6374#discussion_r1943975905
##
catalogs/catalog-common/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergConstants.java:
##
@@ -84,4 +84,6 @@ public class IcebergConstants {
jerryshao commented on PR #6376:
URL: https://github.com/apache/gravitino/pull/6376#issuecomment-2638592939
@yuqi1129 can you please update this PR?
--
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 #6376:
URL: https://github.com/apache/gravitino/pull/6376#discussion_r1942989638
##
catalogs/catalog-hadoop/build.gradle.kts:
##
@@ -52,6 +52,8 @@ dependencies {
exclude("org.eclipse.jetty", "*")
exclude("io.netty")
exclude("org.fuse
FANNG1 commented on code in PR #6376:
URL: https://github.com/apache/gravitino/pull/6376#discussion_r1942986409
##
catalogs/catalog-hadoop/build.gradle.kts:
##
@@ -52,6 +52,8 @@ dependencies {
exclude("org.eclipse.jetty", "*")
exclude("io.netty")
exclude("org.fuse
yuqi1129 commented on PR #3283:
URL: https://github.com/apache/gravitino/pull/3283#issuecomment-2636493748
Close it temporarily.
--
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 comme
yuqi1129 closed pull request #3283: [#3282] improvement: Support sort order
when create the Doris table.
URL: https://github.com/apache/gravitino/pull/3283
--
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
yuqi1129 commented on code in PR #6376:
URL: https://github.com/apache/gravitino/pull/6376#discussion_r1942705362
##
catalogs/catalog-hadoop/build.gradle.kts:
##
@@ -101,6 +103,9 @@ tasks {
exclude("javax.servlet-*.jar")
exclude("kerb-*.jar")
exclude("kerby-
metalshanked commented on issue #6382:
URL: https://github.com/apache/gravitino/issues/6382#issuecomment-2636451288
Hi @FANNG1 . I was using the latest Trino version (469). I also tried with
older version (459).
I am using the Trino official Docker image.
--
This is an automated messag
diqiu50 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1942487574
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
diqiu50 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1942487574
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
diqiu50 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1942482374
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
diqiu50 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1942462187
##
clients/filesystem-fuse/src/main.rs:
##
@@ -16,49 +16,214 @@
* specific language governing permissions and limitations
* under the License.
*/
-use fuse3::Er
FANNG1 commented on issue #6395:
URL: https://github.com/apache/gravitino/issues/6395#issuecomment-2636067189
This depends on #6337
--
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
zzzk1 commented on PR #6355:
URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2636060460
> > The MIT license needs to include the correct copyright line. We also
need a copy of it in the licenses directory and the files listed in the LICENSE
file.
>
> @justinmclean
yuqi1129 commented on PR #6355:
URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2636049843
> The MIT license needs to include the correct copyright line. We also need
a copy of it in the licenses directory and the files listed in the LICENSE file.
@justinmclean
yuqi1129 commented on code in PR #6393:
URL: https://github.com/apache/gravitino/pull/6393#discussion_r1942420400
##
core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/CatalogMetaPostgreSQLProvider.java:
##
@@ -43,6 +43,16 @@ public String soft
mchades commented on code in PR #6393:
URL: https://github.com/apache/gravitino/pull/6393#discussion_r1942413280
##
core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/CatalogMetaPostgreSQLProvider.java:
##
@@ -43,6 +43,16 @@ public String softD
FANNG1 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1942412103
##
clients/filesystem-fuse/src/command_args.rs:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license
This is an automated email from the ASF dual-hosted git repository.
fanng 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 71998d9df4 [#6279] feat (gvfs-fuse): Add gvfs-fuse
FANNG1 commented on code in PR #6321:
URL: https://github.com/apache/gravitino/pull/6321#discussion_r1942405075
##
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 closed issue #6279: [Subtask] Add gvfs-fuse integration tests for big
files and open-file flag test cases
URL: https://github.com/apache/gravitino/issues/6279
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
FANNG1 merged PR #6280:
URL: https://github.com/apache/gravitino/pull/6280
--
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
FANNG1 closed issue #6279: [Subtask] Add gvfs-fuse integration tests for big
files and open-file flag test cases
URL: https://github.com/apache/gravitino/issues/6279
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
61 matches
Mail list logo