Widen the nLocks counts in local lock tables from int to int64. This

forestalls potential overflow when the same table (or other object, but
usually tables) is accessed by very many successive queries within a single
transaction.  Per report from Michael Milligan.

Back-patch to 8.0, which is as far back as the patch conveniently applies.
There have been no reports of overflow in pre-8.3 releases, but clearly the
risk existed all along.  (Michael's report suggests that 8.3 may consume lock
counts faster than prior releases, but with no test case to look at it's hard
to be sure about that.  Widening the counts seems a good future-proofing
measure in any event.)

diff --git a/src/include/storage/lock.h b/src/include/storage/lock.h
index 3d14ef04fee9ba884386411fc6b12baf7fc16806..6d591a95e509d2e3d8eb317fe6de3c6ce15d16c9 100644 (file)
--- a/src/include/storage/lock.h
+++ b/src/include/storage/lock.h
@@ -314,7 +314,7 @@ typedef struct LOCALLOCKOWNER
         * Must use a forward struct reference to avoid circularity.
         */
        struct ResourceOwnerData *owner;
-       int                     nLocks;                 /* # of times held by this owner */
+       int64           nLocks;                 /* # of times held by this owner */
 } LOCALLOCKOWNER;
 
 typedef struct LOCALLOCK
@@ -326,7 +326,7 @@ typedef struct LOCALLOCK
        LOCK       *lock;                       /* associated LOCK object in shared mem */
        PROCLOCK   *proclock;           /* associated PROCLOCK object in shmem */
        bool            isTempObject;   /* true if lock is on a temporary object */
-       int                     nLocks;                 /* total number of times lock is held */
+       int64           nLocks;                 /* total number of times lock is held */
        int                     numLockOwners;  /* # of relevant ResourceOwners */
        int                     maxLockOwners;  /* allocated size of array */
        LOCALLOCKOWNER *lockOwners; /* dynamically resizable array */