XComp commented on a change in pull request #18806:
URL: https://github.com/apache/flink/pull/18806#discussion_r808953718



##########
File path: flink-end-to-end-tests/test-scripts/test_pyflink.sh
##########
@@ -223,13 +223,23 @@ function read_msg_from_kafka {
 }
 
 function cat_jm_logs {
-     local log_file_name=${3:-standalonesession}
-     cat $FLINK_LOG_DIR/*$log_file_name*.log
+  cat_logs_in_order "standalonesession"
 }
 
 function cat_tm_logs {
-       local logfile="$FLINK_LOG_DIR/flink*taskexecutor*log"

Review comment:
       @zentol When preparing the backports I noticed that I missed a few 
occurrences because I grepped for ".log" instead of "log" when looking for code 
snippets to change in any `*sh` file. Like this one above




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


Reply via email to