Skip to content

Conversation

@fMeow
Copy link
Contributor

@fMeow fMeow commented Nov 28, 2025

This PR close #2140.

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@github-actions github-actions bot added the enhancement New feature or request label Nov 28, 2025
@yanghua yanghua requested a review from Xuanwo November 28, 2025 07:52
@codecov
Copy link

codecov bot commented Nov 28, 2025

Codecov Report

❌ Patch coverage is 6.66667% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rust/lance-io/src/object_store/providers/hdfs.rs 0.00% 14 Missing ⚠️

📢 Thoughts on this report? Let us know!

@fMeow
Copy link
Contributor Author

fMeow commented Nov 28, 2025

@Xuanwo I change the underlying storage backend from hdfs native object store to opendal.

However, I found the version of opendal in lance obselete. In opendal 0.55, it bump the version of hdfs-native to 0.13. I need a critical feature that introduced in hdfs-natie 0.12.

Any idea when will lance upgrade the version of opendal dependency to 0.55?

@Xuanwo
Copy link
Collaborator

Xuanwo commented Nov 28, 2025

Any idea when will lance upgrade the version of opendal dependency to 0.55?

It's ok to upgrade it for now. Would you like to start a PR for this?

@fMeow
Copy link
Contributor Author

fMeow commented Nov 28, 2025

sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Does lance support hdfs access?

2 participants