Re: [I] Update the license header to use Apache V2 license header instead of Company's one [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4035: Update the license header to use Apache V2 license header instead of Company's one URL: https://github.com/apache/gravitino/issues/4035 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
jerryshao merged PR #4045: URL: https://github.com/apache/gravitino/pull/4045 -- 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

Re: [I] Update the license header to use Apache V2 license header instead of Company's one [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4035: Update the license header to use Apache V2 license header instead of Company's one URL: https://github.com/apache/gravitino/issues/4035 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on PR #4045: URL: https://github.com/apache/gravitino/pull/4045#issuecomment-2205232232 Let's merge it first, if there's any missing, we can fix it later on. This PR is too big and easy to conflict. -- This is an automated message from the Apache Git Service. To respon

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
justinmclean commented on code in PR #4045: URL: https://github.com/apache/gravitino/pull/4045#discussion_r1663599794 ## build.gradle.kts: ## @@ -459,6 +459,7 @@ tasks.rat { "**/licenses/*.txt", "**/licenses/*.md", "integration-test/**", +"docs/*.md", Review

[PR] Update README [gravitino-playground]

2024-07-02 Thread via GitHub
justinmclean opened a new pull request, #49: URL: https://github.com/apache/gravitino-playground/pull/49 Add ASF disclaimer, trademark attribution and Apache Gravitino where needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] [#3732] refactor(client-python): Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
noidname01 commented on code in PR #4046: URL: https://github.com/apache/gravitino/pull/4046#discussion_r1663592094 ## clients/client-python/gravitino/name_identifier.py: ## @@ -26,7 +28,13 @@ class NameIdentifier(DataClassJsonMixin): _DOT: ClassVar[str] = "." _name:

