Content length 0 for second route.rule backend about influxdb-srelay HOT 1 OPEN

Hi @toni-moreno, @matejzero,

First of all ,thanks for the great implementation, it is very useful for us @toni-moreno.

This issue is stale for quite a while, but I wanted to leave an update since we have ran into the same issue.

The issue is most likely occurs due to a problem with the handleWriteSingle function.
https://github.com/toni-moreno/influxdb-srelay/blob/master/pkg/cluster/influxcluster.go#L168-L178

The function will be called in a loop from a parent function that loops through on all matching routes.

The issue is that:
https://github.com/toni-moreno/influxdb-srelay/blob/master/pkg/cluster/influxcluster.go#L178 will be called twice.

The getBuf() function is doing the following:
c.bufPool.Get().(*bytes.Buffer)

bufPool is:
bufPool sync.Pool

sync.Pool.Get() will result in this:
https://golang.org/pkg/sync/#Pool.Get

As you can see the issue lies here:
Get selects an arbitrary item from the Pool, removes it from the Pool, and returns it to the caller.

The first call will remove the data and the second call will only get an empty result.

The one big HACluster with multiple boxes is working, because it is using the handleWriteHA function, which takes the buffer the same way, but loops through on all endpoints with the same variable inside the same function.
https://github.com/toni-moreno/influxdb-srelay/blob/master/pkg/cluster/influxcluster.go#L221-L254

Could you take a look if I am right?

Many thanks,
Peter

from influxdb-srelay.