Merge pull request #28 from reiseburo/alloc-reduction

Identify and address some performance hot spots
reiseburo · May 21, 2020 · f6d4b0a · f6d4b0a
2 parents 6016a1b + 36ce188
commit f6d4b0a
Show file tree

Hide file tree

Showing 9 changed files with 421 additions and 109 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -59,3 +59,6 @@ handlebars = "~3.0.1"
 
 # Needed for time management
 chrono = "~0.4.11"
+
+# Needed to tag rules and actions with their own unique identifiers
+uuid = { version = "~0.8.1", features = ["v4"] }
diff --git a/README.adoc b/README.adoc
@@ -453,6 +453,17 @@ For TLS connections, you can use the `openssl` `s_client` command:
 echo  '<13>1 2020-04-18T15:16:09.956153-07:00 coconut tyler - - [timeQuality tzKnown="1" isSynced="1" syncAccuracy="505061"] hello world' | openssl s_client -connect localhost:6514
 ----
 
+
+=== Profiling
+
+Profiling `hotdog` is best done on a Linux host with the `perf` tool, e.g.
+
+[source,bash]
+----
+RUST_LOG=info perf record --call-graph dwarf -- ./target/debug/hotdog -c ./hotdog.yml
+perf report -ng
+----
+
 == Similar Projects
 
 `hotdog` was originally motivated by challenges with

diff --git a/src/kafka.rs b/src/kafka.rs
@@ -166,7 +166,8 @@ impl Kafka {
                                 Ok(_) => {
                                     timer.stop(handle);
                                     m.counter("kafka.submitted").count(1);
-                                    m.counter(&format!("kafka.submitted.{}", &kmsg.topic)).count(1);
+                                    m.counter(&format!("kafka.submitted.{}", &kmsg.topic))
+                                        .count(1);
                                 }
                                 Err((err, _)) => {
                                     match err {