-
Couldn't load subscription status.
- Fork 79
SLVS-2481 Cache analyzer assembly contents #6396
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
SLVS-2481 Cache analyzer assembly contents #6396
Conversation
…read to improve performance of first analysis.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Caching should be done at
RoslynAnalysisConfigurationProviderlevel to simplify implementation and design; - If no analyzers and no active rules have changed,
RoslynAnalysisConfigurationas a whole should be cached; - Cache miss scenario doesn't need to be optimized, as it only can happen on config scope or binding change
|
@georgii-borovinskikh-sonarsource, I have introduced the The caching of the analyzer assembly contents should stay inside the |
src/RoslynAnalyzerServer/Analysis/Configuration/RoslynAnalysisConfigurationProvider.cs
Outdated
Show resolved
Hide resolved
src/RoslynAnalyzerServer/Analysis/Configuration/RoslynAnalysisConfigurationProvider.cs
Outdated
Show resolved
Hide resolved
src/RoslynAnalyzerServer/Analysis/Configuration/RoslynAnalysisConfigurationProvider.cs
Outdated
Show resolved
Hide resolved
src/RoslynAnalyzerServer/Analysis/Configuration/AnalysisConfigurationParametersCache.cs
Outdated
Show resolved
Hide resolved
src/RoslynAnalyzerServer/Analysis/Configuration/RoslynAnalysisConfigurationProvider.cs
Outdated
Show resolved
Hide resolved
…sut Resharper configuration
…ameters used to calculate those configurations
🤖 Pull Request summaryUpdates Roslyn analyzer configuration to use caching and preloading. • Adds caching to Review focus areas:
|
a2d47ce
into
feature/sqvs-roslyn-plugin



SLVS-2481