summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane2015-09-30 02:06:59 +0000
committerTom Lane2015-09-30 02:07:16 +0000
commit6057f61b4d511d8321f295d827ced13809e5a1ae (patch)
tree41697e4a6793f991e0b061831936b1ccb7d8a016
parenta16db3a07da9c47a5ef5b1d18f9c45bdb893b243 (diff)
Small improvements in comments in async.c.
We seem to have lost a line somewhere along the way in the comment block that discusses async.c's locks, because it suddenly refers to "both locks" without previously having mentioned more than one. Add a sentence to make that read more sanely. Also, refer to the "pos of the slowest backend" not the "tail of the slowest backend", since we have no per-backend value called "tail".
-rw-r--r--src/backend/commands/async.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/backend/commands/async.c b/src/backend/commands/async.c
index 3b71174b82..91baede4e3 100644
--- a/src/backend/commands/async.c
+++ b/src/backend/commands/async.c
@@ -224,6 +224,7 @@ typedef struct QueueBackendStatus
* When holding the lock in EXCLUSIVE mode, backends can inspect the entries
* of other backends and also change the head and tail pointers.
*
+ * AsyncCtlLock is used as the control lock for the pg_notify SLRU buffers.
* In order to avoid deadlocks, whenever we need both locks, we always first
* get AsyncQueueLock and then AsyncCtlLock.
*
@@ -234,7 +235,7 @@ typedef struct QueueBackendStatus
typedef struct AsyncQueueControl
{
QueuePosition head; /* head points to the next free location */
- QueuePosition tail; /* the global tail is equivalent to the tail
+ QueuePosition tail; /* the global tail is equivalent to the pos
* of the "slowest" backend */
TimestampTz lastQueueFillWarn; /* time of last queue-full msg */
QueueBackendStatus backend[FLEXIBLE_ARRAY_MEMBER];