Backend Flowchart / Developer doc
Hello,
Don't know if this the correct list to submit this issue.
On the https://www.postgresql.org/developer/backend/, it mentions a
Query.qual field which does not exist in the Query structure.
Thanks
On 2020-Apr-15, Marc Recht� wrote:
Hello,
Don't know if this the correct list to submit this issue.
On the https://www.postgresql.org/developer/backend/, it mentions a
Query.qual field which does not exist in the Query structure.
I think that diagram (and the accompanying wiki page) requires a *lot*
of more significant updating than just that field. It is wildly
outdated.
--
�lvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
On Thu, Apr 16, 2020 at 11:17:29AM -0400, Alvaro Herrera wrote:
On 2020-Apr-15, Marc Recht� wrote:
Hello,
Don't know if this the correct list to submit this issue.
On the https://www.postgresql.org/developer/backend/, it mentions a
Query.qual field which does not exist in the Query structure.I think that diagram (and the accompanying wiki page) requires a *lot*
of more significant updating than just that field. It is wildly
outdated.
I developed the attached patch to fix the items I saw needed updating.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EnterpriseDB https://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +Attachments:
backend.difftext/x-diff; charset=us-asciiDownload+6-5
Thread moved to www for patch application. Please apply the attached
patch to update the text of the backend flowchart. Thanks.
---------------------------------------------------------------------------
On Tue, Apr 21, 2020 at 11:31:09AM -0400, Bruce Momjian wrote:
On Thu, Apr 16, 2020 at 11:17:29AM -0400, Alvaro Herrera wrote:
On 2020-Apr-15, Marc Recht� wrote:
Hello,
Don't know if this the correct list to submit this issue.
On the https://www.postgresql.org/developer/backend/, it mentions a
Query.qual field which does not exist in the Query structure.I think that diagram (and the accompanying wiki page) requires a *lot*
of more significant updating than just that field. It is wildly
outdated.I developed the attached patch to fix the items I saw needed updating.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EnterpriseDB https://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +Attachments:
backend.difftext/x-diff; charset=us-asciiDownload+6-5
On 4/23/20 1:06 PM, Bruce Momjian wrote:
Thread moved to www for patch application. Please apply the attached
patch to update the text of the backend flowchart. Thanks.
Well, we first review and test it ;) There was a broken anchor tag,
which I fixed, and have now committed the patch.
Thanks!
Jonathan
On Thu, Apr 23, 2020 at 01:32:52PM -0400, Jonathan Katz wrote:
On 4/23/20 1:06 PM, Bruce Momjian wrote:
Thread moved to www for patch application. Please apply the attached
patch to update the text of the backend flowchart. Thanks.Well, we first review and test it ;) There was a broken anchor tag,
which I fixed, and have now committed the patch.
Thanks. I should have tested it myself.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EnterpriseDB https://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +On 4/23/20 1:33 PM, Bruce Momjian wrote:
On Thu, Apr 23, 2020 at 01:32:52PM -0400, Jonathan Katz wrote:
On 4/23/20 1:06 PM, Bruce Momjian wrote:
Thread moved to www for patch application. Please apply the attached
patch to update the text of the backend flowchart. Thanks.Well, we first review and test it ;) There was a broken anchor tag,
which I fixed, and have now committed the patch.Thanks. I should have tested it myself.
No worries. Content updates that span only a few lines tend to be fairly
easy to review/test. The important piece is the contribution.
Jonathan