Re: [PATCH] Refactor pqformat.{c,h} and protocol.h - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
Date
Msg-id ZpavFHpMHdY9fJFc@nathan
Whole thread Raw
In response to Re: [PATCH] Refactor pqformat.{c,h} and protocol.h  (Aleksander Alekseev <aleksander@timescale.com>)
Responses Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
List pgsql-hackers
I took a closer look at 0001.

diff --git a/src/include/libpq/protocol.h b/src/include/libpq/protocol.h
index 4b8d440365..8c0f095edf 100644
--- a/src/include/libpq/protocol.h
+++ b/src/include/libpq/protocol.h
@@ -47,6 +47,7 @@
 #define PqMsg_EmptyQueryResponse    'I'
 #define PqMsg_BackendKeyData        'K'
 #define PqMsg_NoticeResponse        'N'
+#define PqMsg_Progress              'P'
 #define PqMsg_AuthenticationRequest 'R'
 #define PqMsg_ParameterStatus        'S'
 #define PqMsg_RowDescription        'T'

As discussed elsewhere [0], we can add the leader/worker protocol
characters to protocol.h, but they should probably go in a separate
section.  I'd recommend breaking that part out to a separate patch, too.

[0] https://postgr.es/m/20230829161555.GB2136737%40nathanxps13

-- 
nathan



pgsql-hackers by date:

Previous
From: Aleksander Alekseev
Date:
Subject: Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
Next
From: Dean Rasheed
Date:
Subject: Re: Restrict EXPLAIN (ANALYZE) for RLS and security_barrier views