(gravitino) branch branch-1.1 updated: [Cherry-pick to branch-1.1] [#10175]improvement(PartitionOperations): include partition path parameter in getPartition error handling (#10388) (#10484)

[jshao]

This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch branch-1.1
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-1.1 by this push:
     new d8d2c5e68b [Cherry-pick to branch-1.1] 
[#10175]improvement(PartitionOperations): include partition path parameter in 
getPartition error handling (#10388) (#10484)
d8d2c5e68b is described below

commit d8d2c5e68b1e775d0aa83aa2b942d0790abaf792
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Mon Mar 23 09:59:44 2026 +0800

    [Cherry-pick to branch-1.1] [#10175]improvement(PartitionOperations): 
include partition path parameter in getPartition error handling (#10388) 
(#10484)
    
    **Cherry-pick Information:**
    - Original commit: 5a3c59534d0e49fd11ce0fc56530132d8fe0a307
    - Target branch: `branch-1.1`
    - Status: ✅ Clean cherry-pick (no conflicts)
    
    Co-authored-by: Sambhavi Pandey 
<48976443+pandeysam...@users.noreply.github.com>
---
 .../server/web/rest/PartitionOperations.java          |  2 +-
 .../server/web/rest/TestPartitionOperations.java      | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git 
a/server/src/main/java/org/apache/gravitino/server/web/rest/PartitionOperations.java
 
b/server/src/main/java/org/apache/gravitino/server/web/rest/PartitionOperations.java
index 654f401e33..d817e3e48d 100644
--- 
a/server/src/main/java/org/apache/gravitino/server/web/rest/PartitionOperations.java
+++ 
b/server/src/main/java/org/apache/gravitino/server/web/rest/PartitionOperations.java
@@ -162,7 +162,7 @@ public class PartitionOperations {
             return response;
           });
     } catch (Exception e) {
-      return ExceptionHandlers.handlePartitionException(OperationType.GET, "", 
table, e);
+      return ExceptionHandlers.handlePartitionException(OperationType.GET, 
partition, table, e);
     }
   }
 
diff --git 
a/server/src/test/java/org/apache/gravitino/server/web/rest/TestPartitionOperations.java
 
b/server/src/test/java/org/apache/gravitino/server/web/rest/TestPartitionOperations.java
index ea3dd7ec61..2c27c9c7db 100644
--- 
a/server/src/test/java/org/apache/gravitino/server/web/rest/TestPartitionOperations.java
+++ 
b/server/src/test/java/org/apache/gravitino/server/web/rest/TestPartitionOperations.java
@@ -249,6 +249,25 @@ public class TestPartitionOperations extends JerseyTest {
     Assertions.assertTrue(errorResp2.getMessage().contains("p3"));
   }
 
+  @Test
+  public void testGetPartitionExceptionMessageContainsPartitionName() {
+    doThrow(new NoSuchPartitionException("missing partition"))
+        .when(dispatcher)
+        .getPartition(any(), any());
+
+    Response resp =
+        target(partitionPath(metalake, catalog, schema, table) + "p3")
+            .request(MediaType.APPLICATION_JSON_TYPE)
+            .accept("application/vnd.gravitino.v1+json")
+            .get();
+
+    Assertions.assertEquals(Response.Status.NOT_FOUND.getStatusCode(), 
resp.getStatus());
+
+    ErrorResponse errorResponse = resp.readEntity(ErrorResponse.class);
+    Assertions.assertEquals(ErrorConstants.NOT_FOUND_CODE, 
errorResponse.getCode());
+    Assertions.assertTrue(errorResponse.getMessage().contains("[p3]"));
+  }
+
   @Test
   public void testAddPartition() {
     when(dispatcher.addPartition(any(), any())).thenReturn(partition1);