Skip to content

Improve implementation of classloader caching #7644

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

Merged
merged 2 commits into from
Feb 8, 2019

Conversation

retronym
Copy link
Member

@retronym retronym commented Jan 15, 2019

  • Fix logic error in reference counted closing
  • Add a new policy to cache regardless of timestamps
  • When encountering non file-based classpath entries, be optimistic
    rather than disabling caching.

@scala-jenkins scala-jenkins added this to the 2.13.0-RC1 milestone Jan 15, 2019
@retronym retronym added the WIP label Jan 15, 2019
@retronym retronym changed the title WIP Improve implementation of classloader caching Improve implementation of classloader caching Jan 15, 2019
retronym added a commit to retronym/scala that referenced this pull request Jan 15, 2019
retronym added a commit to retronym/scala that referenced this pull request Jan 15, 2019
@retronym retronym force-pushed the topic/classloader-cache-fix branch 2 times, most recently from b186b9f to d44701b Compare January 22, 2019 00:14
@retronym retronym force-pushed the topic/classloader-cache-fix branch 3 times, most recently from e770317 to b3e1ccb Compare January 22, 2019 00:33
@retronym retronym removed the WIP label Jan 22, 2019
retronym added a commit to retronym/scala that referenced this pull request Jan 22, 2019
  - Fix logic error in reference counted closing
  - Add a new policy to cache regardless of timestamps and
    regardless of directory elements in the classpath.
@retronym retronym force-pushed the topic/classloader-cache-fix branch from b3e1ccb to b4d31c7 Compare February 4, 2019 02:28
retronym added a commit to retronym/scala that referenced this pull request Feb 4, 2019
retronym added a commit to retronym/scala that referenced this pull request Feb 4, 2019
retronym added a commit to retronym/scala that referenced this pull request Feb 4, 2019
@jvican jvican self-requested a review February 4, 2019 08:44
@adriaanm adriaanm self-assigned this Feb 4, 2019
@adriaanm adriaanm merged commit 3259915 into scala:2.13.x Feb 8, 2019
@jvican jvican removed their request for review February 8, 2019 16:52
retronym added a commit to retronym/scala that referenced this pull request Feb 20, 2019
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.

4 participants