search

sksamuel / aedile

Kotlin Wrapper for Caffeine
Apache License 2.0
164 stars 15 forks source link

Divided caches to api and implementation #8

Closed zTrap closed 7 months ago

zTrap commented 1 year ago

Fixes #7

sksamuel commented 1 year ago

Can we think of a better name than wrapper, I thought the old names were fine. Maybe create a new package if we want to reuse them.

zTrap commented 1 year ago

Yes I dont like to use *wrapper in names either, but the old names are clashing with original lib classes and can be imported by mistake. Also the actual functionality of these classes isn't caching, they do (un-)wrapping and (down/up-)streaming, so the old names isn't represent what they do.

As an option I can suggest to use *facade instead of *wrapper.

turboezh commented 1 year ago

Hi. Imho, it's looks more like adapter rather than facade. I can suggest something like (Coroutine)CacheAdapter.