Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5d4bfe50 authored by lifr's avatar lifr Committed by Luke Huang
Browse files

Add one field (sampling_rate_denom) in NetworkDnsEventReported 

The sample rate of DNS stats(to statsd) is 1/sampling_rate_denom.

Bug: 119862317
Test: ./out/host/linux-x86/bin/statsd_testdrive 116
Merged-In: Ifd496f01ceaeb48de0361280c2315c3733c24f66
Merged-In: If142d8a764437e6a349b122b241c72d069071846
(cherry picked from commit 71b457e4e13ae130fceeec45f19c8430c938e172)

Change-Id: Ic1cdc849e024d4c3c220914a0a62316dd2566748
parent 2cc79562

DnsProxyListener.cpp

+5 −5
Original line number Diff line number Diff line
@@ -311,11 +311,11 @@ void reportDnsEvent(int eventType, const android_net_context& netContext, int la 
    const std::string& dnsQueryStats = event.dns_query_events().SerializeAsString();

    stats::BytesField dnsQueryBytesField{dnsQueryStats.c_str(), dnsQueryStats.size()};

    event.set_return_code(static_cast<ReturnCode>(returnCode));

    android::net::stats::stats_write(android::net::stats::NETWORK_DNS_EVENT_REPORTED,

                                     event.event_type(), event.return_code(),

                                     event.latency_micros(), event.hints_ai_flags(),

                                     event.res_nsend_flags(), event.network_type(),

                                     event.private_dns_modes(), dnsQueryBytesField);

    android::net::stats::stats_write(

            android::net::stats::NETWORK_DNS_EVENT_REPORTED, event.event_type(),

            event.return_code(), event.latency_micros(), event.hints_ai_flags(),

            event.res_nsend_flags(), event.network_type(), event.private_dns_modes(),

            dnsQueryBytesField, event.sampling_rate_denom());



    const auto& listeners = ResolverEventReporter::getInstance().getListeners();

    if (listeners.size() == 0) {

stats.proto

+3 −0
Original line number Diff line number Diff line
@@ -249,4 +249,7 @@ message NetworkDnsEventReported { 
    // Additional pass-through fields opaque to statsd.

    // The DNS resolver Mainline module can add new fields here without requiring an OS update.

    optional DnsQueryEvents dns_query_events = 8;



    // The sample rate of DNS stats (to statsd) is 1/sampling_rate_denom.

    optional int32 sampling_rate_denom = 9;

}