Dmitry Nikitin <[email protected]> writes:
>
> https://github.com/postgres/postgres/blob/5d39becf8ba0080c98fee4b63575552f6800b012/src/backend/optimizer/prep/prepjointree.c#L3856
> bms_next_member() is allowed to return the zero as a valid value. Subsequent
> rt_fetch() offsets that
> zero to -1 which leads to the assertion down the code. Nothing wrong here?
> Either zero is simply not
> possible after that bms_next_member() because of some factors behind the code?
Zero isn't a valid relid. If we were to find such a value in that
bitmapset, an assertion would be a fine outcome.
regards, tom lane