Horton

Change bms_add_range to be a no-op for empty ranges

Alvaro Herrera <alvherre@alvh.no-ip.org>

View on postgr.es
Commit: 192d1bbf99699de76473184c57fff9ff4b4246f4
Author: Alvaro Herrera <alvherre@alvh.no-ip.org>
Date: 2018-07-30T22:43:30Z
Releases: 11.0 
Change bms_add_range to be a no-op for empty ranges

In commit 84940644de93, bms_add_range was added with an API to fail with
an error if an empty range was specified.  This seems arbitrary and
unhelpful, so turn that case into a no-op instead.  Callers that require
further verification on the arguments or result can apply them by
themselves.

This fixes the bug that partition pruning throws an API error for a case
involving the default partition of a default partition, as in the
included test case.

Reported-by: Rajkumar Raghuwanshi <rajkumar.raghuwanshi@enterprisedb.com>
Diagnosed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/16590.1532622503@sss.pgh.pa.us

Files

PathChange+/−
src/backend/nodes/bitmapset.c modified +5 −2
src/test/regress/expected/partition_prune.out modified +15 −0
src/test/regress/sql/partition_prune.sql modified +7 −0

Discussion