wenlong88 commented on pull request #18215: URL: https://github.com/apache/flink/pull/18215#issuecomment-1004524026
hi, @slinkydeveloper thanks for the detailed review. I have a question on using Assertj: do we need transfer all of the tests into assertj when touching a test file? Existed tests in the file may cause conflicts in the testing framework, and we need to touch much more tests if doing the migration. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
