Uh oh!
There was an error while loading. Please reload this page.
[Enhance](fe) Iceberg table in HMS catalog supports broker scan - #28107
Conversation
WinkerDu
commented
Dec 7, 2023
run buildall |
WinkerDu
commented
Dec 7, 2023
@morningman@chenlinzhong@luozenglin Please have a review, thx |
doris-robot
commented
Dec 7, 2023
(From new machine)TeamCity pipeline, clickbench performance test result: |
doris-robot
commented
Dec 7, 2023
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
WinkerDu
commented
Dec 7, 2023
run p0 |
| if (props.containsKey(HMSExternalCatalog.BIND_BROKER_NAME)) { | ||
| // Set Iceberg FileIO implementation as `IcebergBrokerIO` when Catalog binding broker is specified. | ||
| props.put("io-impl", "org.apache.doris.datasource.iceberg.broker.IcebergBrokerIO"); |
There was a problem hiding this comment.
I think this property should be added when creating catalog?
There was a problem hiding this comment.
OK, I'll edit the docs to illustrate the usage of this property
WinkerDu
commented
Jan 1, 2024
run buildall |
181af7b to
1479075CompareWinkerDu
commented
Jan 1, 2024
run buildall |
doris-robot
commented
Jan 1, 2024
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G', run with scripts in https://github.com/apache/doris/tree/master/tools/tpch-tools |
doris-robot
commented
Jan 1, 2024
TPC-DS test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G', run with scripts in https://github.com/apache/doris/tree/master/tools/tpcds-tools |
doris-robot
commented
Jan 1, 2024
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G', run with scripts in https://github.com/apache/doris/tree/master/tools/tpch-tools |
doris-robot
commented
Jan 1, 2024
(From new machine)TeamCity pipeline, clickbench performance test result: |
WinkerDu
commented
Jan 2, 2024
run p0 |
1479075 to
57432ebComparemorningman
commented
Jan 2, 2024
run buildall |
doris-robot
commented
Jan 2, 2024
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G', run with scripts in https://github.com/apache/doris/tree/master/tools/tpch-tools |
doris-robot
commented
Jan 2, 2024
(From new machine)TeamCity pipeline, clickbench performance test result: |
PR approved by anyone and no changes requested. |
…he#28107) My organization uses HMS catalog to accelerate Lake query. Sine we have custom distributed file system and hard to integrate to FE / BE, we introduce HMS Catalog broker scan support (apache#24830) and implement custom distributed file system adaption in broker. We want to expand the scope of use to Iceberg table scan in HMS Catalog. This PR introduces broker-scan-related `IcebergBrokerIO`, `BrokerInputFile`, `BrokerInputStream` for Iceberg table scan
…he#28107) My organization uses HMS catalog to accelerate Lake query. Sine we have custom distributed file system and hard to integrate to FE / BE, we introduce HMS Catalog broker scan support (apache#24830) and implement custom distributed file system adaption in broker. We want to expand the scope of use to Iceberg table scan in HMS Catalog. This PR introduces broker-scan-related `IcebergBrokerIO`, `BrokerInputFile`, `BrokerInputStream` for Iceberg table scan
…he#28107) My organization uses HMS catalog to accelerate Lake query. Sine we have custom distributed file system and hard to integrate to FE / BE, we introduce HMS Catalog broker scan support (apache#24830) and implement custom distributed file system adaption in broker. We want to expand the scope of use to Iceberg table scan in HMS Catalog. This PR introduces broker-scan-related `IcebergBrokerIO`, `BrokerInputFile`, `BrokerInputStream` for Iceberg table scan
Proposed changes
Issue Number: close #xxx
My organization uses HMS catalog to accelerate Lake query. Sine we have custom distributed file system and hard to integrate to FE / BE, we introduce HMS Catalog broker scan support (#24830) and implement custom distributed file system adaption in broker.
We want to expand the scope of use to Iceberg table scan in HMS Catalog. This PR introduces broker-scan-related
IcebergBrokerIO,BrokerInputFile,BrokerInputStreamfor Iceberg table scanFurther comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...