mcvsubbu commented on issue #7082:
URL: 
https://github.com/apache/incubator-pinot/issues/7082#issuecomment-867084996


   According to this
   
https://github.com/apache/incubator-pinot/blob/master/pinot-spi/src/main/java/org/apache/pinot/spi/filesystem/PinotFS.java#L78
   DeletedSegments directory will be created on demand, when the first segment 
is deleted.
   
   @kkrugler  the call to remove aged deleted segments, is invoked from 
retention manager. If you never ever delete anything from any table in your 
cluster, you can (for now) disable RetentionManager.  Alternatively, create a 
test table with a segment and delete it so that the parent dir is created on 
deletion. This will avoid your warnings and alerts.
   
   It seems to be that HadoopPinotFS is miscoded. The method isDirectory() is a 
boolean, and if it does not exist or is not a directory, it should return 
false. IMO the fix is in the underlying FS implementation.


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

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to