Re: Parallel Seq Scan
| От | Amit Kapila |
|---|---|
| Тема | Re: Parallel Seq Scan |
| Дата | |
| Msg-id | CAA4eK1KdDYBf9CLOQHnvYNqyNik=JS9GX5QB2o9=FXUB=ofi4g@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: Parallel Seq Scan (Robert Haas <robertmhaas@gmail.com>) |
| Ответы |
Re: Parallel Seq Scan
|
| Список | pgsql-hackers |
On Wed, Sep 23, 2015 at 5:42 AM, Robert Haas <robertmhaas@gmail.com> wrote:
>
> On Tue, Sep 22, 2015 at 3:21 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> readAttrNumberCols uses sizeof(Oid) instead of sizeof(AttrNumber).
>
> I still don't understand why we need to handle PlanInvalItem.
As such this is not required, just to maintain consistency as I have added
>
> On Tue, Sep 22, 2015 at 3:21 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> readAttrNumberCols uses sizeof(Oid) instead of sizeof(AttrNumber).
>
> I still don't understand why we need to handle PlanInvalItem.
As such this is not required, just to maintain consistency as I have added
other similar nodes like PlanRowMark and NestLoopParam.
>
> Actually, come to think of it, I'm not sure we need PlannedStmt
> either.
>
PlannedStmt is needed because we are passing the same from master
to worker for execution and the reason was that Executor interfaces
expect it.
В списке pgsql-hackers по дате отправления: