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
When constructing a range of IonTimestamps, we currently convert the input IonTimestamp values to a BigDecimal number of milliseconds since the epoch using IonTimestamp#getDecimalMillis:
This means that we erroneously allow before/after comparisons of timestamps with mixed precision. For example, we consider 2007-06-01T12:46 to be after2007-06-01T, which we cannot definitively conclude.
We will also ignore precisions greater than milliseconds, considering 2007-06-01T12:46:00.000111 and 2007-06-01T12:46:00.000444 to be equivalent.
The text was updated successfully, but these errors were encountered:
When constructing a range of
IonTimestamp
s, we currently convert the inputIonTimestamp
values to aBigDecimal
number of milliseconds since the epoch usingIonTimestamp#getDecimalMillis
:https://github.com/amzn/ion-schema-kotlin/blob/cde794e69206504227c8526580c28a56dedf18db/src/com/amazon/ionschema/internal/util/RangeIonTimestamp.kt#L39-L47
This means that we erroneously allow before/after comparisons of timestamps with mixed precision. For example, we consider
2007-06-01T12:46
to be after2007-06-01T
, which we cannot definitively conclude.We will also ignore precisions greater than milliseconds, considering
2007-06-01T12:46:00.000111
and2007-06-01T12:46:00.000444
to be equivalent.The text was updated successfully, but these errors were encountered: