Xander-run commented on code in PR #6824:
URL: https://github.com/apache/gravitino/pull/6824#discussion_r2041156828
##
spark-connector/v3.5/spark/src/test/java/org/apache/gravitino/spark/connector/integration/test/jdbc/SparkJdbcMysqlCatalogIT35.java:
##
@@ -23,7 +23,7 @@
import
Xander-run commented on code in PR #6824:
URL: https://github.com/apache/gravitino/pull/6824#discussion_r2041157382
##
spark-connector/spark-common/src/main/java/org/apache/gravitino/spark/connector/jdbc/postgersql/PostgreSqlPropertiesConverter.java:
##
@@ -0,0 +1,75 @@
+/*
+ *
Abyss-lord opened a new pull request, #6905:
URL: https://github.com/apache/gravitino/pull/6905
### What changes were proposed in this pull request?
Support update properties operations for model alteration.
### Why are the changes needed?
Fix: #6526
### Does this
zhoukangcn opened a new pull request, #6904:
URL: https://github.com/apache/gravitino/pull/6904
### What changes were proposed in this pull request?
use `show backends` to get the count of Doris Backend (BE) nodes
### Why are the changes needed?
Fix: #6722
### Does
Abyss-lord commented on PR #6905:
URL: https://github.com/apache/gravitino/pull/6905#issuecomment-2800249592
Hi @jerryshao , could you please review this PR when you have time? I’d
really appreciate your feedback. The reason for combining multiple PRs into a
single submission is to catch th
yuqi1129 merged PR #6904:
URL: https://github.com/apache/gravitino/pull/6904
--
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
This is an automated email from the ASF dual-hosted git repository.
yuqi4733 pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.9 by this push:
new dd3faaa42a [#6722] fix(jdbc-doris):
xunliu commented on code in PR #6842:
URL: https://github.com/apache/gravitino/pull/6842#discussion_r2041263574
##
core/src/main/java/org/apache/gravitino/converter/CatalogIdConverter.java:
##
Review Comment:
hi @hdygxsj , maybe we can move these `IdConvert.java` to ano
diqiu50 merged PR #6899:
URL: https://github.com/apache/gravitino/pull/6899
--
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 #6345:
URL: https://github.com/apache/gravitino/issues/6345#issuecomment-2800596019
the same with #6681
--
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
Abyss-lord commented on issue #6226:
URL: https://github.com/apache/gravitino/issues/6226#issuecomment-2800620695
@jerryshao I was wondering if we should start writing the documentation now.
I'm a bit concerned that there may not be enough time to cover everything once
all the PRs are merge
jerryshao merged PR #6912:
URL: https://github.com/apache/gravitino/pull/6912
--
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 code in PR #6891:
URL: https://github.com/apache/gravitino/pull/6891#discussion_r2041491096
##
server-common/build.gradle.kts:
##
@@ -39,6 +39,9 @@ dependencies {
implementation(libs.bundles.metrics)
implementation(libs.commons.lang3)
implementation(
jerryshao commented on issue #6226:
URL: https://github.com/apache/gravitino/issues/6226#issuecomment-2800636914
Yes, please do it @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
Xander-run commented on code in PR #6824:
URL: https://github.com/apache/gravitino/pull/6824#discussion_r2041157050
##
spark-connector/spark-common/src/main/java/org/apache/gravitino/spark/connector/jdbc/postgersql/PostgreSqlPropertiesConverter.java:
##
@@ -0,0 +1,75 @@
+/*
+ *
jerqi commented on issue #6544:
URL: https://github.com/apache/gravitino/issues/6544#issuecomment-2800282994
> [@jerqi](https://github.com/jerqi) Hi, I am willing to do this issue. Can
you assign it to me!
You can go ahead.
--
This is an automated message from the Apache Git Servic
danhuawang commented on issue #6845:
URL: https://github.com/apache/gravitino/issues/6845#issuecomment-2800285304
> I had the same error at first. I tried to debug and temporarily comment
out the following section in SchemaMetaService.java:
>
> if (!tableEntities.isEmpty()) {
> t
FANNG1 commented on PR #6891:
URL: https://github.com/apache/gravitino/pull/6891#issuecomment-2800305658
@jerryshao 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 comment
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.9-5720f4a45b55cc04e076b432006a1240e6793f23
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 2b334dba3a [#6526] feat(core): Support update propert
jerryshao closed issue #6526: [Subtask] Support update properties operations
for model alteration.
URL: https://github.com/apache/gravitino/issues/6526
--
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 closed issue #6526: [Subtask] Support update properties operations
for model alteration.
URL: https://github.com/apache/gravitino/issues/6526
--
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
github-actions[bot] opened a new pull request, #6916:
URL: https://github.com/apache/gravitino/pull/6916
### What changes were proposed in this pull request?
Support update properties operations for model alteration.
### Why are the changes needed?
Fix: #6526
###
jerryshao closed pull request #6916: [#6526] feat(core): Support update
properties operations for model alteration
URL: https://github.com/apache/gravitino/pull/6916
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
mchades closed issue #6345: [EPIC] Add multiple storage location support for
fileset
URL: https://github.com/apache/gravitino/issues/6345
--
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
FANNG1 commented on code in PR #6891:
URL: https://github.com/apache/gravitino/pull/6891#discussion_r2041491349
##
lineage/build.gradle.kts:
##
@@ -24,6 +24,7 @@ plugins {
dependencies {
implementation(project(":common"))
implementation(project(":core"))
+ implementation
FANNG1 commented on code in PR #6891:
URL: https://github.com/apache/gravitino/pull/6891#discussion_r2041489994
##
core/src/main/java/org/apache/gravitino/listener/EventBus.java:
##
@@ -47,6 +50,11 @@ public class EventBus {
*/
public EventBus(List eventListeners) {
Pranaykarvi commented on PR #6902:
URL: https://github.com/apache/gravitino/pull/6902#issuecomment-2800501608
@jerqi can u please review the changes
--
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
Abyss-lord commented on PR #6912:
URL: https://github.com/apache/gravitino/pull/6912#issuecomment-2800514087
Hi @jerqi , I've completed the code updates and would appreciate your review
of the PR when you have a moment. Here's a summary of the commits:
1. Use instanceof instead of `getCla
mchades opened a new pull request, #6915:
URL: https://github.com/apache/gravitino/pull/6915
### What changes were proposed in this pull request?
- add `location_name` column to `fileset_version_info` table
- replace field `FilesetVersionPO filesetVersionPO` in `FilesetPO` with
`Li
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.9 by this push:
new 6840b5dc4a [MINOR] improvement(core):
jerryshao commented on PR #6485:
URL: https://github.com/apache/gravitino/pull/6485#issuecomment-2800434583
How about defining a transform interface like `T -> T` for pre-event? If we
have to introduce this layer, we'd better make it more clear for the users, The
concept of wrapper seems to
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.9-d65e7c66d65899dfb39af06932e2d08ba3f5271f
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 5792a7ed45 [MINOR] improvement(core): use version referenc
jerryshao commented on PR #6882:
URL: https://github.com/apache/gravitino/pull/6882#issuecomment-2800436297
@FANNG1 please help to review 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 to
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.9-aac8aa53ca2ef17a8443c241dbc10fc18a0d1e1d
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 2d2abf56f4 [#6874] improvement(core): Set `testOnBorrow` t
jamesjaw commented on issue #6544:
URL: https://github.com/apache/gravitino/issues/6544#issuecomment-2800106479
Hi, I am willing to do this issue. Can you assign it to me!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
Xander-run opened a new issue, #6907:
URL: https://github.com/apache/gravitino/issues/6907
### Version
main branch
### Describe what's wrong
SparkConnector list table in non-existed PG schema doesn't throw
`NoSuchNamespaceException`
### Error message and/or stackt
This is an automated email from the ASF dual-hosted git repository.
zhoukang pushed a change to branch
cherry-pick-branch-0.8-a685e0e276b807a75a50e7a8a8c96e81afe00878
in repository https://gitbox.apache.org/repos/asf/gravitino.git
discard 95ef21f0aa [MINOR] docs: fix several typo in docs (#688
jerryshao merged PR #6910:
URL: https://github.com/apache/gravitino/pull/6910
--
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
jerqi commented on PR #6873:
URL: https://github.com/apache/gravitino/pull/6873#issuecomment-2800558846
After communicating with the author, the fix haven't been verified in his
local environment.
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
jerryshao merged PR #6913:
URL: https://github.com/apache/gravitino/pull/6913
--
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 branch-0.9
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.9 by this push:
new 0a243087ac [#6889] feat(fileset): add
jerryshao merged PR #6914:
URL: https://github.com/apache/gravitino/pull/6914
--
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
Xander-run commented on PR #6824:
URL: https://github.com/apache/gravitino/pull/6824#issuecomment-2800076979
Due to the issue tracked in
[apache/gravitino#6907](https://github.com/apache/gravitino/issues/6907), the
testListTables and testListTable tests in SparkJdbcPostgreSqlCatalogIT are
Xander-run commented on issue #6907:
URL: https://github.com/apache/gravitino/issues/6907#issuecomment-2800077699
I'd like to work on 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 above to go to t
jerryshao commented on PR #6825:
URL: https://github.com/apache/gravitino/pull/6825#issuecomment-2800378432
https://github.com/apache/gravitino/blob/fix-version-numbers/clients/filesystem-fuse/Cargo.toml
The version number in this file should also be updated.
--
This is an automated mess
mchades opened a new pull request, #6910:
URL: https://github.com/apache/gravitino/pull/6910
### What changes were proposed in this pull request?
use version reference for version of current database script
### Why are the changes needed?
when updating the script version,
jerqi commented on PR #6599:
URL: https://github.com/apache/gravitino/pull/6599#issuecomment-2800384771
@jerryshao If u don't have an another suggestion, I will merge this PR
tomorrow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
jerryshao commented on code in PR #6891:
URL: https://github.com/apache/gravitino/pull/6891#discussion_r2041366002
##
lineage/build.gradle.kts:
##
@@ -24,6 +24,7 @@ plugins {
dependencies {
implementation(project(":common"))
implementation(project(":core"))
+ implementat
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.9 by this push:
new 1ff297b3d5 [#6874] improvement(core):
jerryshao merged PR #6911:
URL: https://github.com/apache/gravitino/pull/6911
--
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
Abyss-lord closed pull request #6905: [#6526] feat(core): Support update
properties operations for model alteration
URL: https://github.com/apache/gravitino/pull/6905
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
jerqi commented on PR #6898:
URL: https://github.com/apache/gravitino/pull/6898#issuecomment-2800316250
> This can be merged later on. We need time to discuss which version should
be the next version, 0.10 or 1.0.
OK.
--
This is an automated message from the Apache Git Service.
To
jerryshao commented on PR #6886:
URL: https://github.com/apache/gravitino/pull/6886#issuecomment-2800307942
@zhoukangcn can you please address the rebase issue here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
jerryshao commented on PR #6898:
URL: https://github.com/apache/gravitino/pull/6898#issuecomment-2800313833
This can be merged later on. We need time to discuss which version should be
the next version, 0.10 or 1.0.
--
This is an automated message from the Apache Git Service.
To respond t
mchades commented on issue #6845:
URL: https://github.com/apache/gravitino/issues/6845#issuecomment-2800324249
@waukin The error is produced by the code block you commented out:
```java
if (!tableEntities.isEmpty()) {
throw new NonEmptyEntityException(
"Entity %s has sub-
jerqi merged PR #6878:
URL: https://github.com/apache/gravitino/pull/6878
--
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.
roryqi 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 aac8aa53ca [#6874] improvement(core): Set `testOn
github-actions[bot] opened a new pull request, #6911:
URL: https://github.com/apache/gravitino/pull/6911
### What changes were proposed in this pull request?
Set t`estOnBorrow` and `setTestWhileIdle` of JDBC connection pool from
`false` to `true`.
### Why are the changes ne
jerqi closed issue #6874: [Bug report] List catalog in a long run environment,
encounter issue:"org.apache.ibatis.exceptions.PersistenceException: "
URL: https://github.com/apache/gravitino/issues/6874
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
jerqi closed issue #6874: [Bug report] List catalog in a long run environment,
encounter issue:"org.apache.ibatis.exceptions.PersistenceException: "
URL: https://github.com/apache/gravitino/issues/6874
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.9-aac8aa53ca2ef17a8443c241dbc10fc18a0d1e1d
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 2d2abf56f4 [#6874] improvement(core): Set `testOnBorr
Xander-run opened a new issue, #6906:
URL: https://github.com/apache/gravitino/issues/6906
### Version
main branch
### Describe what's wrong
SLF4J: No SLF4J providers were found.
SLF4J: Ignoring binding found at
### Error message and/or stacktrace
Gradle
Xander-run commented on issue #6906:
URL: https://github.com/apache/gravitino/issues/6906#issuecomment-2800069540
I'd be happy to work on 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 s
jerqi commented on issue #6756:
URL: https://github.com/apache/gravitino/issues/6756#issuecomment-2800347520
> Can I take this？
Are you working on 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 ab
Abyss-lord opened a new pull request, #6912:
URL: https://github.com/apache/gravitino/pull/6912
### What changes were proposed in this pull request?
Support update properties operations for model alteration.
### Why are the changes needed?
Fix: #6526
### Does this
jerqi commented on issue #6647:
URL: https://github.com/apache/gravitino/issues/6647#issuecomment-2800366456
@FANNG1 Would you like to raise a PR to optimize the document?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
jerqi commented on PR #6882:
URL: https://github.com/apache/gravitino/pull/6882#issuecomment-2800362429
@FANNG1 Would you like to take an 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 g
Abyss-lord commented on PR #6912:
URL: https://github.com/apache/gravitino/pull/6912#issuecomment-2800367823
Hi @jerryshao , could you please review this PR when you have time? I’d
really appreciate your feedback. The reason for combining multiple PRs into a
single submission is to catch th
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 9a7cd30f24 [#6889] feat(fileset): add multiple loc
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 d65e7c66d6 [MINOR] improvement(core): use version
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.9-9a7cd30f24d4aa38c7c3f18d58f3b30a0b89dedb
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 1d0415cee4 [#6889] feat(fileset): add multiple locati
jerryshao closed issue #6889: [Subtask] add multiple locations APIs for fileset
URL: https://github.com/apache/gravitino/issues/6889
--
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
jerryshao closed issue #6889: [Subtask] add multiple locations APIs for fileset
URL: https://github.com/apache/gravitino/issues/6889
--
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
github-actions[bot] opened a new pull request, #6913:
URL: https://github.com/apache/gravitino/pull/6913
### What changes were proposed in this pull request?
add multiple locations APIs for fileset
### Why are the changes needed?
Fix: #6889
### Does this PR introd
jerryshao closed pull request #6914: [MINOR] improvement(core): use version
reference for version of current database script
URL: https://github.com/apache/gravitino/pull/6914
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
jerryshao merged PR #6897:
URL: https://github.com/apache/gravitino/pull/6897
--
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, #6914:
URL: https://github.com/apache/gravitino/pull/6914
### What changes were proposed in this pull request?
use version reference for version of current database script
### Why are the changes needed?
when updating the scr
danhuawang commented on code in PR #6599:
URL: https://github.com/apache/gravitino/pull/6599#discussion_r2041360049
##
dev/charts/gravitino/resources/scenarios/ci-values.yaml:
##
@@ -0,0 +1,84 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contrib
jerqi commented on code in PR #6912:
URL: https://github.com/apache/gravitino/pull/6912#discussion_r2041359903
##
api/src/main/java/org/apache/gravitino/model/ModelChange.java:
##
@@ -97,4 +119,138 @@ public String toString() {
return "RenameModel " + newName;
}
}
jerqi commented on code in PR #6912:
URL: https://github.com/apache/gravitino/pull/6912#discussion_r2041359903
##
api/src/main/java/org/apache/gravitino/model/ModelChange.java:
##
@@ -97,4 +119,138 @@ public String toString() {
return "RenameModel " + newName;
}
}
jerqi commented on code in PR #6912:
URL: https://github.com/apache/gravitino/pull/6912#discussion_r2041359903
##
api/src/main/java/org/apache/gravitino/model/ModelChange.java:
##
@@ -97,4 +119,138 @@ public String toString() {
return "RenameModel " + newName;
}
}
jerryshao merged PR #6915:
URL: https://github.com/apache/gravitino/pull/6915
--
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 #6890: [Subtask] backend storage supports multiple
locations for fileset
URL: https://github.com/apache/gravitino/issues/6890
--
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 closed issue #6890: [Subtask] backend storage supports multiple
locations for fileset
URL: https://github.com/apache/gravitino/issues/6890
--
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
84 matches
Mail list logo
