slfan1989 commented on PR #564: URL: https://github.com/apache/fluss-rust/pull/564#issuecomment-4561591029
> sorry, @slfan1989 just a very minor question, in this commit [3a41476](https://github.com/apache/fluss-rust/pull/564/commits/3a41476cbc21b7d696f3b4f91d50a34fb2a27b25) where you make changes, i notice you remove one tokio::time::sleep in line 221, but didn't replace with anything. If this case is not applicable for the helper, should we still keep it untouched? @charlesdong1991 Thanks for checking this. You are right — this sleep is after `flush()` and is not part of the table/partition readiness waits covered by the helper. I restored it to keep this PR scoped to replacing readiness-related sleeps only. -- 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]
