Thakkar-Khushang commented on issue #10605:
URL: https://github.com/apache/gravitino/issues/10605#issuecomment-4164254252

   Hi, I'd like to work on this fix. Here's my plan:
   
   1. Extract a null-safe getDropPartitionNames(PartitionStatisticsDropRequest) 
helper in StatisticOperations.java — mirroring the existing getStatisticNames() 
and getPartitionNames() helpers already in the class.
   2. Replace the raw request.getDrops().stream() in the catch block with the 
safe helper.
   3. Add testDropPartitionStatisticsWithNullDrops() in 
TestStatisticOperations.java — sends {"drops":null}, expects HTTP 400 with 
IllegalArgumentException
   
   Will open a draft PR shortly.


-- 
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]

Reply via email to