Dawn Minion
Dawn Minion
@Ashex Thank you for posting this. Hit the exact same issue this morning. Now, what's interesting is this behaviour is not exhibited locally on my development machine - If I...
@Ashex How strange it works fine locally for you as well. Ubuntu 18.04 locally and 16.04 on the server. Hopefully will have time today to run that, and will post...
@Ashex So, after a good amount of debugging, I've narrowed it down to [this call here](https://github.com/skoczen/will/blob/master/will/backends/analysis/history.py#L13) - Disabling this line (and the rest of the logic) makes it so there's...
@Ashex Looking at the code though, doesn't it simply chop it to 19 elements for the context object, but the size in Redis/storage is unlimited? Since only the context object...
This one was quite a refactor - I moved everything as function parameters where originally it read the global vars. None of the public API signatures are changed, so I...
I wasn't quite sure what you folks would prefer here to resolve this, and this seemed the least invasive as it's a simple copy. Let me know if you want...
Oh i see. Could I ask why though? 16 bytes increase seems fairly negligible, and with the removal of uuidMap it would end up being an equal amount of total...
Thank you for the explanation. Wouldn’t the memory usage be the same though? 16 bytes in uuidMap today, or 16 bytes in Path struct. This change only moves it from...
~Wouldn’t it be the same here though? UUID is only set if the path is added via the gRPC AddPath, otherwise the pointer is nil. So if 1mil routes were...
/cc @unicolet