myfjdthink opened a new pull request, #18616: URL: https://github.com/apache/doris/pull/18616
# Proposed changes Issue Number: close #10452 ## Problem summary Describe your changes. ## Checklist(Required) - [ ] Does it affect the original behavior - [ ] Has unit tests been added - [ ] Has document been added or modified - [ ] Does it need to update dependencies - [x] Is this PR support rollback (If NO, please explain WHY) ## Further comments 我修改了 `org.apache.doris.planner.external.IcebergSplitter#getSplits` 方法，在 Iceberg 读取 metadata 获取到 data file path 后将 gs:// 前缀修改为 s3:// 前缀，这样 BE 才能顺利读取文件，但是我不知道为什么 aliyun oss 不需要这样做，这里存疑 I changed the `org.apache.doris.planner.external.IcebergSplitter#getSplits` method changes the gs:// prefix to s3:// prefix after Iceberg reads the metadata to get the data file path, so BE can read the file smoothly, but I don't know why Aliyun oss doesn't need to do this, which is questionable -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
