abdessalems commented on PR #11727:
URL: https://github.com/apache/seatunnel/pull/11727#issuecomment-5473887244

   Thanks @DanielLeens — that's a really useful breakdown on the cacheMode
   default, and agreed on leaving it as a non-blocking follow-up rather than
   fixing it in both PRs.
   @davidzollo @zhangshenghang — this looks ready from the review side: Daniel's
   conclusion is "ready to merge", the unit-test legs are green on the current
   head, and the only red jobs are connector ITs unrelated to the two files this
   touches. Could one of you take the approve/merge step when you have a moment?
   


-- 
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