Commit d177461a authored Jan 06, 2020 by Ruchir Rastogi

Fix bug in stats_event_benchmark

Now, the number of elements in each simulated stats_event is correctly
sampled from a uniform distribution.

Rerunning the benchmarking tests indicates that truncating the buffer
increases the number of cycles require to log an event by only 1.6%.

Test: m libstatssocket_benchmark
Test: bit libstatssocket_benchmark:*
Change-Id: I8d5c1734861038beec8eeb7ce1f84a24bde25a75

parent d8a8988c

libstats/socket/benchmark/stats_event_benchmark.cpp

+2 −1

Original line number	Diff line number	Diff line
		@@ -22,7 +22,8 @@ static struct stats_event* constructStatsEvent() {
		stats_event_set_atom_id(event, 100);

		// randomly sample atom size
		for (int i = 0; i < rand() % 800; i++) {
		int numElements = rand() % 800;
		for (int i = 0; i < numElements; i++) {
		stats_event_write_int32(event, i);
		}