|From:||Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>|
|To:||Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>|
|Cc:||Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>, pgsql-hackers(at)postgresql(dot)org|
|Subject:||Re: Copyright in partition.h and partition.c|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 2017/09/05 21:14, Tom Lane wrote:
> Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> writes:
>> On 2017/09/05 15:48, Etsuro Fujita wrote:
>>> Here is the copyright in partition.h:
>>> * Copyright (c) 2007-2017, PostgreSQL Global Development Group
>>> I think it's reasonable that that matches the copyright in partition.c,
>>> but partition.c has:
>>> * Portions Copyright (c) 1996-2017, PostgreSQL Global Development Group
>>> * Portions Copyright (c) 1994, Regents of the University of California
>>> Is that intentional?
>> No, it's unintentional. The difference may have resulted from copying
>> different files to become partition.h and partition.c, respectively.
>> Maybe, we should change both to say 2016-2017?
>> I don't know the exact rule for how we determine those years. Is there
>> some rule in place about that? When I look at execParallel.c, which
>> supposedly got introduced into the tree recently, I see 1996-2017. OTOH,
>> the files in contrib/bloom all have 2016-2017.
> Our usual practice is to write the copyright like it is in partition.c
> even in new files. This avoids any question about whether any of the
> code was copied-and-pasted from somewhere else in PG. Even if not one
> word in the file can be traced to code that was somewhere else before,
> it seems to me that this is an appropriate thing to do, to give due
> credit to those who came before us.
> In short: we should make partition.h's copyright look like partition.c's
> not vice versa.
Attached patch does that.
|Next Message||Andres Freund||2017-09-06 02:07:07||Re: increasing the default WAL segment size|
|Previous Message||Magnus Hagander||2017-09-05 23:25:59||Re: pg_basebackup behavior on non-existent slot|