Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

profiler: don't upload full profiles if delta profiling is enabled #1187

Merged
merged 3 commits into from Mar 1, 2022

Commits on Feb 28, 2022

  1. profiler: don't upload full profiles if delta profiling is enabled

    The full heap, mutex, and block profiles are not needed when delta
    profiling is enabled. Removing them (especially the full heap profile)
    results in a significant bandwidth and storage cost reduction.
    nsrip-dd committed Feb 28, 2022
    Copy the full SHA
    70354df View commit details
    Browse the repository at this point in the history

Commits on Mar 1, 2022

  1. profiler: make delta profile check logic more clean

    Removed a redundant condition (deltaProfile is documented to return nil
    if delta profiles are disabled) and changed the order of conditions to
    match the default configuration (detla profiles first).
    nsrip-dd committed Mar 1, 2022
    Copy the full SHA
    fd3d644 View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    cfe64c9 View commit details
    Browse the repository at this point in the history