You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "zjregee (via GitHub)" <gi...@apache.org> on 2024/11/12 12:18:14 UTC

[PR] [WIP] OVFS: Modify error handling [opendal]

zjregee opened a new pull request, #5312:
URL: https://github.com/apache/opendal/pull/5312

   # Which issue does this PR close?
   None.
   
   # Rationale for this change
   Previously, error handling in virtiofs_opendal was a bit messy, and many errors were not used correctly. I hope that error handling in virtiofs_opendal can be unified and standardized in this PR to lay a good foundation for future iterations.
   
   In general, I think virtiofs_opendal errors can be divided into two categories, errors that meet expectations (from opendal or basic checks, such as files that don't exist), and errors that don't meet expectations (from programming or the virtiofs framework, such as requests that can't be resolved). Both types of errors can be wrapped as io::Error or raw_os_error, returned to the caller (virtiofs framework, user, or vfs), and logged in the terminal.
   
   # What changes are included in this PR?
   Refactor the current error handling in OVFS.
   
   # Are there any user-facing changes?
   None.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [WIP] OVFS: Modify error handling [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo commented on PR #5312:
URL: https://github.com/apache/opendal/pull/5312#issuecomment-2783049236

   Hi, @zjregee, thank you for working on this! Welcome to start a discussion about the future of ovfs project.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [WIP] OVFS: Modify error handling [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo closed pull request #5312: [WIP] OVFS: Modify error handling
URL: https://github.com/apache/opendal/pull/5312


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org