Hi David,
On 2017/09/27 6:04, David Rowley wrote:
> On 25 September 2017 at 23:04, Amit Langote
> <Langote_Amit_f8@lab.ntt.co.jp> wrote:
>> By the way, I'm now rebasing these patches on top of [1] and will try to
>> merge your refactoring patch in some appropriate way. Will post more
>> tomorrow.
>>
>> [1] https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=9140cf826
>
> Yeah, I see 0001 conflicts with that. I'm going to set this to waiting
> on author while you're busy rebasing this.
Thanks for the reminder. Just thought I'd say that while I'm actually
done rebasing itself (attaching rebased patches to try 'em out), I'm now
considering Robert's comments and will be busy for a bit revising things
based on those comments.
Some notes about the attached patches:
- 0001 includes refactoring that Dilip proposed upthread [1] (added him as
an author). I slightly tweaked his patch -- renamed the function
get_matching_clause to match_clauses_to_partkey, similar to
match_clauses_to_index.
- Code to set AppendPath's partitioned_rels in add_paths_to_append_rel()
revised by 0a480502b09 (was originally introduced in d3cc37f1d80) is
still revised to get partitioned_rels from a source that is not
PlannerInfo.pcinfo_list. With the new code, partitioned_rels won't
contain RT indexes of the partitioned child tables that were pruned.
Thanks,
Amit
[1]
https://www.postgresql.org/message-id/CAFiTN-tGnQzF_4QtbOHT-3hE%3DOvNaMfbbeRxa4UY0CQyF0G8gQ%40mail.gmail.com
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
