Re: Added missing copy related data structures to typedefs.list
On Wed, Dec 30, 2020 at 7:10 PM Amit Kapila wrote: > > On Sat, Dec 26, 2020 at 9:16 PM vignesh C wrote: > > > > On Thu, Dec 17, 2020 at 4:28 AM Bruce Momjian wrote: > > > > > > On Mon, Dec 7, 2020 at 01:56:50PM +0530, vignesh C wrote: > > > > Hi, > > > > > > > > Added missing copy related data structures to typedefs.list, these > > > > data structures were added while copy files were split during the > > > > recent commit. I found this while running pgindent for parallel copy > > > > patches. > > > > The Attached patch has the changes for the same. > > > > Thoughts? > > > > > > Uh, we usually only update the typedefs file before we run pgindent on > > > the master branch. > > > > > > > Ok, Thanks for the clarification. I was not sure as in few of the > > enhancements it was included as part of the patches. > > > > Yeah, I do that while committing patches that require changes in > typedefs. It is not a norm and I am not sure how much value it adds to > do it separately for the missing ones unless you are making changes in > the same file they are used and you are facing unrelated diffs due to > those missing ones. I found this while I was running pgindent for parallel copy patches. I was not sure if this change was left out intentionally or by mistake. I'm fine if it is committed separately or together at a later point. It is not a major problem for my patch since I know the change, I will do the required adjustment when I make changes on top of it, if it is not getting committed. But I felt we can commit this since it is a recent change. Regards, Vignesh EnterpriseDB: http://www.enterprisedb.com
Re: Added missing copy related data structures to typedefs.list
On Sat, Dec 26, 2020 at 9:16 PM vignesh C wrote: > > On Thu, Dec 17, 2020 at 4:28 AM Bruce Momjian wrote: > > > > On Mon, Dec 7, 2020 at 01:56:50PM +0530, vignesh C wrote: > > > Hi, > > > > > > Added missing copy related data structures to typedefs.list, these > > > data structures were added while copy files were split during the > > > recent commit. I found this while running pgindent for parallel copy > > > patches. > > > The Attached patch has the changes for the same. > > > Thoughts? > > > > Uh, we usually only update the typedefs file before we run pgindent on > > the master branch. > > > > Ok, Thanks for the clarification. I was not sure as in few of the > enhancements it was included as part of the patches. > Yeah, I do that while committing patches that require changes in typedefs. It is not a norm and I am not sure how much value it adds to do it separately for the missing ones unless you are making changes in the same file they are used and you are facing unrelated diffs due to those missing ones. -- With Regards, Amit Kapila.
Re: Added missing copy related data structures to typedefs.list
On Thu, Dec 17, 2020 at 4:28 AM Bruce Momjian wrote: > > On Mon, Dec 7, 2020 at 01:56:50PM +0530, vignesh C wrote: > > Hi, > > > > Added missing copy related data structures to typedefs.list, these > > data structures were added while copy files were split during the > > recent commit. I found this while running pgindent for parallel copy > > patches. > > The Attached patch has the changes for the same. > > Thoughts? > > Uh, we usually only update the typedefs file before we run pgindent on > the master branch. > Ok, Thanks for the clarification. I was not sure as in few of the enhancements it was included as part of the patches. Regards, Vignesh EnterpriseDB: http://www.enterprisedb.com
Re: Added missing copy related data structures to typedefs.list
On Mon, Dec 7, 2020 at 01:56:50PM +0530, vignesh C wrote: > Hi, > > Added missing copy related data structures to typedefs.list, these > data structures were added while copy files were split during the > recent commit. I found this while running pgindent for parallel copy > patches. > The Attached patch has the changes for the same. > Thoughts? Uh, we usually only update the typedefs file before we run pgindent on the master branch. -- Bruce Momjian https://momjian.us EnterpriseDB https://enterprisedb.com The usefulness of a cup is in its emptiness, Bruce Lee
Added missing copy related data structures to typedefs.list
Hi, Added missing copy related data structures to typedefs.list, these data structures were added while copy files were split during the recent commit. I found this while running pgindent for parallel copy patches. The Attached patch has the changes for the same. Thoughts? Regards, Vignesh EnterpriseDB: http://www.enterprisedb.com From d656c2f0bfbf68f5ceb98a0eb205e5e77f21602f Mon Sep 17 00:00:00 2001 From: Vignesh C Date: Mon, 7 Dec 2020 13:48:31 +0530 Subject: [PATCH] Added missing copy related data structures. Added missing copy related data structures to typedefs.list, these data structures were added while copy files were split during the recent commit. --- src/tools/pgindent/typedefs.list | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/tools/pgindent/typedefs.list b/src/tools/pgindent/typedefs.list index 974b138..ecf9c3e 100644 --- a/src/tools/pgindent/typedefs.list +++ b/src/tools/pgindent/typedefs.list @@ -421,12 +421,18 @@ ConversionLocation ConvertRowtypeExpr CookedConstraint CopyDest +CopyFormatOptions +CopyFromState +CopyFromStateData CopyInsertMethod CopyMultiInsertBuffer CopyMultiInsertInfo +CopySource CopyState CopyStateData CopyStmt +CopyToState +CopyToStateData Cost CostSelector Counters -- 1.8.3.1