Re: [COMMITTERS] pgsql: Add hash partitioning.

2017-11-13 Thread amul sul 
On Mon, Nov 13, 2017 at 12:41 PM, amul sul  wrote:
> On Sun, Nov 12, 2017 at 3:01 AM, Andreas Seltenreich  
> wrote:
>> Robert Haas writes:
>>
>>> Add hash partitioning.
>>
>> sqlsmith triggers coredumps calling satisfies_hash_partition().
>> ISTM this function is lacking argument validation.  Example:
>>
>
> Thanks for the bug report.  Please find attached patch does the fix.

Updated patch attached -- Adjusted code comment to survive against pgindent.

Regards,
Amul


0001-argument-validation-v2.patch
Description: Binary data

-- 
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

Re: [COMMITTERS] pgsql: Add hash partitioning.

2017-11-12 Thread amul sul 
On Sun, Nov 12, 2017 at 3:01 AM, Andreas Seltenreich  wrote:
> Robert Haas writes:
>
>> Add hash partitioning.
>
> sqlsmith triggers coredumps calling satisfies_hash_partition().
> ISTM this function is lacking argument validation.  Example:
>

Thanks for the bug report.  Please find attached patch does the fix.

Regards,
Amul


0001-argument-validation.patch
Description: Binary data

-- 
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers