Small fix for _copySetConstraintsStmt

Started by Fernando Nasserover 24 years ago3 messagespatches
Jump to latest
#1Fernando Nasser
fnasser@redhat.com

Field is a list of strings, not nodes.

--
Fernando Nasser
Red Hat Canada Ltd. E-Mail: fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9

Attachments:

CONSTRAINTS.PATCHtext/plain; charset=us-ascii; name=CONSTRAINTS.PATCHDownload+2-2
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Fernando Nasser (#1)
Re: Small fix for _copySetConstraintsStmt

Fernando Nasser <fnasser@redhat.com> writes:

Field is a list of strings, not nodes.

Yeah, but listCopy is not the solution, because it'd only copy the cons
cells and not the pointed-to strings. The representation of
ConstraintsSetStmt is really just plain broken. It should use a list
of Value string nodes --- which would make copySetConstraintsStmt okay
as-is. Fixing now...

regards, tom lane

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Fernando Nasser (#1)
Re: Small fix for _copySetConstraintsStmt

Fernando Nasser <fnasser@redhat.com> writes:

Field is a list of strings, not nodes.

I fixed this by fixing the representation of ConstraintsSetStmt.
copyfuncs.c does not need to change.

regards, tom lane