jerqi commented on code in PR #4239:
URL: https://github.com/apache/gravitino/pull/4239#discussion_r1696413803
##
core/src/main/java/org/apache/gravitino/GravitinoEnv.java:
##
@@ -375,4 +385,23 @@ private void initGravitinoServerComponents() {
// Tag manager
this.tagMa
khmgobe commented on PR #4304:
URL: https://github.com/apache/gravitino/pull/4304#issuecomment-2257605562
@yuqi1129
Here are the changes to the code you asked about. When asked why you deleted
the call book, the original code of the call book is read-only.
--
This is an automated mess
khmgobe commented on issue #4305:
URL: https://github.com/apache/gravitino/issues/4305#issuecomment-2257602650
> Note that the check topic call does check that the level length is 3
@justinmclean
What should I assign as the default?
--
This is an automated message from the Apach
xunliu commented on code in PR #4239:
URL: https://github.com/apache/gravitino/pull/4239#discussion_r1696407423
##
core/src/main/java/org/apache/gravitino/GravitinoEnv.java:
##
@@ -317,27 +321,29 @@ private void initGravitinoServerComponents() {
this.idGenerator = new Rando
khmgobe commented on code in PR #4307:
URL: https://github.com/apache/gravitino/pull/4307#discussion_r1696406910
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/authentication/kerberos/KerberosClient.java:
##
@@ -66,7 +65,6 @@ public String login(Str
xunliu commented on code in PR #4239:
URL: https://github.com/apache/gravitino/pull/4239#discussion_r1696209844
##
core/src/main/java/org/apache/gravitino/authorization/AuthorizationUtils.java:
##
@@ -116,4 +123,65 @@ public static void checkRoleNamespace(Namespace namespace)
{
yuqi1129 commented on PR #4302:
URL: https://github.com/apache/gravitino/pull/4302#issuecomment-2257595818
> @yuqi1129 do you have time to review this PR?
I have reviewed it and I'm okay with the change.
--
This is an automated message from the Apache Git Service.
To respond to the
yuqi1129 commented on code in PR #4307:
URL: https://github.com/apache/gravitino/pull/4307#discussion_r1696401596
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/authentication/kerberos/KerberosClient.java:
##
@@ -66,7 +65,6 @@ public String login(St
yuqi1129 commented on code in PR #4257:
URL: https://github.com/apache/gravitino/pull/4257#discussion_r1696394714
##
api/src/main/java/org/apache/gravitino/tag/SupportsTags.java:
##
@@ -58,6 +59,8 @@ public interface SupportsTags {
* @param tagsToAdd The arrays of tag name t
FANNG1 commented on PR #4302:
URL: https://github.com/apache/gravitino/pull/4302#issuecomment-2257586677
@yuqi1129 do you have time 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
FANNG1 commented on code in PR #4302:
URL: https://github.com/apache/gravitino/pull/4302#discussion_r1696395822
##
flink-connector/build.gradle.kts:
##
@@ -27,7 +27,13 @@ repositories {
}
val flinkVersion: String = libs.versions.flink.get()
-val scalaVersion: String = projec
jerqi commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257585566
> Yes, `add a new config option` is also a solution. I think if we can
maintain backward compatibility, we can reuse the previous configuration, so I
didn't introduce a new configuration
jerqi commented on code in PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#discussion_r1696391669
##
server/src/main/java/org/apache/gravitino/server/web/ConfigServlet.java:
##
@@ -51,7 +55,9 @@ public ConfigServlet(ServerConfig serverConfig) {
String config
khmgobe commented on issue #4303:
URL: https://github.com/apache/gravitino/issues/4303#issuecomment-2257579690
I want to improve 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 specific co
justinmclean commented on issue #4305:
URL: https://github.com/apache/gravitino/issues/4305#issuecomment-2257572250
Note that the check topic call does check that the level length is 3
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
khmgobe commented on issue #4305:
URL: https://github.com/apache/gravitino/issues/4305#issuecomment-2257571366
Let me do 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 specific comment.
khmgobe commented on PR #4304:
URL: https://github.com/apache/gravitino/pull/4304#issuecomment-2257570208
@justinmclean Always Have a Goot Time .. !
Among some pager codes that I tried to remove the annotation, I removed the
pager because it was in read-only mode where the annotation
khmgobe opened a new pull request, #4304:
URL: https://github.com/apache/gravitino/pull/4304
What changes were proposed in this pull request?
Modifying declaration parts, removing calls that do not need to be used
Why are the changes needed?
It shouldn't be used, but it's declare
jerqi commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257551301
> Why can't we guarantee compatibility, if upgrade to 0.6.0, the previous
config can also work
Config interface provides different results.
Before 0.6.0, it's a string. After 0.
jerryshao commented on PR #4257:
URL: https://github.com/apache/gravitino/pull/4257#issuecomment-2257546985
@yuqi1129 @jerqi would you please help to review when you have time?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
jenish-thapa commented on issue #4148:
URL: https://github.com/apache/gravitino/issues/4148#issuecomment-2257470628
Hey @danhuawang,
I would like to work on this issue. Can you please assign this to me?
--
This is an automated message from the Apache Git Service.
To respond to the m
coolderli commented on code in PR #4302:
URL: https://github.com/apache/gravitino/pull/4302#discussion_r1696304708
##
flink-connector/build.gradle.kts:
##
@@ -27,7 +27,13 @@ repositories {
}
val flinkVersion: String = libs.versions.flink.get()
-val scalaVersion: String = pro
kohantikanath commented on issue #4270:
URL: https://github.com/apache/gravitino/issues/4270#issuecomment-2257458319
Hi @yuqi1129,
I would love to work on this issue, can you assume this issue?
--
This is an automated message from the Apache Git Service.
To respond to the message, p
lw-yang commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257409543
Why can't we guarantee compatibility, if upgrade to 0.6.0, the previous
config can also work
--
This is an automated message from the Apache Git Service.
To respond to the message, pl
justinmclean commented on issue #3027:
URL: https://github.com/apache/gravitino/issues/3027#issuecomment-2257408714
The issue is that the regex used like this can cause issues. Can you find a
simpler regex or another way of doing it?
--
This is an automated message from the Apache Git Ser
caican00 commented on code in PR #4292:
URL: https://github.com/apache/gravitino/pull/4292#discussion_r1696234494
##
catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/GravitinoPaimonTable.java:
##
@@ -51,7 +64,25 @@ protected TableOper
caican00 commented on code in PR #4292:
URL: https://github.com/apache/gravitino/pull/4292#discussion_r1696234494
##
catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/GravitinoPaimonTable.java:
##
@@ -51,7 +64,25 @@ protected TableOper
jerqi commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257404495
> Yes, `add a new config option` is also a solution. I think if we can
maintain backward compatibility, we can reuse the previous configuration, so I
didn't introduce a new configuration
caican00 commented on code in PR #4292:
URL: https://github.com/apache/gravitino/pull/4292#discussion_r1696234494
##
catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/GravitinoPaimonTable.java:
##
@@ -51,7 +64,25 @@ protected TableOper
caican00 commented on code in PR #4292:
URL: https://github.com/apache/gravitino/pull/4292#discussion_r1696234494
##
catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/GravitinoPaimonTable.java:
##
@@ -51,7 +64,25 @@ protected TableOper
lw-yang commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257398933
Yes, `add a new config option` is also a solution. I think if we can
maintain backward compatibility, we can reuse the previous configuration, so I
didn't introduce a new configuration
justinmclean commented on issue #4023:
URL: https://github.com/apache/gravitino/issues/4023#issuecomment-2257397355
fixed by #3953
--
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 com
justinmclean closed issue #4023: [Improvement] Git ignore file `version.ini`
URL: https://github.com/apache/gravitino/issues/4023
--
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
jerqi commented on code in PR #4239:
URL: https://github.com/apache/gravitino/pull/4239#discussion_r1696226992
##
core/src/main/java/org/apache/gravitino/GravitinoEnv.java:
##
@@ -317,27 +321,29 @@ private void initGravitinoServerComponents() {
this.idGenerator = new Random
jerqi commented on code in PR #4239:
URL: https://github.com/apache/gravitino/pull/4239#discussion_r1696226992
##
core/src/main/java/org/apache/gravitino/GravitinoEnv.java:
##
@@ -317,27 +321,29 @@ private void initGravitinoServerComponents() {
this.idGenerator = new Random
FANNG1 commented on PR #4302:
URL: https://github.com/apache/gravitino/pull/4302#issuecomment-2257394835
we could exclude the Flink connector from Gravitino if the scala version is
2.13, like spark-connector in `setting.gradle.kts`
```
if (scalaVersion == "2.12") {
include("spark-
jerqi commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257390314
> If the `list` is used, the `/config` interface will return `['xx']`, which
includes an additional square bracket, instead of just a string. it require
additional processing for the prev
coolderli commented on PR #4302:
URL: https://github.com/apache/gravitino/pull/4302#issuecomment-2257388122
@FANNG1 Please take a look. Thanks.
--
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
coolderli opened a new pull request, #4302:
URL: https://github.com/apache/gravitino/pull/4302
### What changes were proposed in this pull request?
Flink only supports Scala 2.12. All scala API will removed in the future
version.
https://issues.apache.org/jira/browse/FL
caican00 commented on PR #4288:
URL: https://github.com/apache/gravitino/pull/4288#issuecomment-2257387439
comments have been addressed, and could you help review again? @FANNG1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
caican00 commented on code in PR #4288:
URL: https://github.com/apache/gravitino/pull/4288#discussion_r1696217505
##
catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/TestGravitinoPaimonTable.java:
##
@@ -201,6 +204,80 @@ void testCrea
caican00 commented on code in PR #4288:
URL: https://github.com/apache/gravitino/pull/4288#discussion_r1696217394
##
catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/GravitinoPaimonTable.java:
##
@@ -71,12 +82,17 @@ public static Grav
caican00 commented on code in PR #4288:
URL: https://github.com/apache/gravitino/pull/4288#discussion_r1696217613
##
catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/integration/test/CatalogPaimonBaseIT.java:
##
@@ -283,6 +286,109 @@
caican00 commented on code in PR #4288:
URL: https://github.com/apache/gravitino/pull/4288#discussion_r1696216953
##
catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/GravitinoPaimonTable.java:
##
@@ -52,6 +57,12 @@ protected TableOper
lw-yang commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257386067
If the `list` is used, the `/config` interface will return `['xx']`, which
includes an additional square bracket, instead of just a string. it require
additional processing for the prev
khmgobe commented on issue #4174:
URL: https://github.com/apache/gravitino/issues/4174#issuecomment-2257383497
> > @yuqi1129
> > BUILD SUCCESSFUL in 1m 18s 125 actionable tasks: 83 executed, 42
up-to-date
> > It takes about a minute for me, is it based on the time when I just hit
that
jerqi commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257379824
> We should maintain backward compatibility, so revert the changes to keep
`string` type
First, It won't break backward compatiblity using sequeuece.
Second, how to handle the st
lw-yang commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257377248
We should maintain backward compatibility, so revert the changes to keep
`string` type
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
mchades closed issue #3368: [Subtask] enable pre-assign partition for Doris
catalog
URL: https://github.com/apache/gravitino/issues/3368
--
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
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 4acdc498c [#3368] feat(jdbc-doris): support pre-
jerqi closed pull request #4283: Issue 4169
URL: https://github.com/apache/gravitino/pull/4283
--
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-u
mchades merged PR #3887:
URL: https://github.com/apache/gravitino/pull/3887
--
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
jerqi commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257374838
> @jerqi @lw-yang has reverted the changes about the type of authenticator
from `List` to `String`, Is it okay for you?
Not ok for me. It's weird.
--
This is an automated messag
yuqi1129 closed pull request #4260: [TEST]Test CI
URL: https://github.com/apache/gravitino/pull/4260
--
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: com
yuqi1129 commented on issue #4174:
URL: https://github.com/apache/gravitino/issues/4174#issuecomment-2257368700
> @yuqi1129
>
> BUILD SUCCESSFUL in 1m 18s 125 actionable tasks: 83 executed, 42 up-to-date
>
> It takes about a minute for me, is it based on the time when I just hit
khmgobe commented on issue #4174:
URL: https://github.com/apache/gravitino/issues/4174#issuecomment-2257365799
@yuqi1129
BUILD SUCCESSFUL in 1m 18s
125 actionable tasks: 83 executed, 42 up-to-date
It takes about a minute for me, is it based on the time when I just hit that
khmgobe commented on issue #4185:
URL: https://github.com/apache/gravitino/issues/4185#issuecomment-2257360936
@justinmclean Do you mind if I try 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 g
yuqi1129 commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2257360487
@jerqi
@lw-yang has reverted the changes about the type of authenticator from
`List` to `String`, Is it okay for you?
--
This is an automated message from the Apache Git Serv
yuqi1129 closed pull request #4296: [#4293] improvement(catalogs): add deleted
flag and Error Log
URL: https://github.com/apache/gravitino/pull/4296
--
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
yuqi1129 closed issue #4293: [Improvement] In
SecureHadoopCatalogOperations.java checkif file is deleted
URL: https://github.com/apache/gravitino/issues/4293
--
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
yuqi1129 commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257356359
This issue was resolved in #4297.
--
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
khmgobe commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257355084
> This issue was resolved by #4297 by the way, perhaps the PR can be closed?
sure ! closed please, Have a Good Time !
--
This is an automated message from the Apache Git S
yuqi1129 commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257353998
This issue was resolved by #4297 by the way, perhaps the PR can be closed?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
This is an automated email from the ASF dual-hosted git repository.
yuqi4733 pushed a change to annotated tag kerberos-hive-ci-v0.1.5
in repository https://gitbox.apache.org/repos/asf/gravitino.git
*** WARNING: tag kerberos-hive-ci-v0.1.5 was modified! ***
from 1b6a1556b (commit)
to a
This is an automated email from the ASF dual-hosted git repository.
fanng pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
from a8e9a31a0 [#4294] improvement(catalogs): Improved readability of
trinomial operator usage code (#4297)
add a9e50
FANNG1 closed issue #4269: [Subtask] Add flink-connector doc for 0.6 release
URL: https://github.com/apache/gravitino/issues/4269
--
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.
FANNG1 closed issue #4269: [Subtask] Add flink-connector doc for 0.6 release
URL: https://github.com/apache/gravitino/issues/4269
--
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.
FANNG1 merged PR #4284:
URL: https://github.com/apache/gravitino/pull/4284
--
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
yuqi1129 commented on PR #3887:
URL: https://github.com/apache/gravitino/pull/3887#issuecomment-2257341934
> Overall LGTM. @yuqi1129 Do you want to take another look?
I'm okay with the current changes.
--
This is an automated message from the Apache Git Service.
To respond to the me
mchades opened a new pull request, #4301:
URL: https://github.com/apache/gravitino/pull/4301
### What changes were proposed in this pull request?
use UnparsedExpression to represent unpared column default value
### Why are the changes needed?
Fix: #4300
### Does t
FANNG1 commented on issue #4298:
URL: https://github.com/apache/gravitino/issues/4298#issuecomment-2257338510
@coolderli do you have time to check 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
khmgobe commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257333027
> https://private-user-images.githubusercontent.com/15794564/353289057-9ccf44e7-db19-40e6-bb10-880b23fb95f2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiY
yuqi1129 commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257330045
https://github.com/user-attachments/assets/9ccf44e7-db19-40e6-bb10-880b23fb95f2";>
Please take a look at at the code change.
--
This is an automated message from the Apach
khmgobe commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257325585
@yuqi1129
> @yuqi1129 I hope you have a good day today. Please review it.
>
>Please look at the promotion and I'm confused about the actual change.
Existing cha
yuqi1129 commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257322648
> @yuqi1129 I hope you have a good day today. Please review it.
Please look at your PR and I'm confused about the actual change you made.
--
This is an automated message f
jerqi commented on PR #4248:
URL: https://github.com/apache/gravitino/pull/4248#issuecomment-2257321907
> > It's the best to have the abstraction like
> > ```
> > class UserContext {
> > UserGroupInformation getUGI();
> > doAs();
> > }
> >
> > class SimpleUserC
khmgobe commented on issue #4293:
URL: https://github.com/apache/gravitino/issues/4293#issuecomment-2257320564
@yuqi1129 I hope you have a good day today. Please review it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
coolderli commented on code in PR #4284:
URL: https://github.com/apache/gravitino/pull/4284#discussion_r1696170076
##
docs/flink-connector/flink-connector.md:
##
@@ -0,0 +1,93 @@
+---
+title: "Apache Gravitino Flink connector"
+slug: /flink-connector/flink-connector
+keyword: fl
coolderli commented on PR #4284:
URL: https://github.com/apache/gravitino/pull/4284#issuecomment-2257313172
@FANNG1 I have fixed. Please take a look. Thanks.
--
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
coolderli commented on code in PR #4284:
URL: https://github.com/apache/gravitino/pull/4284#discussion_r1696168591
##
docs/flink-connector/flink-connector.md:
##
@@ -0,0 +1,93 @@
+---
+title: "Apache Gravitino Flink connector"
+slug: /flink-connector/flink-connector
+keyword: fl
coolderli commented on code in PR #4284:
URL: https://github.com/apache/gravitino/pull/4284#discussion_r1696165538
##
docs/flink-connector/flink-connector.md:
##
@@ -0,0 +1,93 @@
+---
+title: "Apache Gravitino Flink connector"
+slug: /flink-connector/flink-connector
+keyword: fl
yuqi1129 closed issue #4294: [Improvement] Use of nested ternary operation is a
little confusing
URL: https://github.com/apache/gravitino/issues/4294
--
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
yuqi1129 closed issue #4294: [Improvement] Use of nested ternary operation is a
little confusing
URL: https://github.com/apache/gravitino/issues/4294
--
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.
yuqi4733 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 a8e9a31a0 [#4294] improvement(catalogs): Improv
yuqi1129 merged PR #4297:
URL: https://github.com/apache/gravitino/pull/4297
--
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
mchades closed issue #4258: [Bug report] Missing description of support type
uuid
URL: https://github.com/apache/gravitino/issues/4258
--
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
mchades closed issue #4258: [Bug report] Missing description of support type
uuid
URL: https://github.com/apache/gravitino/issues/4258
--
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
This is an automated email from the ASF dual-hosted git repository.
mchades pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
from 8c4a32590 [#3785] Throw a clear exception for unsupported Trino
versions when Trino starts (#4268)
add 0234888
mchades merged PR #4299:
URL: https://github.com/apache/gravitino/pull/4299
--
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
ashwin1596 commented on issue #3027:
URL: https://github.com/apache/gravitino/issues/3027#issuecomment-2257220976
I am a bit confused about the issue. We are using the error code for
"Database already exists" defined as:
`@VisibleForTesting static final int CODE_DATABASE_EXISTS = 1007
jingjia88 commented on PR #4271:
URL: https://github.com/apache/gravitino/pull/4271#issuecomment-2256209205
@noidname01 Done! Please help to take a look again, Thank you~
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
jingjia88 commented on code in PR #4271:
URL: https://github.com/apache/gravitino/pull/4271#discussion_r1695399066
##
clients/client-python/gravitino/exceptions/base.py:
##
@@ -54,6 +54,9 @@ class InternalError(GravitinoRuntimeException):
class NotFoundException(GravitinoRuntim
Vishesh-Paliwal commented on PR #4299:
URL: https://github.com/apache/gravitino/pull/4299#issuecomment-2256148817
Hi @mchades , sorry about that , I'm new to open source . Kindly review now
.
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
khmgobe commented on PR #4296:
URL: https://github.com/apache/gravitino/pull/4296#issuecomment-2256132683
To. Reviewer
Please note that the pre-PR content exclusion change code has been changed
to conform to Spotless's rules.
--
This is an automated message from the Apache Git Ser
mchades commented on PR #4299:
URL: https://github.com/apache/gravitino/pull/4299#issuecomment-2256114596
Thanks for your PR. Please adjust the title and description to adhere to the
required format.
--
This is an automated message from the Apache Git Service.
To respond to the message, p
yuqi1129 commented on issue #4266:
URL: https://github.com/apache/gravitino/issues/4266#issuecomment-2256108679
1. It's a good idea to list some commonly used build options and
combinations in the `kts` file.
2. Putting all build options in a conspicuous place like in the `build.kts`
wil
xunliu commented on issue #4266:
URL: https://github.com/apache/gravitino/issues/4266#issuecomment-2256086348
@mchades @yuqi1129 What do you think about this issue, Please give some
suggestions, Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message
khmgobe commented on PR #4297:
URL: https://github.com/apache/gravitino/pull/4297#issuecomment-2256081600
> @khmgobe Please change your PR description according to the PR template.
@yuqi1129 My mistake. I'm sorry. I've changed the description, so please
check it
--
This is an autom
Vishesh-Paliwal commented on issue #4258:
URL: https://github.com/apache/gravitino/issues/4258#issuecomment-2256072171
hi @justinmclean , Created a PR . kindly review .
PS : sorry about earlier one , I'm new to open source
--
This is an automated message from the Apache Git Service
Vishesh-Paliwal opened a new pull request, #4299:
URL: https://github.com/apache/gravitino/pull/4299
What PR does :
Adds description of support type uuid
### What changes were proposed in this pull request?
https://datastrato.ai/docs/0.5.1/manage-relational-metadata-usi
1 - 100 of 151 matches
Mail list logo
