Re: Improve join_search_one_level readibilty (one line change)
Peter Eisentraut > That shows exactly the problem being complained about. I apologize for not using the correct MIME type in my previous email to the pg-hackers mailing list. Upon sending the first email, I realized that my patch was labeled as "application/x-patch" instead of "text/x-patch." To make it more convenient for others to read the patch in the mail archives, I changed the file extension of my v2 patch to ".txt." ( https://www.postgresql.org/message-id/CANWNU8xm07jYUHxGh3XNHtcY37z%2B56-6bDD4piPt6%3DKidiHshQ%40mail.gmail.com ) However, I encountered an issue where cfbot did not apply the ".txt" file. I am still interested in learning **how to submit patches with the proper "text/x-patch" MIME type using Gmail.** I have noticed that some individuals have successfully used Gmail to submit patches with the correct MIME type. If anyone can provide assistance with this matter, I would be grateful. I am willing to contribute any helpful tips regarding using Gmail for patch submission to the Postgres wiki. I believe it is something like Mutt, right?
Re: Improve join_search_one_level readibilty (one line change)
Thank you, Julien, for letting me know that cfbot doesn't test txt files. Much appreciated! v2-0001-Improve-left-deep-tree-dp-algorithm-s-readability.patch Description: Binary data
Re: Improve join_search_one_level readibilty (one line change)
Thank you to Julien Rouhaud and Tender Wang for the reviews. Julien's detailed guide has proven to be incredibly helpful, and I am truly grateful for it. Thank you so much for providing such valuable guidance! I have initiated a new commitfest: https://commitfest.postgresql.org/43/4346/ Furthermore, I have attached a patch that improves the code by moving the initialization of "other_rels_list" outside the if branching. Perhaps Tom Lane would be interested in reviewing this minor change as well? From 5b038577cf4f8acdc094d6e41d3891b559016cae Mon Sep 17 00:00:00 2001 From: DouEnergy Date: Tue, 6 Jun 2023 15:30:14 +0800 Subject: [PATCH v2] Improve left deep tree dp algorithm's readability --- src/backend/optimizer/path/joinrels.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backend/optimizer/path/joinrels.c b/src/backend/optimizer/path/joinrels.c index 2feab2184f..a283e574f3 100644 --- a/src/backend/optimizer/path/joinrels.c +++ b/src/backend/optimizer/path/joinrels.c @@ -109,14 +109,14 @@ join_search_one_level(PlannerInfo *root, int level) List *other_rels_list; ListCell *other_rels; + other_rels_list = joinrels[1]; + if (level == 2) /* consider remaining initial rels */ { - other_rels_list = joinrels[level - 1]; other_rels = lnext(other_rels_list, r); } else/* consider all initial rels */ { - other_rels_list = joinrels[1]; other_rels = list_head(other_rels_list); } -- 2.37.1 (Apple Git-137.1)
Improve join_search_one_level readibilty (one line change)
Hello hackers Attached is my first patch for PostgreSQL, which is a simple one-liner that I believe can improve the code. In the "join_search_one_level" function, I noticed that the variable "other_rels_list" always refers to "joinrels[1]" even when the (level == 2) condition is met. I propose changing: "other_rels_list = joinrels[level - 1]" to "other_rels_list = joinrels[1]" This modification aims to enhance clarity and avoid unnecessary instructions. I would greatly appreciate any review and feedback on this patch as I am a newcomer to PostgreSQL contributions. Your input will help me improve and contribute effectively to the project. I have followed the excellent guide "How to submit a patch by email, 2023 edition" by Peter Eisentraut. Additionally, if anyone has any tips on ensuring that Gmail recognizes my attached patches as the "text/x-patch" MIME type when sending them from the Chrome client, I would be grateful for the advice. Or maybe the best practice is to use Git send-email ? Thank you for your time. Best regards Alex Hsieh From 18d2ed81d3640881082bf37d6e3021ffe671d215 Mon Sep 17 00:00:00 2001 From: DouEnergy Date: Fri, 2 Jun 2023 20:58:07 +0800 Subject: [PATCH] Improve left deep tree dp algorithm's readability --- src/backend/optimizer/path/joinrels.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/optimizer/path/joinrels.c b/src/backend/optimizer/path/joinrels.c index 2feab2184f..bca78d07fa 100644 --- a/src/backend/optimizer/path/joinrels.c +++ b/src/backend/optimizer/path/joinrels.c @@ -111,7 +111,7 @@ join_search_one_level(PlannerInfo *root, int level) if (level == 2) /* consider remaining initial rels */ { -other_rels_list = joinrels[level - 1]; +other_rels_list = joinrels[1]; other_rels = lnext(other_rels_list, r); } else/* consider all initial rels */ -- 2.37.1 (Apple Git-137.1)