jtarun opened a new issue #6868: URL: https://github.com/apache/incubator-pinot/issues/6868
setDirectoryPermissions() in SegmentV1V2ToV3FormatConverter ignores the exception as it is unable to set the permissions in Windows. This leaves the segment in bad state, while the job outputDir is still zipped and uploaded to Pinot by the ingestion job. Two fixes are required here - 1) Throw the exception when segment creation fails at any stage. This will at least ensure that ingestion job throws exception when ingestion jobs are run on non-Posix compliant systems. 2) Make this permission setting optional based on some configuration to make it compatible to Windows. Or preferrably use a platform independent way of setting permissions on the directory. -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
