Aireed commented on PR #3657: URL: https://github.com/apache/amoro/pull/3657#issuecomment-3050954898
`Uniformly change the default value of the execution interval of the above executor to 1d` Thanks for your contribution first, and I have a concern. It is inappropriate to unify the cleanup execution interval to 1 day for all tasks because features like expiresnapshot work in conjunction with table-level configurations (snapshot.keep.duration, default to 12hours). If the execution interval exceeds the table's configured retention period, the expected results will not be achieved. WDYT -- 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]
