Typo in planner README
Started by Daniel Gustafssonover 7 years ago2 messages
The README didn’t get the memo when set_base_rel_pathlist was renamed in
6543d81d659f417. Use the right name, set_base_rel_pathlists, as per the
attached.
cheers ./daniel
Attachments:
typo-planner_README.patchapplication/octet-stream; name=typo-planner_README.patch; x-unix-mode=0644Download
From 939b8ca4fca0061ada20dff3a1bc26b8feab20d3 Mon Sep 17 00:00:00 2001
From: Daniel Gustafsson <daniel@yesql.se>
Date: Thu, 7 Jun 2018 11:48:03 +0200
Subject: [PATCH] Fix typo in documentation
---
src/backend/optimizer/README | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/backend/optimizer/README b/src/backend/optimizer/README
index 87a38f9aaa..15af9ceff5 100644
--- a/src/backend/optimizer/README
+++ b/src/backend/optimizer/README
@@ -320,7 +320,7 @@ set up for recursive handling of subqueries
split up the qual into restrictions (a=1) and joins (b=c)
find qual clauses that enable merge and hash joins
----make_one_rel()
- set_base_rel_pathlist()
+ set_base_rel_pathlists()
find seqscan and all index paths for each base relation
find selectivity of columns used in joins
make_rel_from_joinlist()
--
2.14.1.145.gb3622a4ee
Re: Typo in planner README
On Thu, Jun 7, 2018 at 12:28 PM, Daniel Gustafsson <daniel@yesql.se> wrote:
The README didn’t get the memo when set_base_rel_pathlist was renamed in
6543d81d659f417. Use the right name, set_base_rel_pathlists, as per the
attached.
Applied, thanks.
--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>