ryzhyk opened a new pull request, #14863:
URL: https://github.com/apache/datafusion/pull/14863

   ## Which issue does this PR close?
   
   - Closes #14862
   
   ## Rationale for this change
   
   When datafusion is used in a workspace that enables the `rkyv-64` feature in 
the `chrono` crate, this triggered a Rust compilation error:
   
   ```
   error[E0277]: can't compare `Option<&std::string::String>` with `Option<&mut 
std::string::String>`.
   ```
   
   The root cause of the error is incorrect type unification in the Rust 
compiler, as explained in https://github.com/rkyv/rkyv/issues/434.
   
   ## What changes are included in this PR?
   
   The workaround pushes the compiler in the right direction by converting the 
mutable reference to an immutable one manually.
   
   ## Are these changes tested?
   
   I don't think this requires additional tests.
   
   ## Are there any user-facing changes?
   
   no


-- 
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: github-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to