This is an automated email from the ASF dual-hosted git repository.
jmclean 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 b6cfe91efc [#10224] Use LIST operation type in fileset listFiles
exception handling (#10225)
b6cfe91efc is described below
commit b6cfe91efcd362ffbf926e19ddd4d7578b0a8d43
Author: YuF-9468 <[email protected]>
AuthorDate: Fri Mar 6 12:54:07 2026 +0800
[#10224] Use LIST operation type in fileset listFiles exception handling
(#10225)
## Summary
Fixes #10224.
`FilesetOperations#listFiles` previously handled exceptions with
`OperationType.LOAD`, which is incorrect for a list-files endpoint. This
PR switches it to `OperationType.LIST` so error handling semantics match
the operation.
## Changes
- In
`server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java`,
changed the exception handling operation type in `listFiles(...)` from
`LOAD` to `LIST`.
## Validation
- Code change is minimal and scoped to exception mapping in one
endpoint.
Co-authored-by: YuF-9468 <[email protected]>
---
.../java/org/apache/gravitino/server/web/rest/FilesetOperations.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java
b/server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java
index 45a000ede8..5dcc3833c6 100644
---
a/server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java
+++
b/server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java
@@ -264,7 +264,7 @@ public class FilesetOperations {
return response;
});
} catch (Exception e) {
- return ExceptionHandlers.handleFilesetException(OperationType.LOAD,
fileset, schema, e);
+ return ExceptionHandlers.handleFilesetException(OperationType.LIST,
fileset, schema, e);
}
}