Skip to content

C++: Add cpp/extraction-information query#21512

Draft
paldepind wants to merge 1 commit intogithub:mainfrom
paldepind:cpp/extraction-information
Draft

C++: Add cpp/extraction-information query#21512
paldepind wants to merge 1 commit intogithub:mainfrom
paldepind:cpp/extraction-information

Conversation

@paldepind
Copy link
Contributor

@paldepind paldepind commented Mar 19, 2026

This PR adds a cpp/extraction-information query similar to the $LANG/extraction-information query for Rust, Java, and C#.

The implementation is consistent with the other languages. For now the query only includes metrics for resolved calls. The difference between this metric and the "Calls with explicit target" in cpp/telemetry/extraction-metrics is that only call in the source are considered.

This distinction is important for evaluating BMN with dependency installation. In the table below the first row shows the existing metric and the remaining rows show the new metric for the project nmap.

We believe that dependency installation has a detrimental effect on this project, but the existing metric looks like an improvement. This is because a lot of additional calls are added in dependencies which causes the number to increase. We the new metric the degradation in the database is clear.

traced BMN BMN + deps inst
Existing call with explicit target 41,261 43,192 51,688
Percentage of calls with call target 100 79.18 71.709
Number of calls with call target 28,983 28,983 22,729
Number of calls with missing call target 0 7,621 8,967

Note that I didn't modify the existing cpp/telemetry/extraction-metrics for two reasons:

  • Going forward I think we should use cpp/extraction-information for consistency with other languages.
  • I didn't want to break anything existing. Maybe cpp/telemetry/extraction-metrics is hooked up to some telemetry and dashboards?

@github-actions github-actions bot added the C++ label Mar 19, 2026
@paldepind paldepind force-pushed the cpp/extraction-information branch from e212be4 to 1af128f Compare March 19, 2026 13:28
@paldepind paldepind force-pushed the cpp/extraction-information branch from 1af128f to 4c525ce Compare March 19, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant