Daniele
Daniele
Oh also, `luarocks make` requires the rockspec file as argument. I don't know if this changed in a recent version of luarocks or why it used to work for you
> I believe if /usr/local isn't writable by default on your machine, that it may be your responsibility to either: Well, /usr/local on my laptop is not writable exactly to...
np. Fwiw we've been running this patch in production for a month now and didn't have any issue.
@thibaultcha Sorry, I completely missed you comments. protocol v3: https://github.com/apache/cassandra/blob/trunk/doc/native_protocol_v3.spec#L130 > this logic will not prevent two workers from using similar stream ids at the same time True, but that's...
> Then what is the goal of the mutex in this patch? Wasn't it to prevent two workers from using the same stream id? What I wanted to prevent is...
Sorry for the delay on this. I now have some spare time so I'll work in finish upstreaming this change. I have already made most changes suggested in the above...
A bunch of changes here: I removed the mutex as discussed. This meant I could move the stream_id implementation inside `cassandra/init.lua` since I don't need anything openresty specific anymore. To...
I've also rebased on master so hopefully the tests should now pass
@adriancole bump. zipkin2-api.yaml says which fields you can leave absent. zipkin-api.yaml doesn't say anything about it. Since they're not listed as required, I'd assume everything is optional and I can...