gaborgsomogyi commented on issue #23850: [SPARK-26949][SS] Prevent 'purge' to 
remove needed batch files in CompactibleFileStreamLog
URL: https://github.com/apache/spark/pull/23850#issuecomment-466094815
 
 
   As long as the functionality is not needed I'm fine with the 
`UnsupportedOperationException`.
   I've double checked and not used anywhere.
   
   As a potential user of this or any kind of function I expect to do what I 
want without any internal thinking. Because of this option 1 and 2 are having 
side effects from my perspective.
   
   If it would be really required I think the compacted file can be 
deserialized => remove the batches => serialized again with proper content 
(maybe you meant the same by `maintain logs`).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to