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

Use a different path for the Kernel cache if possible #2036

Merged
merged 5 commits into from
Dec 26, 2019

Conversation

sergeyklay
Copy link
Contributor

Hello!

  • Type: bug fix
  • Link to issue: -

In raising this pull request, I confirm the following:

  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I updated the CHANGELOG

Small description of change:

This patch fixes a cache collision issue. The issue is after creating the cache and filling it with a project-specific configuration, there is no way to invalidate it. Any next project will use the same Kernel cache and the same Kernel configuration (if any).

For more details see the test.

Thanks

This patch fixes a cache collision issue. The issue is after creating the
cache and filling it with a project-specific configuration, there is no
way to invalidate it. Any next project will use the same cache and the
same Kernel configuration (if any).
@codecov
Copy link

codecov bot commented Dec 26, 2019

Codecov Report

Merging #2036 into development will increase coverage by <.01%.
The diff coverage is 48.78%.

@@                Coverage Diff                @@
##             development    #2036      +/-   ##
=================================================
+ Coverage          35.62%   35.63%   +<.01%     
- Complexity          8216     8218       +2     
=================================================
  Files                562      562              
  Lines              45330    45340      +10     
=================================================
+ Hits               16149    16157       +8     
- Misses             29181    29183       +2

@sergeyklay sergeyklay merged commit 75b425d into development Dec 26, 2019
@sergeyklay sergeyklay deleted the fix/invalidate-cache branch December 26, 2019 23:10
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.

1 participant