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.

Sign up for GitHub

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

Jump to bottom

Report ML-powered query enablement in the init status report #907

Merged
merged 10 commits into from Feb 7, 2022
Merged

Report ML-powered query enablement in the init status report #907

merged 10 commits into from Feb 7, 2022

Commits on Feb 3, 2022

  1. Convert status report comments to documentation

    @henrymercer
    henrymercer committed Feb 3, 2022
    Copy the full SHA
    a005206 View commit details
    Browse the repository at this point in the history

  2. Add ML-powered queries enablement to init status report 

    We report this information in the `init` status report rather than the
`analyze` status report so we can gather data about timeouts.
    @henrymercer
    henrymercer committed Feb 3, 2022
    Copy the full SHA
    1cddec9 View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2022

  1. Only add ML-powered queries pack if the user didn't manually request it

    @henrymercer
    henrymercer committed Feb 4, 2022
    Copy the full SHA
    9f32fc9 View commit details
    Browse the repository at this point in the history

  2. Add "multiple" report for ML-powered JS query enablement 

    When multiple ML-powered JS packs are in scope (an unsupported
scenario), the status report is "multiple".
    @henrymercer
    henrymercer committed Feb 4, 2022
    Copy the full SHA
    537b2f8 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into henrymercer/report-ml-powered-query-enablement

    @henrymercer
    henrymercer committed Feb 4, 2022
    Copy the full SHA
    ad40e4a View commit details
    Browse the repository at this point in the history

  4. Update getMlPoweredJsQueriesStatus doc

    @henrymercer
    henrymercer committed Feb 4, 2022
    Copy the full SHA
    501fe7f View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2022

  1. Nit: Simplify code with optional chaining

    @henrymercer
    henrymercer committed Feb 7, 2022
    Copy the full SHA
    f888be7 View commit details
    Browse the repository at this point in the history

  2. Limit cardinality of ML-powered JS queries status report 

    Some platforms that ingest this status report charge based on the
cardinality of the fields, so here we restrict the version strings we
support to a fixed set.
    @henrymercer
    henrymercer committed Feb 7, 2022
    Copy the full SHA
    c95a3d8 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into henrymercer/report-ml-powered-query-enablement

    @henrymercer
    henrymercer committed Feb 7, 2022
    Copy the full SHA
    cc622a0 View commit details
    Browse the repository at this point in the history

  4. Add more documentation for ML-powered JS queries status report 

    Also be more explicit about which version strings are reportable in
the code.
    @henrymercer
    henrymercer committed Feb 7, 2022
    Copy the full SHA
    03c64ef View commit details
    Browse the repository at this point in the history