Re: Parallel Seq Scan
От | Antonin Houska |
---|---|
Тема | Re: Parallel Seq Scan |
Дата | |
Msg-id | 3441.1436949884@localhost обсуждение исходный текст |
Ответ на | Re: Parallel Seq Scan (Amit Kapila <amit.kapila16@gmail.com>) |
Ответы |
Re: Parallel Seq Scan
|
Список | pgsql-hackers |
Amit Kapila <amit.kapila16@gmail.com> wrote: > Attached, find the rebased version of patch. [I haven't read this thread so far, sorry for possibly redundant comment.] I noticed that false is passed for required_outer agrument of create_partialseqscan_path(), while NULL seems to be cleaner in terms of C language. But in terms of semantics, I'm not sure this is correct anyway. Why does create_parallelscan_paths() not accept the actual rel->lateral_relids, just like create_seqscan_path() does? (See set_plain_rel_pathlist().) If there's reason for your approach, I think it's worth a comment. BTW, emacs shows whitespace on otherwise empty line parallelpath.c:57. -- Antonin Houska Cybertec Schönig & Schönig GmbH Gröhrmühlgasse 26 A-2700 Wiener Neustadt Web: http://www.postgresql-support.de, http://www.cybertec.at
В списке pgsql-hackers по дате отправления: