OIiveirra commented on code in PR #69591:
URL: https://github.com/apache/doris/pull/69591#discussion_r4374255007
##########
regression-test/suites/query_p0/test_array_with_single_row_max_than_uint16.groovy:
##########
@@ -51,11 +36,24 @@ suite("test_array_with_single_row_max_than_uint16",
"query") {
)
"""
// prepare data
- qt_sql """
- insert into ${testTable} select * from local(
- "file_path" = "${outFilePath}/arr_max.orc",
- "backend_id" = "${be_id}",
- "format" = "orc");"""
+ String bucket = getS3BucketName()
+ String key =
"regression/local_tvf_migration/${UUID.randomUUID()}/arr_max.orc"
+ def s3Client = getS3Client()
+ s3Client.putObject(bucket, key, new File(context.config.dataPath,
"query_p0/arr_max.orc"))
Review Comment:
Fixed in 30ce3e745781086aaf214e820393d4c1d04b13e1; the same adaptation is
applied to the other affected branch PRs. Both ORC fixture loads now use the
already-declared Azure Blob SDK for Azure upload and cleanup, and retain the
AWS client for S3-compatible providers. The TVF receives the provider and
resolved endpoint; the Azure placeholder maps to the account endpoint, while
explicit endpoints are preserved. Original ORC files, table definitions and all
count/value assertions are unchanged. All eight branch scripts compiled with
the existing SDK dependencies. Twelve loopback tests using the real Azure/S3
SDKs verified byte-identical uploads, endpoint selection and cleanup after
successful or injected-failing SQL calls. SQL was intercepted in that probe; no
Doris runtime regression or live-cloud validation is claimed. No production or
framework dependency changes were made.
--
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]
