From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] Something for the TODO list: deprecating abstime and friends |
Date: | 2017-12-17 22:24:19 |
Message-ID: | 20171217222419.GA30764@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Dec 13, 2017 at 12:05:06AM -0800, Andres Freund wrote:
> On 2017-07-15 18:00:34 -0400, Tom Lane wrote:
> > * contrib/spi/timetravel depends on abstime columns to represent what
> > would nowadays be better done as a tstzrange. I'd have thought we
> > could maybe toss that example module overboard, but we just today got
> > a question about its usage, so I'm afraid it's not quite dead yet.
> > What shall we do with that?
>
> Looking at the code I'd be pretty strongly inclined to scrap it.
>
>
> Before I'd discovered this thread, I'd started to write up a
> patch. Attached. It's clearly not fully done. Questions I'd while
> hacking things up:
> - what to do with contrib/spi/timetravel - I'd just removed it from the
> relevant Makefile for now.
Porting it to tstzrange seems friendlier, however:
- Does that look like significant work to do this port?
- How about the work involved with upgrading existing installations?
> - nabstime.c currently implements timeofday() which imo is a pretty
> weird function. I'd be quite inclined to remove it at the same time as
> this.
+1 for binning it.
Best,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Munro | 2017-12-17 22:29:38 | Re: [sqlsmith] Parallel worker executor crash on master |
Previous Message | Jeremy Finzel | 2017-12-17 21:27:54 | Re: Backfill bgworker Extension? |