Aireed commented on PR #3726:
URL: https://github.com/apache/amoro/pull/3726#issuecomment-3198948606

   > @Aireed thanks for your contribution. I left one inline comment. Please 
let me know what you think about it.
   
   Thansk for your review.
   TableRuntimeRefreshExecutor indeed involves more than just executing 
evaluations (evaluator); it also encompasses other logic, such as updating 
properties. Therefore, it cannot simply be regarded as an execute interval.
   
   i discussed with @zhoujinsong  offline, we agreed that the evaluator should 
only be triggered when the optimize state is idle, as this makes it easier to 
understand
   
   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]

Reply via email to