Обсуждение: Incorrect debug info printed in generate_partition_wise_join_paths

generate_partition_wise_join_paths prints debug info, if
OPTIMIZER_DEBUG, using debug_print_rel at the end of each iteration for
collecting non-dummy child-joins, but I noticed that we pass to that
function the parent's RelOptInfo, not such a child-join's RelOptInfo.  I
don't think it's intentional, so here is a patch for fixing that.

Best regards,
Etsuro Fujita

Yes, that's the correct fix. We should be printing debug information
about the child and not the parent. Sorry for that bug and thanks for
fixing it.

On Fri, Dec 8, 2017 at 3:22 PM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:
> generate_partition_wise_join_paths prints debug info, if
> OPTIMIZER_DEBUG, using debug_print_rel at the end of each iteration for
> collecting non-dummy child-joins, but I noticed that we pass to that
> function the parent's RelOptInfo, not such a child-join's RelOptInfo.  I
> don't think it's intentional, so here is a patch for fixing that.
>
> Best regards,
> Etsuro Fujita



-- 
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company

(2017/12/11 17:24), Ashutosh Bapat wrote:
> Yes, that's the correct fix. We should be printing debug information
> about the child and not the parent.

Thanks for reviewing!

Best regards,
Etsuro Fujita

On Mon, Dec 11, 2017 at 9:05 PM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:
> (2017/12/11 17:24), Ashutosh Bapat wrote:
>> Yes, that's the correct fix. We should be printing debug information
>> about the child and not the parent.
>
> Thanks for reviewing!

Committed.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

(2017/12/13 0:56), Robert Haas wrote:
> Committed.

Thanks!

Best regards,
Etsuro Fujita