github-actions[bot] commented on code in PR #69591:
URL: https://github.com/apache/doris/pull/69591#discussion_r4374692531
##########
regression-test/suites/datatype_p0/uuid/test_uuid_export.groovy:
##########
@@ -19,25 +19,12 @@
// UUID annotations survive nested file export and direct reload into native
UUID columns.
suite("test_uuid_export", "p0,external") {
- String localPath = context.config.otherConfigs.get("uuidLocalExportPath")
- boolean localMode = localPath != null
String token = UUID.randomUUID().toString()
- String outputPath
- String sinkProperties
- String sourceProperties
- if (localMode) {
- outputPath = "file://${localPath}_${token}_"
- sinkProperties = ""
- def backends = sql "SHOW BACKENDS"
- sourceProperties = "\"backend_id\"=\"${backends[0][0]}\""
- } else {
- outputPath =
"s3://${context.config.otherConfigs.get('s3BucketName')}/uuid_file_export/${token}_"
- sinkProperties = """
+ String outputPath =
"s3://${context.config.otherConfigs.get('s3BucketName')}/uuid_file_export/${token}_"
+ String sinkProperties = """
Review Comment:
[P2] Preserve Azure routing in the UUID export migration
For supported `s3Source="azure"` with the default endpoint and previously
valid `uuidLocalExportPath`/`uuidLocalTvfPath` mappings, removing `localMode`
now forces this property block. It forwards `azure-endpoint` verbatim and omits
`provider`, so `WITH S3` and the readback TVF bind an S3-compatible provider at
the placeholder host instead of the Azure account; the export fails before its
completion/readback assertions. Resolve the Azure placeholder and pass the
matching provider to both operations. This is a distinct Doris EXPORT/reload
path from the fixture SDK uploaders covered by the existing Azure threads; the
already-remote fallback without those local mappings was previously deficient.
--
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]
