INTERNAL: get NullValue instance from arcus server

naver / arcus-spring

ARCUS as a caching provider for the Spring Cache Abstraction

Apache License 2.0

26 stars 16 forks source link

Closed oliviarla closed 4 months ago

oliviarla commented 4 months ago

ArcusCache 클래스가 AbstractValueAdaptingCache 클래스를 상속받도록 변경합니다. 이를 위해 lookup 메서드를 재정의했습니다.
이를 통해 ArcusCache 클래스에서 기본적으로 AbstractValueAdaptingCache 클래스의 get 메서드가 사용됩니다. 그리고 이 때 NullValue 객체 조회 시 null을 반환해주게 됩니다.
ArcusCacheConfiguration 클래스에서는 cacheNullValues 속성을 설정할 수 있도록 합니다.
기존에 get(Object key) 메서드에서는 wantToGetException 속성이 유효했지만, 현재는 해당 설정이 무효하여 예외가 발생하면 그대로 전파됩니다.

oliviarla commented 4 months ago

@uhm0311 리뷰 반영했습니다.

uhm0311 commented 4 months ago

@oliviarla

allowNullValue 여부에 따라 null을 저장하려고 시도했을 때 성공/실패를 검증하는 테스트 코드도 추가해주세요.

oliviarla commented 4 months ago

@uhm0311 테스트 코드를 추가하기 위해 NullValue.INSTANCE 정적 필드를 이용해 객체를 가져오려고 했으나, 현재 spring 4.3.0 버전 기준으로는 package private 이므로 NullValue 객체를 생성할 수 없습니다.

spring 4.3.10 버전 이후부터는 public으로 바뀌었기 때문에 NullValue.INSTANCE 정적 필드를 사용할 수 있으나, 현재로서는 reflection을 통해서만 객체를 생성할 수 있습니다.

spring 패치 버전을 올리고 테스트 코드를 작성할 지, 아니면 reflection으로 작성해둘지 의견을 주시면 감사하겠습니다.

oliviarla commented 4 months ago

reflection을 이용한 테스트 코드 작성해두었습니다.
nullValue 객체가 싱글톤으로 관리되고 있어 ArcusCache의 부모 클래스로부터 생성된 nullValue 객체를 가져오기 위해 메서드를 사용했습니다.

oliviarla commented 4 months ago

@uhm0311 이미 테스트코드에서 reflection을 통해 protected 접근 제어자인 toStoreValue 메서드를 사용하고 있는데, 어떤 의미인가요?

uhm0311 commented 4 months ago

@oliviarla

리플렉션이나 4.3.10으로 업그레이드 하지 않고도 NullValue 접근이 가능하다면 시도해보라는 의미였습니다. 불가능하다면 4.3.10으로 업그레이드 하시죠.