Harnoor7 commented on code in PR #14503:
URL: https://github.com/apache/pinot/pull/14503#discussion_r1858234529


##########
pinot-controller/src/test/java/org/apache/pinot/controller/api/resources/PinotSegmentUploadDownloadRestletResourceTest.java:
##########
@@ -249,4 +267,58 @@ public void testValidateMultiPartForBatchSegmentUpload() {
     // validate – should not throw exception
     
PinotSegmentUploadDownloadRestletResource.validateMultiPartForBatchSegmentUpload(bodyParts);
   }
+
+  @Test
+  public void testUploadSegmentWithMissingTmpDir()

Review Comment:
   name of this test method was wrong have corrected it. This method was the 
one from where we received a runtime exception. Hence covering this method 
specifically.



-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to