feat: refactor use of Error::io

[lichuang]

fix #5601

[wjones127]

Thanks for starting this. I have several suggestions.

Feel free to suggest that we add or change an error type. For example, maybe we want to make a more general NotFound error that's separate from FileNotFound error.

[lichuang]

@wjones127 i have resolved your suggestions, please review this pr again:)

[wjones127]

@lichuang Generally looks good. Can you get the code compiling and tests passing? Then I can do a final review.

[lichuang]

@wjones127 I have fixed compiling error, please run workflow, and review it again.

[codecov]

Codecov Report

❌ Patch coverage is 15.23810% with 89 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
rust/lance/src/dataset/hash_joiner.rs	0.00%	14 Missing and 2 partials ⚠️
rust/lance/src/dataset/blob.rs	0.00%	12 Missing ⚠️
rust/lance-core/src/error.rs	35.29%	11 Missing ⚠️
rust/lance-table/src/io/commit/dynamodb.rs	0.00%	10 Missing ⚠️
rust/lance/src/dataset/fragment.rs	35.71%	8 Missing and 1 partial ⚠️
...lance-index/src/scalar/inverted/tokenizer/jieba.rs	0.00%	6 Missing ⚠️
rust/lance-datafusion/src/sql.rs	0.00%	2 Missing and 1 partial ⚠️
rust/lance-file/src/previous/writer/mod.rs	0.00%	3 Missing ⚠️
rust/lance-file/src/reader.rs	0.00%	2 Missing ⚠️
...nce-index/src/scalar/inverted/tokenizer/lindera.rs	0.00%	2 Missing ⚠️
... and 12 more

📢 Thoughts on this report? Let us know!

[lichuang]

@wjones127 I have fixed the test case, let's run the workflows again.

[lichuang]

@wjones127 I have fixed the test case, let's run the workflows again.

[wjones127]

Great work! Thanks for doing this.