[HACKERS] Redundant comment in ExecutePlan

Поиск
Список
Период
Сортировка
От Rafia Sabih
Тема [HACKERS] Redundant comment in ExecutePlan
Дата
Msg-id CAOGQiiOJRPzRbL+_5N+1XqS+-MEwE9zVj+r2kQOj2Z7p+Bix7w@mail.gmail.com
обсуждение исходный текст
Ответы Re: [HACKERS] Redundant comment in ExecutePlan  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
Hello all,
I noticed there is a redundant comment in ExecutePlan,

/*
* If a tuple count was supplied, we must force the plan to run without
* parallelism, because we might exit early.  Also disable parallelism
* when writing into a relation, because no database changes are allowed
* in parallel mode.
*/
if (numberTuples || dest->mydest == DestIntoRel)
use_parallel_mode = false;

/*
* If a tuple count was supplied, we must force the plan to run without
* parallelism, because we might exit early.
*/
if (use_parallel_mode)
EnterParallelMode();

I felt it is not required the second time, hence attached is the patch
to remove this second comment.
-- 
Regards,
Rafia Sabih
EnterpriseDB: http://www.enterprisedb.com/

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

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Erik Rijkers
Дата:
Сообщение: Re: \if, \elseif, \else, \endif (was Re: [HACKERS] PSQL commands:\quit_if, \quit_unless)
Следующее
От: Andreas Karlsson
Дата:
Сообщение: Re: [HACKERS] \h tab-completion