Dominik Charousset

Results 200 comments of Dominik Charousset

@awelzel if you want to give it a spin: the branch `issue/gh-276` has the necessary changes.

> The branch unfortunately doesn't compile for me. I've only tested with Clang, sorry for not cross-checking the patch with other vendors! After reproducing this on GCC, it looks like...

While we've dropped the actual ALM transport, the new store implementation is merged.

> The hex output below looks like it's mostly small 4 byte numbers, or maybe enum values, for which this happens. Any ideas what's going on here? Seems like the...

> error: Failed to load value for key (59.154.216.47, description) while importing Broker store persistenceStore to table I've stepped through the sources. This error gets printed if `broker::store::get` fails. And...

@awelzel thanks for the confirmation. Isn't `0.0001 sec` causing Zeek to basically run in a hot loop? Do you see memory piling up during the run? I would expect that...

I've reproduced this locally with this small patch to Zeek to see what Broker is complaining about: ```diff diff --git a/src/broker/Manager.cc b/src/broker/Manager.cc index 37e7f9d4c..4b1376470 100644 --- a/src/broker/Manager.cc +++ b/src/broker/Manager.cc @@...

I believe so. I believe I did some benchmarking for this (with and without batching), probably a good idea to revisit that now.

> The proposal is to switch both types to `uint64` internally That would mean that Zeek can't represent dates before 1970. Did you also consider `int64_t` instead? Zeek could represent...

Here's a starting point for the Telemetry documentation: https://github.com/zeek/zeek/wiki/Telemetry. So far it covers how to get Zeek to expose the Metrics via Prometheus and gives a brief overview plus some...