github-actions[bot] commented on code in PR #65520:
URL: https://github.com/apache/doris/pull/65520#discussion_r3568655585
##########
regression-test/suites/external_table_p0/tvf/test_file_tvf_s3.groovy:
##########
@@ -82,6 +82,21 @@ suite("test_file_tvf_s3", "p0") {
"format" = "parquet"
);
"""
+ for (int attempt = 1; attempt <= 2; attempt++) {
+ try {
+ order_qt_s3_tvf tvfSql
Review Comment:
The retry is wrapped around `order_qt_s3_tvf`, but that call has already
advanced the regression output iterator before it executes the SQL. In
verification mode `quickRunTest` reserves the expected `s3_tvf` block with
`context.getOutputIterator().next()` before the JDBC call, and a caught timeout
does not rewind it; the next attempt drains that block and compares against the
following one. This suite has 38 `file_s3_tvf` calls and exactly 38 `-- !s3_tvf
--` blocks, so one transient timeout can turn into a later `Missing output
block for tag 's3_tvf'` even if the retry succeeds. The retry needs to happen
below the quick-test output reservation boundary, or otherwise execute/retry
the query before recording/comparing exactly one output block.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
