Re: [HACKERS] Minor issues with code comments related to abbreviated keys

2015-01-23 Thread Robert Haas
On Fri, Jan 23, 2015 at 2:59 PM, Peter Geoghegan wrote: > On Thu, Jan 22, 2015 at 5:17 PM, Peter Geoghegan wrote: >> Attached patch fixes minor issues in code comments that relate to >> abbreviated keys. > > There should also be a description of hyperLogLog in the new README > file within ./src/b

Re: [HACKERS] Minor issues with code comments related to abbreviated keys

2015-01-23 Thread Peter Geoghegan
On Thu, Jan 22, 2015 at 5:17 PM, Peter Geoghegan wrote: > Attached patch fixes minor issues in code comments that relate to > abbreviated keys. There should also be a description of hyperLogLog in the new README file within ./src/backend/lib/ I suggest: hyperloglog.c - a streaming cardinality e

[HACKERS] Minor issues with code comments related to abbreviated keys

2015-01-22 Thread Peter Geoghegan
Attached patch fixes minor issues in code comments that relate to abbreviated keys. -- Peter Geoghegan diff --git a/src/backend/utils/adt/varlena.c b/src/backend/utils/adt/varlena.c index c79b641..cfa1921 100644 --- a/src/backend/utils/adt/varlena.c +++ b/src/backend/utils/adt/varlena.c @@ -2088,