Re: New TS API TSSyncStatFindName()

2020-04-14 Thread Sudheer Vinukonda
> > However, this results in not setting the Sync CB for the counter while also > not updating the api_rsb_index. This can potentially affect subsequent API > stats as they end up getting stat ids that are already in use by previously > used stats that did not end up ca

Re: New TS API TSSyncStatFindName()

2020-04-14 Thread Leif Hedstrom
equent API > stats as they end up getting stat ids that are already in use by previously > used stats that did not end up calling `TSStatCreate` > > To address this, I'm proposing to add a new TS API `TSSyncStatFindName()` > which returns found only if the stat is in the hash tabl

New TS API TSSyncStatFindName()

2020-04-14 Thread Sudheer Vinukonda
To address this, I'm proposing to add a new TS API `TSSyncStatFindName()` which returns found only if the stat is in the hash table and it's sync_cb is set. This API will replace the usage of `TSStatFindName` in header-rewrite and other plugins that we internally have which have simil