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

Add caching to configurationless migration #866

Closed
wants to merge 1 commit into from
Closed

Conversation

stroxler
Copy link
Contributor

Summary:
This diff adds functionality around caching operations determining which files are included in a given .pyre_configuration.local.

Determining which files are included can be expensive. We need to perform operations to determine the following:

  • What are all files that can be included in this configuration? We ask hg (or perform a glob) for this
  • What are all files that should be excluded from this configuration?
  • What are all files included in nested configurations where those configurations should have priority over this one? We perform the first step in this list once for each nested configuration.

The steps above are performed once for each file we're linting, so determining which files should be included is an expensive process. We attempt to mitigate it by caching a lot of these operations.

Differential Revision: D57681433

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57681433

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57681433

stroxler pushed a commit that referenced this pull request May 23, 2024
Summary:
Pull Request resolved: #866

This diff adds functionality around caching operations determining which files are included in a given .pyre_configuration.local.

Determining which files are included can be expensive. We need to perform operations to determine the following:
* What are all files that can be included in this configuration? We ask `hg` (or perform a glob) for this
* What are all files that should be excluded from this configuration?
* What are all files included in nested configurations where those configurations should have priority over this one? We perform the first step in this list once for each nested configuration.

The steps above are performed once for each file we're linting, so determining which files should be included is an expensive process. We attempt to mitigate it by caching a lot of these operations.

Differential Revision: D57681433
Summary:

This diff adds functionality around caching operations determining which files are included in a given .pyre_configuration.local.

Determining which files are included can be expensive. We need to perform operations to determine the following:
* What are all files that can be included in this configuration? We ask `hg` (or perform a glob) for this
* What are all files that should be excluded from this configuration?
* What are all files included in nested configurations where those configurations should have priority over this one? We perform the first step in this list once for each nested configuration.

The steps above are performed once for each file we're linting, so determining which files should be included is an expensive process. We attempt to mitigate it by caching a lot of these operations.

Reviewed By: MaggieMoss

Differential Revision: D57681433
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57681433

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 573e772.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants