jackylee-ch opened a new pull request, #10190: URL: https://github.com/apache/paimon/pull/10190
## Purpose Reading a Vortex file on OSS, `to_vortex_specified` builds the virtual-hosted endpoint as `https://<bucket>.<fs.oss.endpoint>`. If `fs.oss.endpoint` was configured with a scheme — e.g. `https://oss-cn-hangzhou.aliyuncs.com`, a common form — the result is the malformed `https://<bucket>.https://oss-cn-hangzhou.aliyuncs.com` and the read fails. `lance_utils` already strips the scheme for exactly this reason; `vortex_utils` did not. ## Change - Strip a leading `http://` / `https://` from `fs.oss.endpoint` before composing the endpoint, mirroring `lance_utils`. A scheme-less endpoint is unchanged. ## Tests - A scheme-carrying endpoint no longer yields the doubled scheme; a plain endpoint, the `oss://`→`s3://` path rewrite and the security token are unchanged. Verified non-vacuous (without the fix the scheme case produces `https://<bucket>.https://<host>`). Written with Claude Code; verification is mine. -- 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]
