Rich-T-kid commented on issue #10281:
URL: https://github.com/apache/arrow-rs/issues/10281#issuecomment-5014002097

   Thanks, this definitely helps! I think the crux of the three issues comes 
down to panic handling.
   It may be a good idea to split this into separate sub-issues. This issue 
could be renamed to something like panics break invariants in arrow-buffer 
crate, and then we create sub-issues covering each of the three issues listed. 
The issues seem fundamentally separate, so they can be worked on in parallel. 
It'd be nice to include the description/repro for each issue as well.
   
   
   as for the issues themselves did you have anything in mind to fix them?
   
   I believe issue 1 should be resolved with #10301, as it avoids panics when 
the mutex is poisoned 
   
   ```rust
   let mut guard = self
               .reservation
               .lock()
               .unwrap_or_else(std::sync::PoisonError::into_inner);
   ```


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to