Re: [patch] Adding an assertion to report too long hash table name

2022-11-07 Thread Zhang Mingli
Hi, On Sep 29, 2022, 09:38 +0800, Xiaoran Wang , wrote: > Hi, > The max size for the shmem hash table name is SHMEM_INDEX_KEYSIZE - 1. but > when the caller uses a longer hash table name, it doesn't report any error, > insteadit just uses the first SHMEM_INDEX_KEYSIZE -1 chars as the hash

Re: [patch] Adding an assertion to report too long hash table name

2022-09-29 Thread Junwang Zhao
LGTM +1 On Thu, Sep 29, 2022 at 9:38 AM Xiaoran Wang wrote: > > Hi, > > The max size for the shmem hash table name is SHMEM_INDEX_KEYSIZE - 1. > but when the caller uses a longer hash table name, it doesn't report any > error, instead > it just uses the first SHMEM_INDEX_KEYSIZE -1 chars as the

[patch] Adding an assertion to report too long hash table name

2022-09-28 Thread Xiaoran Wang
Hi, The max size for the shmem hash table name is SHMEM_INDEX_KEYSIZE - 1. but when the caller uses a longer hash table name, it doesn't report any error, instead it just uses the first SHMEM_INDEX_KEYSIZE -1 chars as the hash table name. I created some shmem hash tables with the same prefix