Do not produce dependency items for the quick pulse calls

[SergeyKanzhelev]

Using app insights in VS I keep seeing these “Dependecy /QuickPulseService.svc” lines show up

What is this? It’s kind of annoying that it shows up every 5 seconds!

VS shows all telemetry items – filtered and not filtered. One of the reasons for this is that we wanted to show “sampled out” requests in the local search. If QP generate dependencies events – SDK spends many cycles to allocate and initialize those. So this is clearly a wasted effort.

I think the solution is to filter those calls in Dependency Tracking module, not in the filter.

[SergeyKanzhelev]

I think both should be happening:

1. Filtering in Dependency Collector by default
2. As @gardnerjr pointed - this should be included into filtering processor:

if (IsQuickPulse(item))
{
    TelemetryDebugWriter.WriteTelemetry(item, "QuickPulse Filtering");
    return;
}

It will ends up with the prefix: Application Insights Telemetry (filtered by QuickPulse Filtering): in the output that will be marked by VSIX

[gardnerjr]

the above was just what i did locally in a filter to make them go away in one of my projects and test the filtering, but yes, there should be some filtering by the AI SDK itself, it shouldn't be considering outbound quickpulse calls to be "dependencies" in my application.

from the SDK side, i don't think there should even be the TelemetryDebugWriter call, as it isn't real telemetry. it should get filtered out silently, it isn't telemetry being sent by my application and not clog up output windows or debug tools.

[cijothomas]

The quickpulse dependency is showing in Ibiza portal as well..

[gardnerjr]

@cijothomas yes, because they're getting sent, that's the bug :) once they're sent, they show up everywhere. VS search, AA, Ibiza, Draft, powerbi, etc.

[cijothomas]

yea will fix it.. too much noise this is causing while debugging.

[AlexBulankou]

We misunderstood the issue :( We thought it was only showing in VS. Yes, need to fix for 2.2 Stable.