jerryshao commented on code in PR #4005:
URL: https://github.com/apache/gravitino/pull/4005#discussion_r1677282945
##########
.github/workflows/backend-integration-test.yml:
##########
@@ -37,6 +37,8 @@ jobs:
- gradle/**
- integration-test/**
- integration-test-common/**
+ - iceberg-common/**
Review Comment:
"extensions" feels like giving user a feeling that iceberg support is not a
core feature in Gravitino. But actually not, Iceberg support is a core feature
in Gravitino, so using "extensions" is not proper. I would suggest to use
"iceberg" directly, add when you build a package, directly put
iceberg-rest-server as a folder, not in the "extensions".
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
