pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: bt22nakamorit <bt22nakamorit@oss.nttdata.com>
To: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: JUMBLE_SIZE macro in two files
Date: Tue, 04 Oct 2022 17:41:12 +0900
Message-ID: <bb4ebd0412da9b1ac87a5eb2a3646bf1@oss.nttdata.com> (raw)

Hi,

queryjumble.c and queryjumble.h both define a macro JUMBLE_SIZE = 1024.
Since queryjumble.c includes queryjumble.h, the JUMBLE_SIZE definition 
in queryjumble.c should be deleted.
Thoughts?

Tatsu

Attachments:


jumbsize_def.patch (text/x-diff, 476B) download
diff --git a/src/backend/utils/misc/queryjumble.c b/src/backend/utils/misc/queryjumble.c
index a8508463e7..dd1ffbd546 100644
--- a/src/backend/utils/misc/queryjumble.c
+++ b/src/backend/utils/misc/queryjumble.c
@@ -37,8 +37,6 @@
 #include "parser/scansup.h"
 #include "utils/queryjumble.h"
 
-#define JUMBLE_SIZE                            1024    /* query serialization buffer size */
-
 /* GUC parameters */
 int                    compute_query_id = COMPUTE_QUERY_ID_AUTO;


view thread (6+ messages)  latest in thread

Message-ID: <bb4ebd0412da9b1ac87a5eb2a3646bf1@oss.nttdata.com>
Permalink:  ../bb4ebd0412da9b1ac87a5eb2a3646bf1@oss.nttdata.com/
Also on:    postgresql.org/message-id/bb4ebd0412da9b1ac87a5eb2a3646bf1@oss.nttdata.com

 ·  · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: bt22nakamorit@oss.nttdata.com, pgsql-hackers@lists.postgresql.org
  Subject: Re: JUMBLE_SIZE macro in two files
  In-Reply-To: <bb4ebd0412da9b1ac87a5eb2a3646bf1@oss.nttdata.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox