opendal icon indicating copy to clipboard operation
opendal copied to clipboard

feat(services/hdfs): refactor HdfsBackend to use HdfsCore for client operations

Open uruemu opened this issue 8 months ago • 4 comments

Which issue does this PR close?

Addresses #5702: Split service logic to backend and core modules

Part of #5702.

Rationale for this change

What changes are included in this PR?

This pull request refactors the GridFsBackend by introducing a new GridFsBackendCore struct to hosting connection functionality and state. GridFsCore implements the kv::Adapters trait, allowing the removal of the Adapter struct

Are there any user-facing changes?

uruemu avatar Apr 16 '25 01:04 uruemu

Hi, seems all hdfs tests are failed: https://github.com/apache/opendal/actions/runs/14482506154/job/40622110318?pr=6033

Xuanwo avatar Apr 18 '25 10:04 Xuanwo

@Xuanwo yes, just took some time to fix that. The PR is ready for review

uruemu avatar Apr 21 '25 01:04 uruemu

@erickguan Will come back to this PR this week. Sorry, I've been a bit busy

uruemu avatar Apr 30 '25 18:04 uruemu

@uruemu Thanks for working on this and your schedule!

erickguan avatar Apr 30 '25 19:04 erickguan