[COMMITTERS] pgsql: Cleanup some problems in new Perl test code

2015-12-07 Thread Alvaro Herrera
Cleanup some problems in new Perl test code Noted by Tom Lane: - PostgresNode had a BEGIN block which created files, contrary to perlmod suggestions to do that only on INIT blocks. - Assign ports randomly rather than starting from 90600. Noted by Noah Misch: - Change use of no-longer-set PGPORT

[COMMITTERS] pgsql: Fix another oversight in checking if a join with LATERAL refs is

2015-12-07 Thread Tom Lane
Fix another oversight in checking if a join with LATERAL refs is legal. It was possible for the planner to decide to join a LATERAL subquery to the outer side of an outer join before the outer join itself is completed. Normally that's fine because of the associativity rules, but it doesn't work if

[COMMITTERS] pgsql: Fix another oversight in checking if a join with LATERAL refs is

2015-12-07 Thread Tom Lane
Fix another oversight in checking if a join with LATERAL refs is legal. It was possible for the planner to decide to join a LATERAL subquery to the outer side of an outer join before the outer join itself is completed. Normally that's fine because of the associativity rules, but it doesn't work if

[COMMITTERS] pgsql: Fix another oversight in checking if a join with LATERAL refs is

2015-12-07 Thread Tom Lane
Fix another oversight in checking if a join with LATERAL refs is legal. It was possible for the planner to decide to join a LATERAL subquery to the outer side of an outer join before the outer join itself is completed. Normally that's fine because of the associativity rules, but it doesn't work if

[COMMITTERS] pgsql: Fix another oversight in checking if a join with LATERAL refs is

2015-12-07 Thread Tom Lane
Fix another oversight in checking if a join with LATERAL refs is legal. It was possible for the planner to decide to join a LATERAL subquery to the outer side of an outer join before the outer join itself is completed. Normally that's fine because of the associativity rules, but it doesn't work if

[COMMITTERS] pgsql: PostgresNode: wrap correctly around port number range end

2015-12-07 Thread Alvaro Herrera
PostgresNode: wrap correctly around port number range end Per note from Tom Lane Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/7ac5d9b31637b1856c7ac9cb625bcca694a2790a Modified Files -- src/test/perl/PostgresNode.pm |3 ++- 1 file changed, 2 inserti

[COMMITTERS] pgsql: Simplify LATERAL-related calculations within add_paths_to_joinre

2015-12-07 Thread Tom Lane
Simplify LATERAL-related calculations within add_paths_to_joinrel(). While convincing myself that commit 7e19db0c09719d79 would solve both of the problems recently reported by Andreas Seltenreich, I realized that add_paths_to_joinrel's handling of LATERAL restrictions could be made noticeably simp