[Go] the output is not passed to emitResult in scalarExecutor.executeSpans which may cause the function execute result is wrong

[bigwhite]

Describe the bug, including details regarding any error messages, version, and platform.

When I read the code in executor.go， I found a maybe problem:

initially, output is a copy of prealloc. After some operation to output, the output may be different to prealloc, e.g. some slice header. (BufferSpan.Buf)

The code may pass the output to emitResult rather than the prealloc.

I'm not sure this is a real problem because I've no test case to produce the problem.

Component(s)

Go

[zeroshade]

I think you might actually be correct here, it would be great if we could come up with a test case that exemplifies it first though before trying to put a fix in.