Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-24 Thread Antoine Pitrou
Hello, The vote carries with 8 +1 votes (of which 4 binding) and no -1 votes. I will submit a PR soon to enhance Schema.fbs. Regards Antoine. Le 19/08/2021 à 18:20, Antoine Pitrou a écrit : Hello, I would like to propose clarifying the allowed value range for the Time types. Specifical

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-23 Thread Fan Liya
+1 On Fri, Aug 20, 2021 at 11:37 PM Micah Kornfield wrote: > +1 (binding) > > On Fri, Aug 20, 2021 at 7:46 AM Keith Kraus > wrote: > > > +1 (non-binding) > > > > On Fri, Aug 20, 2021 at 9:49 AM Rok Mihevc wrote: > > > > > +1 (non-binding) > > > > > > On Fri, Aug 20, 2021 at 3:46 PM Jorge Cardo

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-20 Thread Micah Kornfield
+1 (binding) On Fri, Aug 20, 2021 at 7:46 AM Keith Kraus wrote: > +1 (non-binding) > > On Fri, Aug 20, 2021 at 9:49 AM Rok Mihevc wrote: > > > +1 (non-binding) > > > > On Fri, Aug 20, 2021 at 3:46 PM Jorge Cardoso Leitão > > wrote: > > > > > > +1 > > > > > > On Fri, Aug 20, 2021 at 2:43 PM Dav

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-20 Thread Keith Kraus
+1 (non-binding) On Fri, Aug 20, 2021 at 9:49 AM Rok Mihevc wrote: > +1 (non-binding) > > On Fri, Aug 20, 2021 at 3:46 PM Jorge Cardoso Leitão > wrote: > > > > +1 > > > > On Fri, Aug 20, 2021 at 2:43 PM David Li wrote: > > > > > +1 > > > > > > On Thu, Aug 19, 2021, at 18:33, Weston Pace wrote:

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-20 Thread Rok Mihevc
+1 (non-binding) On Fri, Aug 20, 2021 at 3:46 PM Jorge Cardoso Leitão wrote: > > +1 > > On Fri, Aug 20, 2021 at 2:43 PM David Li wrote: > > > +1 > > > > On Thu, Aug 19, 2021, at 18:33, Weston Pace wrote: > > > +1 > > > > > > On Thu, Aug 19, 2021 at 9:18 AM Wes McKinney > > wrote: > > > > > > >

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-20 Thread Jorge Cardoso Leitão
+1 On Fri, Aug 20, 2021 at 2:43 PM David Li wrote: > +1 > > On Thu, Aug 19, 2021, at 18:33, Weston Pace wrote: > > +1 > > > > On Thu, Aug 19, 2021 at 9:18 AM Wes McKinney > wrote: > > > > > > +1 > > > > > > On Thu, Aug 19, 2021 at 6:20 PM Antoine Pitrou > wrote: > > > > > > > > > > > > Hello,

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-20 Thread David Li
+1 On Thu, Aug 19, 2021, at 18:33, Weston Pace wrote: > +1 > > On Thu, Aug 19, 2021 at 9:18 AM Wes McKinney wrote: > > > > +1 > > > > On Thu, Aug 19, 2021 at 6:20 PM Antoine Pitrou wrote: > > > > > > > > > Hello, > > > > > > I would like to propose clarifying the allowed value range for the Tim

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-19 Thread Weston Pace
+1 On Thu, Aug 19, 2021 at 9:18 AM Wes McKinney wrote: > > +1 > > On Thu, Aug 19, 2021 at 6:20 PM Antoine Pitrou wrote: > > > > > > Hello, > > > > I would like to propose clarifying the allowed value range for the Time > > types. Specifically, I would propose that: > > > > 1) allowed values fal

Re: [VOTE][Format] Clarify allowed value range for the Time types

2021-08-19 Thread Wes McKinney
+1 On Thu, Aug 19, 2021 at 6:20 PM Antoine Pitrou wrote: > > > Hello, > > I would like to propose clarifying the allowed value range for the Time > types. Specifically, I would propose that: > > 1) allowed values fall between 0 (included) and 86400 seconds > (excluded), adjusted for the time uni

[VOTE][Format] Clarify allowed value range for the Time types

2021-08-19 Thread Antoine Pitrou
Hello, I would like to propose clarifying the allowed value range for the Time types. Specifically, I would propose that: 1) allowed values fall between 0 (included) and 86400 seconds (excluded), adjusted for the time unit; 2) leap seconds cannot be represented (see above: 86400 is outsi