INTERNAL: save configuration instance in ArcusCache - Githubissues

naver / arcus-spring

ARCUS as a caching provider for the Spring Cache Abstraction

Apache License 2.0

26 stars 16 forks source link

INTERNAL: save configuration instance in ArcusCache #100

Closed oliviarla closed 2 months ago

oliviarla commented 2 months ago

🔗 Related Issue

https://github.com/naver/arcus-spring/issues/65

⌨️ What I did

인자가 없는 생성자를 deprecate 시켜 사용하지 않도록 권장합니다. 대신 필수 인자를 입력해야 하는 생성자를 제공합니다.
반드시 내부에서만 사용되어야 하는 필드인 keyLockProvider를 제외하고는 모두 configuration을 통해 접근하도록 변경했습니다.
이를 통해 필수 인자가 아닌 옵션들의 검증은 ArcusConfiguration 한 군데에서만 거치면 됩니다. setter, getter도 ArcusCache 클래스에서 제공해줄 필요가 없습니다.
참고로 redis, couchbase도 XXXCache 클래스에 XXXCacheConfiguration 클래스를 필드로 두고 있습니다.

jhpark816 commented 2 months ago

@uhm0311 최종 리뷰 바랍니다.