[GitHub] [flink] FangYongs commented on a diff in pull request #22882: [FLINK-32370][runtime] Revert print error log in AbstractHandler and skip check exceptions in jdbc driver e2e tests which will check the test results itself

[via GitHub]

FangYongs commented on code in PR #22882:
URL: https://github.com/apache/flink/pull/22882#discussion_r1248421262


##########
flink-end-to-end-tests/run-nightly-tests.sh:
##########
@@ -185,7 +185,7 @@ function run_group_2 {
     run_test "Batch SQL end-to-end test using hybrid selective shuffle" 
"$END_TO_END_DIR/test-scripts/test_batch_sql.sh hybrid_selective"
     run_test "Streaming SQL end-to-end test using planner loader" 
"$END_TO_END_DIR/test-scripts/test_streaming_sql.sh" "skip_check_exceptions"
     run_test "Streaming SQL end-to-end test using planner with Scala version" 
"$END_TO_END_DIR/test-scripts/test_streaming_sql.sh scala-planner" 
"skip_check_exceptions"
-    run_test "Sql Jdbc Driver end-to-end test" 
"$END_TO_END_DIR/test-scripts/test_sql_jdbc_driver.sh"
+    run_test "Sql Jdbc Driver end-to-end test" 
"$END_TO_END_DIR/test-scripts/test_sql_jdbc_driver.sh skip_check_exceptions"

Review Comment:
   @zentol Yes, the test_sql_jdbc_driver will validate its query results in 
another files, all exceptions and error logs during this process are mainly for 
future issue analysis and are acceptable



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