morningman opened a new pull request, #65150:
URL: https://github.com/apache/doris/pull/65150
## Proposed changes
Cherry-pick #64562 to branch-4.0.
`regression-test/suites/load_p2/tvf/test_s3_tvf.groovy` set both a
virtual-host style URI and `use_path_style=true` on six active S3 TVF
attributes. The two settings conflict: Aliyun OSS rejects path-style
access for this bucket with `HTTP 403 SecondLevelDomainForbidden`
(`Please use virtual hosted style to access`) after the P2 environment
switched to the Aliyun internal Beijing endpoint. This removes the six
active `use_path_style=true` settings whose URI is already virtual-host
style, so the SDK uses the addressing style OSS requires.
Test-only change.
### Backport note
The upstream commit also rewrote the `catch` block failure path to
`assertTrue(...)`. branch-4.0 already has functionally-equivalent
handling of that block from #64997
(`if (!attribute.expectFiled) { ...; throw ex }`, which rethrows the
original exception), so the one conflicting hunk was resolved by keeping
the existing branch-4.0 code. Pass/fail behavior is identical; only the
failure message differs. The six `use_path_style` removals — the actual
fix — apply unchanged.
Cherry-picked from #64562
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
