justinmclean opened a new issue, #7683:
URL: https://github.com/apache/gravitino/issues/7683
### What would you like to be improved?
Fix fileset location resolution code so that catalog-level locations are
correctly read from the catalog properties instead of the schema properties.
In
/Users/justinmclean/gravitino/core/src/main/java/org/apache/gravitino/authorization/AuthorizationUtils.java
```
String schemaLocation =
schema.properties().get(FILESET_SCHEMA_LOCATION);
if (StringUtils.isNotBlank(schemaLocation)) {
locations.add(schemaLocation);
} else if
(catalogObj.properties.properties().containsKey(FILESET_CATALOG_LOCATION)) {
String catalogLocation =
schema.properties().get(FILESET_CATALOG_LOCATION);
if (StringUtils.isNotBlank(catalogLocation)) {
schemaLocation = catalogLocation + "/" + schema.name();
locations.add(schemaLocation);
}
```
The line after the else if is incorrectly using schema when it should use
catalogObj.
### How should we improve?
_No response_
--
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]
