Invalid comment in ParallelQueryMain

Started by Julien Rouhaudabout 4 years ago4 messageshackers
Jump to latest
#1Julien Rouhaud
rjuju123@gmail.com

Hi,

While reading code around I just noticed that I failed to adapt a comment a
couple of lines above a removed line in 0f61727b75b9. Patch attached.

Attachments:

v1-0001-Fix-comment-in-ParallelQueryMain.patchtext/plain; charset=us-asciiDownload+1-2
#2Matthias van de Meent
boekewurm+postgres@gmail.com
In reply to: Julien Rouhaud (#1)
Re: Invalid comment in ParallelQueryMain

On Sat, 26 Mar 2022 at 17:01, Julien Rouhaud <rjuju123@gmail.com> wrote:

Hi,

While reading code around I just noticed that I failed to adapt a comment a
couple of lines above a removed line in 0f61727b75b9. Patch attached.

+1, seems OK to me.

#3Michael Paquier
michael@paquier.xyz
In reply to: Matthias van de Meent (#2)
Re: Invalid comment in ParallelQueryMain

On Sat, Mar 26, 2022 at 05:12:05PM +0100, Matthias van de Meent wrote:

On Sat, 26 Mar 2022 at 17:01, Julien Rouhaud <rjuju123@gmail.com> wrote:

While reading code around I just noticed that I failed to adapt a comment a
couple of lines above a removed line in 0f61727b75b9. Patch attached.

+1, seems OK to me.

Yep. Will fix.
--
Michael

#4Julien Rouhaud
rjuju123@gmail.com
In reply to: Michael Paquier (#3)
Re: Invalid comment in ParallelQueryMain

Hi,

On Sun, Mar 27, 2022 at 04:13:00PM +0900, Michael Paquier wrote:

On Sat, Mar 26, 2022 at 05:12:05PM +0100, Matthias van de Meent wrote:

On Sat, 26 Mar 2022 at 17:01, Julien Rouhaud <rjuju123@gmail.com> wrote:

While reading code around I just noticed that I failed to adapt a comment a
couple of lines above a removed line in 0f61727b75b9. Patch attached.

+1, seems OK to me.

Yep. Will fix.

For the archive's sake, this has been pushed as-of
411b91360f2711e36782b68cd0c9bc6de44d3384.

Thanks!