Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rVBwg-00FPUt-8X for pgsql-hackers@arkaria.postgresql.org; Wed, 31 Jan 2024 14:52:10 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1rVBwe-00EbVC-T1 for pgsql-hackers@arkaria.postgresql.org; Wed, 31 Jan 2024 14:52:08 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rVBwe-00EbV3-Io for pgsql-hackers@lists.postgresql.org; Wed, 31 Jan 2024 14:52:08 +0000 Received: from mail-ua1-x935.google.com ([2607:f8b0:4864:20::935]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1rVBwc-004jCA-98 for pgsql-hackers@postgresql.org; Wed, 31 Jan 2024 14:52:08 +0000 Received: by mail-ua1-x935.google.com with SMTP id a1e0cc1a2514c-7ce603b9051so2150527241.2 for ; Wed, 31 Jan 2024 06:52:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1706712724; x=1707317524; darn=postgresql.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=MZEUJXe1NAs5+8wDRAJJisACzSeOf1ryvyvVAcHKud8=; b=mEelFJp+nVzmWuy/t7HIgia5fmBWXIC7/YJStfDA53/A6SlvbPEEAiNN8Pidmw7caN 3W9ER+tHioHvR/TNkWalgWzhCT/AUoXstgVx/Zy9bE/+gDdkAeMRYQl/qDp0qGHx/D1K NPZ56NHa84fnMorg8a2XtvTJLK7ys6C4wGpUqYog2yk3dPxF5VKmNdetqvSLA2Je59Zg pA4sYmUOpZ5Ak4m1mP+w2Unmg0bEawRWEYN1vetKsr576OOQCA/bwvEgn+AegB7iA2+i yWXF/IJFZ9R454rE36936xAenB9C8SAOPI0SYhvoH1m6ZAcrxXV258BmPw3pU2qTSmo4 WbIQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706712724; x=1707317524; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=MZEUJXe1NAs5+8wDRAJJisACzSeOf1ryvyvVAcHKud8=; b=tVzrj695tNluXnlAWRLLPihjN+8qDkn6b6+EvJoyI7Rz0b2mmekxexkcj48th+3AV/ wg5OKz9jtHX2QcPXaE3Gs8ucaoBc1hT9oLSpKMzJLx0N5/J1wZY0C8SlfTygRmLU/vUg RUcpUvY+j/DQqgQI4vC8uhtffbVLEma4qBpyRGNL00Hc7d86iAycsD3kApRbFb/UguIb L9PSh/+0429GMXIhQneb75JSzaOpPWkxrV5T1X1wqIbfkg5CiZIwz+hHU5dA/EpxzeNe Is3Ah3FQzy+qgGolgILFrRPnRE7AUm+bb5erWzCyiInqnFd1EBPl1+068XNGT8w+JGw2 czbg== X-Gm-Message-State: AOJu0Yz+FX0NxctQfN9vNMKS1kjTppcJNHQtEpjlwzgBm747owK7reEw 6WBAwNHIXl7LZvOdApi8bwsfjcPO+GXkgaws4MsxTDNjeifk2J6iH3as560cDe5l/EXiqBn6DUb Qnk69GmlVH2jNlr3jNIafjvVjfWI= X-Google-Smtp-Source: AGHT+IGy8mmRmOXKQZPHreGRkKDNULOn2FKx1aY7ydHxcIi9rXyt7JVRdVT8IufnME+HnL0XQ8mBAANJF0bQxzV/to8= X-Received: by 2002:a67:fb91:0:b0:46a:fd09:211b with SMTP id n17-20020a67fb91000000b0046afd09211bmr1490938vsr.20.1706712724259; Wed, 31 Jan 2024 06:52:04 -0800 (PST) MIME-Version: 1.0 References: <202401181711.qxjxpnl3ohnw@alvherre.pgsql> <202401221619.ragf2vnbyopl@alvherre.pgsql> In-Reply-To: From: jian he Date: Wed, 31 Jan 2024 22:51:52 +0800 Message-ID: Subject: Re: remaining sql/json patches To: Amit Langote Cc: Alvaro Herrera , Andres Freund , Erik Rijkers , Andrew Dunstan , PostgreSQL-development Content-Type: text/plain; charset="UTF-8" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Hi. minor issues. I am wondering do we need add `pg_node_attr(query_jumble_ignore)` to some of our created structs in src/include/nodes/parsenodes.h in v39-0001-Add-SQL-JSON-query-functions.patch diff --git a/src/backend/parser/parse_jsontable.c b/src/backend/parser/parse_jsontable.c new file mode 100644 index 0000000000..25b8204dc6 --- /dev/null +++ b/src/backend/parser/parse_jsontable.c @@ -0,0 +1,718 @@ +/*------------------------------------------------------------------------- + * + * parse_jsontable.c + * parsing of JSON_TABLE + * + * Portions Copyright (c) 1996-2022, PostgreSQL Global Development Group + * Portions Copyright (c) 1994, Regents of the University of California + * + * + * IDENTIFICATION + * src/backend/parser/parse_jsontable.c + * + *------------------------------------------------------------------------- + */ 2022 should change to 2024.