Currently when we perform an index seek, we end up doing multiple unnecessary copies of the index key. Most likely we could construct the input KeyString (the seek point) in place if we refactor to pass a buffer as an input parameter (e.g. to IndexEntryComparison::makeKeyStringFromSeekPointForSeek). Similarly, the key returned by the index seems to be copied more than necessary, and we may be able to do some refactoring here to improve things as well. There may be other places (besides the seek interface) where we are doing unnecessary copies as well.

These copies are likely trivial for workloads where the index entries are being read from disk, but in the case where all data is in cache, the copies can have a non-trivial effect on overall performance.