On 1/14/21 10:22 PM, Simon Riggs wrote: > On Thu, Jan 14, 2021 at 5:46 PM Surafel Temesgen <surafel3...@gmail.com> > wrote: > >> On Fri, Jan 8, 2021 at 7:50 PM Ryan Lambert <r...@rustprooflabs.com> wrote: >>> >>> I prefer to have them hidden by default. This was mentioned up-thread with >>> no decision, it seems the standard is ambiguous. MS SQL appears to have >>> flip-flopped on this decision [1]. > > I think the default should be like this: > > SELECT * FROM foo FOR SYSTEM_TIME AS OF ... > should NOT include the Start and End timestamp columns > because this acts like a normal query just with a different snapshot timestamp > > SELECT * FROM foo FOR SYSTEM_TIME BETWEEN x AND y > SHOULD include the Start and End timestamp columns > since this form of query can include multiple row versions for the > same row, so it makes sense to see the validity times
I don't read the standard as being ambiguous about this at all. The columns should be shown just like any other column of the table. I am not opposed to being able to set an attribute on columns allowing them to be excluded from "*" but that is irrelevant to this patch. -- Vik Fearing
