Incorrect comment for memset() on pgssHashKey?

Поиск
Список
Период
Сортировка
От Japin Li
Тема Incorrect comment for memset() on pgssHashKey?
Дата
Msg-id MEYP282MB16696317B5DA7D0D92306149B627A@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM
обсуждение исходный текст
Ответы Re: Incorrect comment for memset() on pgssHashKey?  (Heikki Linnakangas <hlinnaka@iki.fi>)
Список pgsql-hackers
Hi,

Commit 6b4d23feef introduces a toplevel field in pgssHashKey, which leads
padding.  In pgss_store(), it comments that memset() is required when
pgssHashKey is without padding only.

@@ -1224,9 +1227,14 @@ pgss_store(const char *query, uint64 queryId,
     query = CleanQuerytext(query, &query_location, &query_len);

     /* Set up key for hashtable search */
+
+    /* memset() is required when pgssHashKey is without padding only */
+    memset(&key, 0, sizeof(pgssHashKey));
+
     key.userid = GetUserId();
     key.dbid = MyDatabaseId;
     key.queryid = queryId;
+    key.toplevel = (exec_nested_level == 0);

     /* Lookup the hash table entry with shared lock. */
     LWLockAcquire(pgss->lock, LW_SHARED);

However, we need memset() only when pgssHashKey has padding, right?

-- 
Regrads,
Japin Li.

diff --git a/contrib/pg_stat_statements/pg_stat_statements.c b/contrib/pg_stat_statements/pg_stat_statements.c
index 21bede29fe..f6ed87f1e6 100644
--- a/contrib/pg_stat_statements/pg_stat_statements.c
+++ b/contrib/pg_stat_statements/pg_stat_statements.c
@@ -1257,7 +1257,7 @@ pgss_store(const char *query, uint64 queryId,
 
     /* Set up key for hashtable search */
 
-    /* memset() is required when pgssHashKey is without padding only */
+    /* memset() is required when pgssHashKey is with padding only */
     memset(&key, 0, sizeof(pgssHashKey));
 
     key.userid = GetUserId();

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Dilip Kumar
Дата:
Сообщение: Re: Improving btree performance through specializing by key shape, take 2
Следующее
От: Peter Geoghegan
Дата:
Сообщение: Re: ReadRecentBuffer() doesn't scale well