Обсуждение: Remove redundant AttStatsSlot initialization in eqjoinsel()

Поиск
Список
Период
Сортировка

Remove redundant AttStatsSlot initialization in eqjoinsel()

От
Chao Li
Дата:
Hi Hackers,

I noticed this problem while reviewing Tom’s patch [1]. The function eqjoinsel() unnecessarily zero-out sslot1 and sslot2 before calling get_attstatsslot() because get_attstatsslot() will unconditionally zero-out sslot in the first place.

I searched over the source tree, none of other callers of get_attstatsslot() initialize sslot. So for consistency, we should remove the unnecessary zero-out from eqjoinsel().


Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/




Вложения