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
Update timing fields using the suffix `_ms` to be in milliseconds.
Doing a quick grep through the repo I only found a few instances of this in the delete.rs file.
Closesdelta-io#2256
Update timing fields using the suffix `_ms` to be in milliseconds.
Doing a quick grep through the repo I only found a few instances of this in the delete.rs file.
Closesdelta-io#2256
# Description
Update timing fields using the suffix `_ms` to be in milliseconds.
Doing a quick grep through the repo I only found a few instances of this
in the delete.rs file.
# Related Issue(s)
Closes#2256
# Documentation
Environment
Delta-rs version: latest
Binding: rust
Environment:
Bug
What happened:
Sometimes duration fields suffixed with
_ms
report time as milliseconds (expected), but other times they are reported as microseconds.https://github.com/delta-io/delta-rs/blob/main/crates/core/src/operations/update.rs#L400
https://github.com/delta-io/delta-rs/blob/main/crates/core/src/operations/delete.rs#L204
What you expected to happen:
All fields using
_ms
should be in milliseconds._ms
-> milliseconds_us
-> microsecondsHow to reproduce it:
More details:
Microsecond granularity is likely not needed for any operations that traverse the network.
The text was updated successfully, but these errors were encountered: