blemale / scaffeine

93%

master: 94%

LAST BUILD ON BRANCH Scala_1-12_follow_up

branch: Scala_1-12_follow_up

CHANGE BRANCH

x

Reset

• Scala_1-12_follow_up
• 002fc690be53c741ecb9aa3cbd978c477350d08b
• 0795898eefad968d9ce67f145a397247d8412f4a
• 100d96e5e9996add0635a0a39075db65925f83ba
• 449ba9bda8ab3eae28425c719dceeaef6dfeaa64
• 6560ac4b0bcf7644e572153ab7bff152b6c9e0bc
• 8678608ea1ec76fef254c6c2d292e6dea1cc6a0f
• Add_caffeine_2_5_0_support
• Add_depency_graph_plugin
• Add_sbt_updates_plugin
• Add_scalac_flags
• Add_snyk_badge
• Better_snyk_integration
• Build_remove_sbt-pgp
• Bump_caffeine_to_2_3_5
• Bump_caffeine_version
• Bump_caffeine_version_to_2_5_6
• Bump_dependencies
• Bump_sbt-pgp
• Bump_scala_2-13_to_RC1
• Bump_scala_2-13_to_RC2
• Cleanup_badges_in_README
• Cross_compile_scala_2_13
• Dont_push_snapshot_for_PR
• Fix_release_process
• Fix_scaladoc
• Fix_snapshot_releases_on_travis
• Refator_travisci_build
• Remove_hardcoded_version_in_README
• Remove_java8compat_dependency
• Remove_useless_executioncontext
• Scala_2_12
• Support_caffeine_2-7
• Update_README
• Update_caffeine_to_2_9_1
• Update_caffeine_version
• Update_examples_with_new_version_2-0-0
• Update_readme
• Update_readme-1
• Update_readme_with_new_version
• Update_scala_and_scoverage
• Update_scalafmt-core_to_2.5.1
• Update_scalatest_to_3.2.0-M1
• Update_version_in_README
• Upgrade_sbt_to_version_1-2-8
• Upgrade_scala_to_version_2-11-12
• Upgrade_scala_to_version_2-11-12_and_2-12-8
• a519c9073f326bae97103de6ee39581235b0e610
• bastien.lemale/Migrate_to_sbt-ci-release
• bastien.lemale/Migrate_to_scalafmt
• bastien.lemale/Small_clean_ups
• bastien.lemale/remove_sbt-coursier
• bastien.lemale/update_sbt
• blemale-patch-1
• blemale/add_test_for_eviction_listener
• blemale/migrate_to_github_actions
• blemale/update_caffeine_to_3-0-2
• e4d651c662f39425d0f68b5629a0f36af1cbebec
• f0b264a7c6318f13c544016e92773f5c566d57d0
• main
• master
• update/scalafmt-core-2.7.4
• update_caffeine_to_2.8.0
• update_caffeine_version_to_2_8_4
• update_scalatest_scalactic_to_3.2.0
• use_openjdk_for_ci
• v1.0.0
• v1.1.0
• v1.2.0
• v1.3.0
• v2.0.0
• v2.0.0-M1
• v2.0.0-M2
• v2.0.0-M3
• v2.1.0
• v2.2.0
• v2.3.0
• v2.4.0
• v2.5.0
• v2.6.0
• v3.0.0
• v3.1.0
• v4.0.0
• v4.0.1
• v4.0.2

Build # 80

Build Type

push

travis-ci

Committed by blemale

Commit Message

Core: Fix bulk loading with scala 2.12

When allLoader is not specified when building a LoadingCache or an
AsyncLoadingCache the method getAll is broken on scala 2.12 and
throw an java.lang.UnsupportedOperationException.

Indeed Caffeine use reflection to know if loadAll is overridden and
it seems that scala 2.12 bytecode generation break this, see
https://github.com/ben-manes/caffeine/blob/v2.3.3/caffeine/src/main/java/com/github/benmanes/caffeine/cache/LocalLoadingCache.java#L54.

To fix the issue this commit implements the adapters in java
instead of scala, thus when not providing an allLoader the loadAll
method is not seen overridden.

Run Details

62 of 67 relevant lines covered (92.54%)

0.93 hits per line