Re: [Postgres-xc-developers] Query Planning bug ?

[Tomonari K. <kat...@po...>]

Hi Ashtosh,

Thanks for your comment!

 > The code changes are fine, but, the comment in pgxc_collect_RTE_walker()
 > seems to specific. I think it should read, "create a copy of query's 
range
 > table, so that it can be linked with other RTEs in the collector's 
context."
 >
I've revised the comment as you suggested.

 > In the test file, isn't there any way, you can add the offending statent
 > without any wrapper function test_execute_direct_all_xc_nodes()? We 
should
 > minimize the test-code to use only the minimal set of features. Since 
this
 > statement no more gets into infinite loop, please do not use the 
statement
 > timeout.

I've gotten rid of using statement timeout.
But I couldn't come up any idea to testing more simply,
so the function is remaining as same as before patch.
Any ideas?

Here is the new patch.
(against dec40008b3d689911566514614c5111c0a61327d)

regards,
-------------
NTT Software Corporation
Tomonari Katsumata