Small patch for pgstat.c: fix comment + pgindent

Started by Aleksander Alekseevalmost 10 years ago2 messages
#1Aleksander Alekseev
a.alekseev@postgrespro.ru
1 attachment(s)

Hello

I noticed:

http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=b6fb6471f6afaf649e52f38269fd8c5c60647669

... that comments for procedures pgstat_progress_update_param and
pgstat_progress_end_command are identical. Here is a patch that fixes
this. Also one empty line added after running pgindent on this file.

Best regards,
Aleksander

Attachments:

pgstat.difftext/x-patchDownload
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c
index ce5da3e..5005fe5 100644
--- a/src/backend/postmaster/pgstat.c
+++ b/src/backend/postmaster/pgstat.c
@@ -2733,6 +2733,7 @@ pgstat_bestart(void)
 	beentry->st_activity[pgstat_track_activity_query_size - 1] = '\0';
 	beentry->st_progress_command = PROGRESS_COMMAND_INVALID;
 	beentry->st_progress_command_target = InvalidOid;
+
 	/*
 	 * we don't zero st_progress_param here to save cycles; nobody should
 	 * examine it until st_progress_command has been set to something other
@@ -2904,7 +2905,7 @@ pgstat_progress_update_param(int index, int64 val)
 /*-----------
  * pgstat_progress_end_command() -
  *
- * Update index'th member in st_progress_param[] of own backend entry.
+ * Clear command progress indicator of own backend entry.
  *-----------
  */
 void
#2Amit Langote
Langote_Amit_f8@lab.ntt.co.jp
In reply to: Aleksander Alekseev (#1)
Re: Small patch for pgstat.c: fix comment + pgindent

Hi,

Thanks for the report.

On 2016/03/10 18:05, Aleksander Alekseev wrote:

Hello

I noticed:

http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=b6fb6471f6afaf649e52f38269fd8c5c60647669

... that comments for procedures pgstat_progress_update_param and
pgstat_progress_end_command are identical. Here is a patch that fixes
this. Also one empty line added after running pgindent on this file.

This was reported on an in-progress thread[1]/messages/by-id/8f95982489b849c7ae662f20e908ad52@MP-MSGSS-MBX007.msg.nttdata.co.jp and I sent a patch in reply:

Thanks,
Amit

[1]: /messages/by-id/8f95982489b849c7ae662f20e908ad52@MP-MSGSS-MBX007.msg.nttdata.co.jp
/messages/by-id/8f95982489b849c7ae662f20e908ad52@MP-MSGSS-MBX007.msg.nttdata.co.jp

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers