SEZ9 commented on issue #12750:
URL: https://github.com/apache/seatunnel/issues/12750#issuecomment-6160383475

   Thanks for the detailed write-up and for confirming the scope. The plan 
sounds good: wait for #12631 and #12688 to merge, rebase onto current `dev` 
without their pending changes, and keep the output read-only and evidence-based 
with explicit coverage gaps and separately reported prepare/commit pressure.
   
   One small ask for the PR description: please include a short example of the 
`--diagnose` output (queue ID, target vertex, measured window, bucket width, 
ratios) so reviewers can see how the candidate is presented without implying 
root cause. Please link the PR here once the dependencies have merged.
   
   <!-- streview-comment:1744 -->


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