Skip to content

Comments

refactor: Migrate WebHDFS service to context based http client#5893

Merged
Xuanwo merged 2 commits intoapache:mainfrom
miroim:migrate-webhdfs-context
Mar 27, 2025
Merged

refactor: Migrate WebHDFS service to context based http client#5893
Xuanwo merged 2 commits intoapache:mainfrom
miroim:migrate-webhdfs-context

Conversation

@miroim
Copy link
Member

@miroim miroim commented Mar 27, 2025

Which issue does this PR close?

Closes #5677
Part of #5702

Rationale for this change

What changes are included in this PR?

  • This pull request refactors the WebhdfsBackend by introducing a new WebhdfsCore struct to encapsulate the core functionality and state. The changes aim to improve code organization and maintainability.
  • Updated methods to use AccessorInfo for HTTP requests.

Are there any user-facing changes?

@miroim miroim requested a review from Xuanwo as a code owner March 27, 2025 07:16
@github-actions github-actions bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Mar 27, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, thank you for this!

@Xuanwo Xuanwo merged commit d724d9d into apache:main Mar 27, 2025
242 checks passed
@Xuanwo Xuanwo mentioned this pull request Mar 27, 2025
31 tasks
@miroim miroim deleted the migrate-webhdfs-context branch March 27, 2025 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate all services to context based

2 participants