Re: [PR] [#4007] improvement: Use template to reduce Privileges duplicate codes [gravitino]

2024-07-02 Thread via GitHub
rich7420 commented on PR #4010: URL: https://github.com/apache/gravitino/pull/4010#issuecomment-2205206025 @jerqi Sorry , I forgot to push. Thank you for your reply. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [#3732] refactor(client-python): Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
noidname01 commented on code in PR #4046: URL: https://github.com/apache/gravitino/pull/4046#discussion_r1663587824 ## clients/client-python/gravitino/filesystem/gvfs.py: ## @@ -570,11 +570,14 @@ def _load_fileset_from_server(self, identifier: NameIdentifier) -> Fileset:

Re: [PR] [#3732] refactor(client-python): Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
shaofengshi commented on code in PR #4046: URL: https://github.com/apache/gravitino/pull/4046#discussion_r1663585252 ## clients/client-python/gravitino/name_identifier.py: ## @@ -26,7 +28,13 @@ class NameIdentifier(DataClassJsonMixin): _DOT: ClassVar[str] = "." _name

[PR] Add ASF headers [gravitino-playground]

2024-07-02 Thread via GitHub
justinmclean opened a new pull request, #47: URL: https://github.com/apache/gravitino-playground/pull/47 Change file headers to ASF headers where needed -- 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

[I] Replace file headers with ASF where needed [gravitino-playground]

2024-07-02 Thread via GitHub
justinmclean opened a new issue, #46: URL: https://github.com/apache/gravitino-playground/issues/46 Replace Datastrato file headers with ASF headers -- 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

Re: [PR] [#3732] refactor(client-python): Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
shaofengshi commented on code in PR #4046: URL: https://github.com/apache/gravitino/pull/4046#discussion_r1663583318 ## clients/client-python/gravitino/filesystem/gvfs.py: ## @@ -570,11 +570,14 @@ def _load_fileset_from_server(self, identifier: NameIdentifier) -> Fileset:

Re: [PR] [MINOR] [#4037] Upgrade twine version to fix python client deploy task [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on PR #4038: URL: https://github.com/apache/gravitino/pull/4038#issuecomment-2205194312 > > Hi @xloya can you please update the branch, we have fixed several CI issues to make CI work. > > @jerryshao Done, it seems to be need approve to trigger workflow after tran

Re: [I] [Subtask] Fix or update several broken CI workflows [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 commented on issue #4042: URL: https://github.com/apache/gravitino/issues/4042#issuecomment-2205190024 > We can discuss with ASF Infra what might be possible. Thanks, if there's any progress, please leave a message to let us know. -- This is an automated message from the Ap

Re: [PR] [#4007] improvement: Use template to reduce Privileges duplicate codes [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on PR #4010: URL: https://github.com/apache/gravitino/pull/4010#issuecomment-2205189984 > > > > @qqqttt123 According to the idea, Maybe I can change code like this to add `ALLOW_INSTANCE` and `DENY_INSTANCE`. What do you think? > > > > ``` > > > > public static class

Re: [PR] [#3755] improvement(client-python): Support OAuth2TokenProvider for Python client [gravitino]

2024-07-02 Thread via GitHub
noidname01 commented on code in PR #4011: URL: https://github.com/apache/gravitino/pull/4011#discussion_r1663573099 ## clients/client-python/gravitino/utils/http_client.py: ## @@ -124,23 +124,38 @@ def _make_request(self, opener, request, timeout=None): raise exc fr

Re: [PR] [MINOR] [#4037] Upgrade twine version to fix python client deploy task [gravitino]

2024-07-02 Thread via GitHub
xloya commented on PR #4038: URL: https://github.com/apache/gravitino/pull/4038#issuecomment-2205188131 > Hi @xloya can you please update the branch, we have fixed several CI issues to make CI work. @jerryshao Done, it seems to be need approve to trigger workflow after transfer to th

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
justinmclean commented on code in PR #4045: URL: https://github.com/apache/gravitino/pull/4045#discussion_r1663570291 ## build.gradle.kts: ## @@ -459,6 +459,7 @@ tasks.rat { "**/licenses/*.txt", "**/licenses/*.md", "integration-test/**", +"docs/*.md", Review

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on code in PR #4045: URL: https://github.com/apache/gravitino/pull/4045#discussion_r1663568318 ## build.gradle.kts: ## @@ -459,6 +459,7 @@ tasks.rat { "**/licenses/*.txt", "**/licenses/*.md", "integration-test/**", +"docs/*.md", Review Com

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
justinmclean commented on code in PR #4045: URL: https://github.com/apache/gravitino/pull/4045#discussion_r1663566097 ## build.gradle.kts: ## @@ -459,6 +459,7 @@ tasks.rat { "**/licenses/*.txt", "**/licenses/*.md", "integration-test/**", +"docs/*.md", Review

Re: [PR] [#4007] improvement: Use template to reduce Privileges duplicate codes [gravitino]

2024-07-02 Thread via GitHub
rich7420 commented on PR #4010: URL: https://github.com/apache/gravitino/pull/4010#issuecomment-2205181458 > > > @qqqttt123 According to the idea, Maybe I can change code like this to add `ALLOW_INSTANCE` and `DENY_INSTANCE`. What do you think? > > > ``` > > > public static class Cre

Re: [PR] [#4039] Update NOTICE files to ASF norms [gravitino]

2024-07-02 Thread via GitHub
jerryshao merged PR #4043: URL: https://github.com/apache/gravitino/pull/4043 -- 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

Re: [I] [Improvement] Update NOTICE files [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4039: [Improvement] Update NOTICE files URL: https://github.com/apache/gravitino/issues/4039 -- 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 unsubscr

(gravitino) branch main updated: [#4039] Update NOTICE files to ASF norms (#4043)

2024-07-02 Thread jshao
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 30b11ce4a [#4039] Update NOTICE files to ASF norms

[PR] [#4048] Update README with incubator disclaimer [gravitino]

2024-07-02 Thread via GitHub
justinmclean opened a new pull request, #4049: URL: https://github.com/apache/gravitino/pull/4049 ### What changes were proposed in this pull request? Update README with incubator disclaimer and updated links and used Apache Gravitino where needed. ### Why are the changes neede

Re: [PR] [#3755] improvement(client-python): Support OAuth2TokenProvider for Python client [gravitino]

2024-07-02 Thread via GitHub
noidname01 commented on code in PR #4011: URL: https://github.com/apache/gravitino/pull/4011#discussion_r1663561382 ## clients/client-python/gravitino/auth/oauth2_token_provider.py: ## @@ -0,0 +1,44 @@ +""" +Copyright 2024 Datastrato Pvt Ltd. +This software is licensed under the

Re: [PR] [#3371] feat(flink-connector): support basic table operation [gravitino]

2024-07-02 Thread via GitHub
coolderli commented on code in PR #3795: URL: https://github.com/apache/gravitino/pull/3795#discussion_r1663559654 ## flink-connector/src/main/java/com/datastrato/gravitino/flink/connector/utils/TypeUtils.java: ## @@ -0,0 +1,42 @@ +/* + * Copyright 2024 Datastrato Pvt Ltd. + * T

Re: [PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on code in PR #4045: URL: https://github.com/apache/gravitino/pull/4045#discussion_r1663557356 ## build.gradle.kts: ## @@ -459,6 +459,7 @@ tasks.rat { "**/licenses/*.txt", "**/licenses/*.md", "integration-test/**", +"docs/*.md", Review Com

Re: [PR] [MINOR] [#4037] Upgrade twine version to fix python client deploy task [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on PR #4038: URL: https://github.com/apache/gravitino/pull/4038#issuecomment-2205171379 Hi @xloya can you please update the branch, we have fixed several CI issues to make CI work. -- This is an automated message from the Apache Git Service. To respond to the message,

(gravitino) branch main updated (0e59cc854 -> 33ca5812c)

2024-07-02 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from 0e59cc854 [#4031] build: Add the asf yaml for Gravitino (#4033) add 33ca5812c [#4031][followup] build: Revert the

Re: [PR] [#4031][followup] build: Revert the partial protected branches [gravitino]

2024-07-02 Thread via GitHub
jerryshao merged PR #4047: URL: https://github.com/apache/gravitino/pull/4047 -- 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

Re: [PR] [#3371] feat(flink-connector): support basic table operation [gravitino]

2024-07-02 Thread via GitHub
coolderli commented on code in PR #3795: URL: https://github.com/apache/gravitino/pull/3795#discussion_r1663538685 ## flink-connector/src/main/java/com/datastrato/gravitino/flink/connector/catalog/BaseCatalog.java: ## @@ -337,6 +411,29 @@ public void alterPartitionColumnStatisti

[PR] [#4031][followup] build: Revert the partial protected branches [gravitino]

2024-07-02 Thread via GitHub
jerqi opened a new pull request, #4047: URL: https://github.com/apache/gravitino/pull/4047 ### What changes were proposed in this pull request? Apache infra doesn't support `branch-*`. ### Why are the changes needed? This is a follow-up pull request of #4033 Fix: #403

Re: [PR] [#3732] refactor(client-python): Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
noidname01 commented on PR #4046: URL: https://github.com/apache/gravitino/pull/4046#issuecomment-2205136418 @shaofengshi @xloya @jerryshao Please review this PR, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[PR] [#3732] refactor(client-python): Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
noidname01 opened a new pull request, #4046: URL: https://github.com/apache/gravitino/pull/4046 ### What changes were proposed in this pull request? After the refactoring work in Java client is done (https://github.com/apache/gravitino/issues/3626), the Python client should a

Re: [I] [Subtask] Fix or update several broken CI workflows [gravitino]

2024-07-02 Thread via GitHub
justinmclean commented on issue #4042: URL: https://github.com/apache/gravitino/issues/4042#issuecomment-2205132214 We can discuss with ASF Infra what might be possible. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[PR] [#4035] Update ASF headers in all files [gravitino]

2024-07-02 Thread via GitHub
justinmclean opened a new pull request, #4045: URL: https://github.com/apache/gravitino/pull/4045 ### What changes were proposed in this pull request? Update all ASF headers, including the short version in documents and generated headers. ### Why are the changes needed?

[PR] [#4040] Update Rat check to ignore Datastrato headers [gravitino]

2024-07-02 Thread via GitHub
justinmclean opened a new pull request, #4044: URL: https://github.com/apache/gravitino/pull/4044 ### What changes were proposed in this pull request? No longer accept Datastrato headers. ### Why are the changes needed? See above. Fix: # (issue) ### Does

[PR] [#4039] Update NOTICE files to ASF norms [gravitino]

2024-07-02 Thread via GitHub
justinmclean opened a new pull request, #4043: URL: https://github.com/apache/gravitino/pull/4043 ### What changes were proposed in this pull request? Update NOTICE files to comply with ASF policy. ### Why are the changes needed? To comply with ASF policy. Fix: #40

Error while running github feature from refs/heads/main:.asf.yaml in gravitino!

2024-07-02 Thread Apache Infrastructure
An error occurred while running github feature in .asf.yaml!: [GitHub] Request error with message: "Branch not found". (status code: 404)

(gravitino) branch main updated: [#4031] build: Add the asf yaml for Gravitino (#4033)

2024-07-02 Thread jshao
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 0e59cc854 [#4031] build: Add the asf yaml for Grav

Re: [I] [Improvement] Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4031: [Improvement] Add the asf yaml for Gravitino URL: https://github.com/apache/gravitino/issues/4031 -- 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.

Re: [I] [Improvement] Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4031: [Improvement] Add the asf yaml for Gravitino URL: https://github.com/apache/gravitino/issues/4031 -- 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.

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerryshao merged PR #4033: URL: https://github.com/apache/gravitino/pull/4033 -- 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

Re: [I] [EPIC] Works to do after entering ASF Incubating [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on issue #4034: URL: https://github.com/apache/gravitino/issues/4034#issuecomment-2205056002 > We should modify the license and document of playground, too. Yes, please also create issues to track this. -- This is an automated message from the Apache Git Service.

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
shaofengshi commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663450394 ## .asf.yaml: ## @@ -0,0 +1,61 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file

Re: [I] [EPIC] Works to do after entering ASF Incubating [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on issue #4034: URL: https://github.com/apache/gravitino/issues/4034#issuecomment-2205044643 We should modify the license and document of playground, too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] [#4007] improvement: Use template to reduce Privileges duplicate codes [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on PR #4010: URL: https://github.com/apache/gravitino/pull/4010#issuecomment-2205040458 > > @qqqttt123 According to the idea, Maybe I can change code like this to add `ALLOW_INSTANCE` and `DENY_INSTANCE`. What do you think? > > ``` > > public static class CreateCatalo

Re: [PR] [#4007] improvement: Use template to reduce Privileges duplicate codes [gravitino]

2024-07-02 Thread via GitHub
rich7420 commented on PR #4010: URL: https://github.com/apache/gravitino/pull/4010#issuecomment-2205037934 @xunliu thank you. I appreciate 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 th

(gravitino) branch main updated: [#4032] Fix(CI): Remove some CI actions to make CI back to normal (#4030)

2024-07-02 Thread jshao
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 6180bc650 [#4032] Fix(CI): Remove some CI actions

Re: [I] [Bug report] Some CI pipeline can't work after transfer Gravitino to Apache [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4032: [Bug report] Some CI pipeline can't work after transfer Gravitino to Apache URL: https://github.com/apache/gravitino/issues/4032 -- 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

Re: [I] [Bug report] Some CI pipeline can't work after transfer Gravitino to Apache [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed issue #4032: [Bug report] Some CI pipeline can't work after transfer Gravitino to Apache URL: https://github.com/apache/gravitino/issues/4032 -- 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

Re: [PR] [#4032] Fix(CI): Remove some CI actions to make CI back to normal [gravitino]

2024-07-02 Thread via GitHub
jerryshao merged PR #4030: URL: https://github.com/apache/gravitino/pull/4030 -- 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

Re: [PR] [#4032] Fix(CI): Remove some CI actions to make CI back to normal [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 commented on PR #4030: URL: https://github.com/apache/gravitino/pull/4030#issuecomment-2205009485 > @yuqi1129 Can you please create an issue to track jacoco issue, I think we can use codecov instead. I see. -- This is an automated message from the Apache Git Service. To r

Re: [PR] [#4032] Fix(CI): Remove some CI actions to make CI back to normal [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on PR #4030: URL: https://github.com/apache/gravitino/pull/4030#issuecomment-2205008886 @yuqi1129 Can you please create an issue to track jacoco issue, I think we can use codecov instead. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663430464 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distr

Re: [PR] [MINOR] [#4037] Upgrade twine version to fix python client deploy task [gravitino]

2024-07-02 Thread via GitHub
xloya closed pull request #4038: [MINOR] [#4037] Upgrade twine version to fix python client deploy task URL: https://github.com/apache/gravitino/pull/4038 -- 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

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663430189 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distr

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663424244 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file d

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663423194 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file d

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663421650 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file d

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663420700 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distr

[PR] [MINOR] [#4037] Upgrade twine version to fix python client deploy task [gravitino]

2024-07-02 Thread via GitHub
xloya opened a new pull request, #4038: URL: https://github.com/apache/gravitino/pull/4038 ### What changes were proposed in this pull request? Currently twine 5.0.0 will cause Python client deploy task failed, upgrade twine version to fix this problem. ### Why are the changes

[I] [MINOR] Upgrade twine version to fix python client deploy task [gravitino]

2024-07-02 Thread via GitHub
xloya opened a new issue, #4037: URL: https://github.com/apache/gravitino/issues/4037 ### What would you like to be improved? ``` Traceback (most recent call last): File "/builds/P6p6yuG-/0/gravitino/.gradle/python/Linux/Miniforge3/envs/python-3.8/bin/twine", line 5, in

[I] [EPIC] Works to do after entering ASF Incubating [gravitino]

2024-07-02 Thread via GitHub
jerryshao opened a new issue, #4034: URL: https://github.com/apache/gravitino/issues/4034 ### Describe the proposal There are many works we need to do after entering the ASF incubator, so here listed several things we should do recently: ### Task list - [ ] #4032 - [

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663412635 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file di

Re: [PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerqi commented on code in PR #4033: URL: https://github.com/apache/gravitino/pull/4033#discussion_r1663412578 ## .asf.yaml: ## @@ -0,0 +1,55 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distr

Re: [I] [Subtask] Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
shaofengshi commented on issue #3732: URL: https://github.com/apache/gravitino/issues/3732#issuecomment-2204974989 Reopen this as there are other on-going tasks being made by Lance and Tim. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

[I] [Subtask] Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
shaofengshi opened a new issue, #3732: URL: https://github.com/apache/gravitino/issues/3732 ### Describe the subtask After the refactoring work in Java client is done (https://github.com/datastrato/gravitino/issues/3626), the Python client should also get updated to align with the Ja

[PR] [#4031] build: Add the asf yaml for Gravitino [gravitino]

2024-07-02 Thread via GitHub
jerqi opened a new pull request, #4033: URL: https://github.com/apache/gravitino/pull/4033 ### What changes were proposed in this pull request? Add the asf yaml for Gravitino ### Why are the changes needed? Fix: #4031 ### Does this PR introduce _any_ user-facing chang

Re: [I] [Subtask] Fix duplicate-code Pylint rule for client-python [gravitino]

2024-07-02 Thread via GitHub
noidname01 commented on issue #3767: URL: https://github.com/apache/gravitino/issues/3767#issuecomment-2204965265 Hi @ian910297, thanks for your interest in this issue. If you have any question, feel free to comment and let us know👍 -- This is an automated message from the Apache Git Serv

Re: [PR] [#2625] fix: cannot compile under docker bash [gravitino]

2024-07-02 Thread via GitHub
liujinhui1994 closed pull request #2626: [#2625] fix: cannot compile under docker bash URL: https://github.com/apache/gravitino/pull/2626 -- 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

(gravitino) branch main updated: [#3732] refactor: update Python client to align with the Java client API: gravitino_metalake, gravitino_client (#4025)

2024-07-02 Thread liuxun
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 1134386e4 [#3732] refactor: update Python client

Re: [PR] [#3732] refactor: update Python client to align with the Java client API: gravitino_metalake, gravitino_client [gravitino]

2024-07-02 Thread via GitHub
xunliu merged PR #4025: URL: https://github.com/apache/gravitino/pull/4025 -- 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

Re: [I] [Subtask] Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
xunliu closed issue #3732: [Subtask] Update Python client to align with the Java client API URL: https://github.com/apache/gravitino/issues/3732 -- 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

Re: [I] [Subtask] Update Python client to align with the Java client API [gravitino]

2024-07-02 Thread via GitHub
xunliu closed issue #3732: [Subtask] Update Python client to align with the Java client API URL: https://github.com/apache/gravitino/issues/3732 -- 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

[I] [Bug report] Some CI pipeline can't work after transfer Gravitino to Apache [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 opened a new issue, #4032: URL: https://github.com/apache/gravitino/issues/4032 ### Version main branch ### Describe what's wrong ![image](https://github.com/apache/gravitino/assets/15794564/97fc60bf-5139-446f-843a-bac4daf94287) https://github

Re: [PR] [#3064] feat(jdbc-doris): Supports partition management in Doris catalog [gravitino]

2024-07-02 Thread via GitHub
xiaozcy commented on code in PR #3961: URL: https://github.com/apache/gravitino/pull/3961#discussion_r1663385378 ## api/src/main/java/com/datastrato/gravitino/exceptions/NotPartitionedTableException.java: ## @@ -0,0 +1,35 @@ +/* + * Copyright 2024 Datastrato Pvt Ltd. + * This so

Re: [PR] [Minor] improvement(CI): Remove some CI actions to make CI back to normal [gravitino]

2024-07-02 Thread via GitHub
jerryshao commented on PR #4030: URL: https://github.com/apache/gravitino/pull/4030#issuecomment-2204907738 Create an issue to track 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 to the spec

[PR] [Test] Remove some CI actions to make CI back to normal [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 opened a new pull request, #4030: URL: https://github.com/apache/gravitino/pull/4030 ### What changes were proposed in this pull request? (Please outline the changes and how this PR fixes the issue.) ### Why are the changes needed? (Please clarify why the

Re: [I] [Subtask] Fix duplicate-code Pylint rule for client-python [gravitino]

2024-07-02 Thread via GitHub
ian910297 commented on issue #3767: URL: https://github.com/apache/gravitino/issues/3767#issuecomment-2204889789 I’m interested in working on this issue. I ran an initial pylint test. It looks like there are 20 issues to fix. I will submit them in batches. -- This is an automated

Re: [PR] [#3968] improvement(core): Disable KV entity store and optimize CI [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 closed pull request #3975: [#3968] improvement(core): Disable KV entity store and optimize CI URL: https://github.com/apache/gravitino/pull/3975 -- 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

Re: [PR] [#3896] feat(catalog-lakehouse-paimon): Support kerberos auth for Paimon FilesystemCatalog [gravitino]

2024-07-02 Thread via GitHub
caican00 commented on code in PR #3970: URL: https://github.com/apache/gravitino/pull/3970#discussion_r1663367080 ## catalogs/catalog-lakehouse-paimon/src/main/java/com/datastrato/gravitino/catalog/lakehouse/paimon/authentication/kerberos/FilesystemBackendProxy.java: ## @@ -0,0

Re: [PR] [#3371] feat(flink-connector): support basic table operation [gravitino]

2024-07-02 Thread via GitHub
FANNG1 commented on code in PR #3795: URL: https://github.com/apache/gravitino/pull/3795#discussion_r1663345010 ## flink-connector/src/main/java/com/datastrato/gravitino/flink/connector/catalog/BaseCatalog.java: ## @@ -145,32 +164,87 @@ public List listViews(String s) throws Da

[PR] [#3968] improvement(core): Disable KV entity store and optimize CI [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 opened a new pull request, #3975: URL: https://github.com/apache/gravitino/pull/3975 ### What changes were proposed in this pull request? - Disabling support for the KV entity store and adjusting the tests accordingly. - Change CI about backend option `jdbcBackend`

Re: [PR] [#3968] improvement(core): Disable KV entity store and optimize CI [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 closed pull request #3975: [#3968] improvement(core): Disable KV entity store and optimize CI URL: https://github.com/apache/gravitino/pull/3975 -- 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

[PR] [SIP] Split Iceberg REST service [gravitino]

2024-07-02 Thread via GitHub
FANNG1 opened a new pull request, #4005: URL: https://github.com/apache/gravitino/pull/4005 ### What changes were proposed in this pull request? (Please outline the changes and how this PR fixes the issue.) ### Why are the changes needed? (Please clarify why the c

Re: [PR] [SIP] Split Iceberg REST service [gravitino]

2024-07-02 Thread via GitHub
FANNG1 closed pull request #4005: [SIP] Split Iceberg REST service URL: https://github.com/apache/gravitino/pull/4005 -- 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 unsubsc

Re: [PR] [#1061] Improvement(server): Bad request may not response JSON content [gravitino]

2024-07-02 Thread via GitHub
featherchen commented on code in PR #3879: URL: https://github.com/apache/gravitino/pull/3879#discussion_r1663101659 ## server/src/test/java/com/datastrato/gravitino/server/web/rest/TestMetalakeOperations.java: ## @@ -178,6 +183,19 @@ public void testCreateMetalake() { Asse

Re: [PR] [#1061] Improvement(server): Bad request may not response JSON content [gravitino]

2024-07-02 Thread via GitHub
featherchen commented on code in PR #3879: URL: https://github.com/apache/gravitino/pull/3879#discussion_r1663101659 ## server/src/test/java/com/datastrato/gravitino/server/web/rest/TestMetalakeOperations.java: ## @@ -178,6 +183,19 @@ public void testCreateMetalake() { Asse

Re: [PR] [#1061] Improvement(server): Bad request may not response JSON content [gravitino]

2024-07-02 Thread via GitHub
featherchen commented on code in PR #3879: URL: https://github.com/apache/gravitino/pull/3879#discussion_r1663082736 ## server/src/test/java/com/datastrato/gravitino/server/web/rest/TestMetalakeOperations.java: ## @@ -178,6 +183,19 @@ public void testCreateMetalake() { Asse

[PR] [Minor] improvement(test): Fix the path of integration log is wrong. [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 opened a new pull request, #4029: URL: https://github.com/apache/gravitino/pull/4029 ### What changes were proposed in this pull request? Fix the path of the integration test is not as expected. ### Why are the changes needed? The path is not right.

Re: [PR] [#4018] feat(core): Add tag management logic for Tag System (Part 1) [gravitino]

2024-07-02 Thread via GitHub
jerryshao closed pull request #4019: [#4018] feat(core): Add tag management logic for Tag System (Part 1) URL: https://github.com/apache/gravitino/pull/4019 -- 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

Re: [PR] [Minor] improvement(test): Fix the path of integration log is wrong. [gravitino]

2024-07-02 Thread via GitHub
yuqi1129 closed pull request #4029: [Minor] improvement(test): Fix the path of integration log is wrong. URL: https://github.com/apache/gravitino/pull/4029 -- 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

[PR] [#4018] feat(core): Add tag management logic for Tag System (Part 1) [gravitino]

2024-07-02 Thread via GitHub
jerryshao opened a new pull request, #4019: URL: https://github.com/apache/gravitino/pull/4019 ### What changes were proposed in this pull request? This PR tracks the work of adding the core logics for tag management. ### Why are the changes needed? This is a part of work