carols10cents opened a new pull request #1000:
URL: https://github.com/apache/arrow-rs/pull/1000


   # Which issue does this PR close?
   
   Closes #999.
   
   # Rationale for this change
   
   In the case of chrono, making sure these crates aren't turning on a feature 
name that's deprecated and will be removed eventually, and that they're turning 
on the minimal number of features actually used to minimize the features 
projects depending on these crates will be forced to turn on.
   
   In the case of indexmap, by explicitly enabling the "std" feature since 
arrow isn't targetting no_std environments, this might save a bit of build time 
done by the indexmap crate at build time to try to automatically detect whether 
the target has std available or not.
   
   # What changes are included in this PR?
   
   Adjustments to the feature sets of chrono and indexmap
   
   # Are there any user-facing changes?
   
   If projects depending on arrow/parquet are making use of the coincidence 
that arrow/parquet turns on chrono features, this might mean they have to turn 
on the features themselves, but that would be a bug in that project's use of 
chrono.
   
   I can't imagine the indexmap change would affect anyone except for maybe 
building slightly faster :)


-- 
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