It seems that the api to increment metrics requires a full string value even though the string is only used for debug logging

This leads to callers needing to construct temporary strings that will allocate and deallocate from the heap immediately afterwards.

Ideally this should be replaced by a StringData object to avoid memory allocations plus copying.