Skip to content

Conversation

Jamesbarford
Copy link
Contributor

Updates the schema.md to the newer formatting we were using. I have a couple of questions;

  • Do we still use collector_progress? (answered - no we do not use it in the new system)
  • Do we still use artifact_collection_duration?
  • What is cache in raw_self_profile?
  • Is include in the pull_request_build table a comma separated string of options?
  • Is exclude in the pull_request_build table a comma separated string of options?

@Jamesbarford Jamesbarford force-pushed the chore/update-schema.md branch from 3c4b637 to 064769a Compare October 20, 2025 09:33
@Kobzol
Copy link
Member

Kobzol commented Oct 20, 2025

Do we still use artifact_collection_duration?

Not in the new system, but probably some code still reads it.

What is cache in raw_self_profile?

cache is the older name for scenario.

Is include in the pull_request_build table a comma separated string of options?

Yes to both include and exclude.

@Jamesbarford Jamesbarford force-pushed the chore/update-schema.md branch from 064769a to dfa5eb3 Compare October 21, 2025 07:23
Copy link
Member

@Kobzol Kobzol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Left a few comments.

@Kobzol Kobzol enabled auto-merge October 21, 2025 10:20
@Kobzol Kobzol added this pull request to the merge queue Oct 21, 2025
Merged via the queue into rust-lang:master with commit bfbfe92 Oct 21, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants