zhangwl9 commented on PR #3657: URL: https://github.com/apache/amoro/pull/3657#issuecomment-3055723354
> 表级配置（snapshot.keep.duration，默认为 12 小时） > Could you explain why it's necessary to combine these four parts into one? As far as I know, they were originally a single executor and later split into multiple ones. Also, modifying configuration options is generally discouraged, as it can lead to incompatibility with old versions.你能解释为什么需要将这四个部分合并成一个吗？据我所知，它们原本是一个执行器，后来才拆分成多个。此外，修改配置选项通常是不被推荐的，因为它可能导致与旧版本的兼容性问题。 > > If multiple maintenance tasks are handled by the same thread, a long-running or faulty cleanup on one table could block or delay the cleanup of other tables.如果多个维护任务由同一个线程处理，一个表上的长时间运行或故障清理可能会阻塞或延迟其他表的清理。 @xxubai Before this pr, the execution of all four executors will call 'PeriodicTableScheduler#loadTable', which will result in the same table object being loaded multiple times, the metastore generating more connections, and the memory footprint spiking, leading to increased GC pressure, so I want to reduce the number of times 'PeriodicTableScheduler#loadTable' is loaded -- 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]
