You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Would it make sense to re-export NanoSeconds etc since, technically, they're used in TimeUnit which is re-exported in parquet::basic? (or provide helper constructors like TimeUnit::nanoseconds(), or even TimestampType::nanoseconds(bool) to reduce the boilerplate even further)
// Yes, you can do parquet::basic::TimeUnit::NANOS(Default::default()) - which clippy would complain about.
The text was updated successfully, but these errors were encountered:
Minor/cosmetic/consistency request:
Would it make sense to re-export
NanoSeconds
etc since, technically, they're used inTimeUnit
which is re-exported inparquet::basic
? (or provide helper constructors likeTimeUnit::nanoseconds()
, or evenTimestampType::nanoseconds(bool)
to reduce the boilerplate even further)// Yes, you can do
parquet::basic::TimeUnit::NANOS(Default::default())
- which clippy would complain about.The text was updated successfully, but these errors were encountered: