[
https://issues.apache.org/jira/browse/AVRO-3475?focusedWorklogId=765701&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-765701
]
ASF GitHub Bot logged work on AVRO-3475:
----------------------------------------
Author: ASF GitHub Bot
Created on: 03/May/22 21:19
Start Date: 03/May/22 21:19
Worklog Time Spent: 10m
Work Description: martin-g commented on PR #1626:
URL: https://github.com/apache/avro/pull/1626#issuecomment-1116669108
Thank you, @zcsizmadia !
Issue Time Tracking
-------------------
Worklog Id: (was: 765701)
Remaining Estimate: 23h (was: 23h 10m)
Time Spent: 1h (was: 50m)
> Enforce time-millis and time-micros specification
> -------------------------------------------------
>
> Key: AVRO-3475
> URL: https://issues.apache.org/jira/browse/AVRO-3475
> Project: Apache Avro
> Issue Type: Improvement
> Components: csharp
> Reporter: Zoltan Csizmadia
> Priority: Minor
> Labels: pull-request-available
> Fix For: 1.12.0
>
> Original Estimate: 24h
> Time Spent: 1h
> Remaining Estimate: 23h
>
> During fixing AVRO-3471, I noticed that negative in/long values or negative
> TimeSpan values are allowed for time-millis and time-micros.
> [https://avro.apache.org/docs/current/spec.html] states that, it is supposed
> to be the number of us/ms after midnight 00:00:00. The following TimSpan
> values do not cause any overflow exceptions:
>
> "-00:00:00.001"
> "-999999.00:00:00"
>
> # This change should throw exceptions in cases where the code has not thrown
> exceptions before
> # OutOfRange exception message should be changed to `'time-millis/micros'
> value must be at least '00:00:00' and less than '1.00:00:00'.` to reflect the
> spec precisely
>
> It might make sense to introduce this change only in 1.12.0, because of all
> the above changes
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
