Re: [PATCH] Transaction traceability - txid_status(bigint)

От

Craig Ringer

Тема

Дата

23 августа 2016 г. в 16:59:43

Msg-id

CAMsr+YHzb6DLZ05gqagM-BoOhAcO-U5152ijfELiw4c22-e3+Q@mail.gmail.com

обсуждение

Ответ на

Re: [PATCH] Transaction traceability - txid_status(bigint) (Robert Haas)

Список

pgsql-hackers

Дерево обсуждения

[PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 20 августа 2016 г. в 13:24:53

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 20 августа 2016 г. в 13:47:05

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 22 августа 2016 г. в 17:03:34

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 августа 2016 г. в 00:31:30

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 августа 2016 г. в 00:55:13

Re: [PATCH] Transaction traceability - txid_status(bigint) Petr Jelinek <petr@2ndquadrant.com> 23 августа 2016 г. в 08:02:24

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 августа 2016 г. в 09:27:12

Re: [PATCH] Transaction traceability - txid_status(bigint) Petr Jelinek <petr@2ndquadrant.com> 23 августа 2016 г. в 10:04:30

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 23 августа 2016 г. в 14:18:57

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 23 августа 2016 г. в 14:34:23

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 августа 2016 г. в 16:59:43

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 23 августа 2016 г. в 19:10:42

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 29 августа 2016 г. в 03:25:48

Re: [PATCH] Transaction traceability - txid_status(bigint) Andres Freund <andres@anarazel.de> 29 августа 2016 г. в 03:45:36

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 29 августа 2016 г. в 07:53:46

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 1 сентября 2016 г. в 05:08:54

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 1 сентября 2016 г. в 05:30:03

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 2 сентября 2016 г. в 12:17:00

Re: [PATCH] Transaction traceability - txid_status(bigint) Simon Riggs <simon@2ndquadrant.com> 2 сентября 2016 г. в 12:30:53

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 2 сентября 2016 г. в 12:38:46

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 2 сентября 2016 г. в 13:02:06

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 2 сентября 2016 г. в 13:46:26

Re: [PATCH] Transaction traceability - txid_status(bigint) Petr Jelinek <petr@2ndquadrant.com> 2 сентября 2016 г. в 15:29:43

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 16 сентября 2016 г. в 00:52:30

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 16 сентября 2016 г. в 13:29:03

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 20 сентября 2016 г. в 01:54:43

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 20 сентября 2016 г. в 14:46:54

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 21 сентября 2016 г. в 07:40:42

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 21 сентября 2016 г. в 14:16:56

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 27 сентября 2016 г. в 01:23:31

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 21 декабря 2016 г. в 11:02:15

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 21 декабря 2016 г. в 19:30:44

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 22 декабря 2016 г. в 02:49:40

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 22 декабря 2016 г. в 04:55:06

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 22 декабря 2016 г. в 08:12:10

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 22 декабря 2016 г. в 20:26:22

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 декабря 2016 г. в 13:00:43

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 28 декабря 2016 г. в 13:00:37

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 января 2017 г. в 09:32:13

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 24 января 2017 г. в 18:49:57

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 25 января 2017 г. в 08:44:04

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 9 марта 2017 г. в 21:55:25

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 10 марта 2017 г. в 10:00:16

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 11 марта 2017 г. в 00:09:52

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 11 марта 2017 г. в 09:32:08

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 14 марта 2017 г. в 00:43:03

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 14 марта 2017 г. в 05:22:59

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 14 марта 2017 г. в 14:57:08

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 20 марта 2017 г. в 08:38:18

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 21 марта 2017 г. в 20:49:49

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 22 марта 2017 г. в 04:49:55

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 22 марта 2017 г. в 06:35:12

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 23 марта 2017 г. в 21:29:39

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 24 марта 2017 г. в 09:27:44

Re: [PATCH] Transaction traceability - txid_status(bigint) Peter Eisentraut <peter.eisentraut@2ndquadrant.com> 25 марта 2017 г. в 02:31:37

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 25 марта 2017 г. в 03:08:01

Re: [PATCH] Transaction traceability - txid_status(bigint) Peter Eisentraut <peter.eisentraut@2ndquadrant.com> 25 марта 2017 г. в 07:12:48

Re: [PATCH] Transaction traceability - txid_status(bigint) David Steele <david@pgmasters.net> 31 марта 2017 г. в 17:31:20

Re: [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 31 марта 2017 г. в 17:46:33

Re: [PATCH] Transaction traceability - txid_status(bigint) David Steele <david@pgmasters.net> 31 марта 2017 г. в 18:08:05

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 31 марта 2017 г. в 20:28:15

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 24 марта 2017 г. в 19:28:24

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Simon Riggs <simon@2ndquadrant.com> 22 марта 2017 г. в 10:13:32

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 22 марта 2017 г. в 20:41:47

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 марта 2017 г. в 06:25:07

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 марта 2017 г. в 07:22:23

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Simon Riggs <simon@2ndquadrant.com> 22 марта 2017 г. в 21:08:55

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 22 марта 2017 г. в 21:10:28

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 23 марта 2017 г. в 04:40:49

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 13 марта 2017 г. в 06:22:48

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Craig Ringer <craig@2ndquadrant.com> 25 января 2017 г. в 10:02:26

Re: [HACKERS] [PATCH] Transaction traceability - txid_status(bigint) Michael Paquier <michael.paquier@gmail.com> 1 февраля 2017 г. в 09:10:21

Re: [PATCH] Transaction traceability - txid_status(bigint) Robert Haas <robertmhaas@gmail.com> 29 августа 2016 г. в 06:01:41

On 23 August 2016 at 22:18, Robert Haas <robertmhaas@gmail.com> wrote:

On Mon, Aug 22, 2016 at 8:55 PM, Craig Ringer <craig@2ndquadrant.com> wrote:
> Updated patch series attached. As before, 0-4 intended for commit, 5 just
> because it'll be handy to have around for people doing wraparound related
> testing.
>
> Again, thanks for taking a look.

/me reviews a bit more deeply.

In 0001, it seems to me that "in-progress" should be "in progress".

Fine by me. I was on the fence about it anyway.

+CREATE TYPE txid_status AS ENUM ('committed', 'in-progress', 'aborted');

I'm not really that keen on this approach. I don't think we need to
introduce a new data type for this, and I would rather not use SQL,
either. It would be faster and simpler to just return the appropriate
string from a C function defined directly.

Also fine by me. You're right, keep it simple. It means the potential set of values isn't discoverable the same way, but ... meh. Using it usefully means reading the docs anyway.

The remaining 2 patches of interest are attached - txid_status() and txid_convert_if_recent(). Thanks for committing txid_current_if_assigned().

Now I'd best stop pretending I'm in a sensible timezone.

Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

В списке pgsql-hackers по дате отправления

Предыдущее

От: Robert Haas

Дата: 23 августа 2016 г. в 16:56:54

Сообщение: Re: Index Onlys Scan for expressions

Следующее

От: Craig Ringer

Дата: 23 августа 2016 г. в 17:02:15

Сообщение: Re: Logical decoding of sequence advances, part II