Park JunHyun

@HarryKim93 ARCUS에서 prefix는 캐시 아이템들의 논리적인 그룹으로 RDBMS에서 table에 해당하는 개념입니다. 현재 prefix 사용 목적은 아래 2가지 입니다. - prefix 단위로 `아이템 통계 (with stats prefixes)`와 `연산 통계 (with stats...

@HarryKim93 @dongwooklee96 오늘 바쁜 일들이 많아 확인하지 못했는 데, 조만간 확인하고 의견을 달도록 하겠습니다.

@HarryKim93 의견입니다. - p = 1/4로 하면 좋겠습니다. - 매크로 or 상수 변수로 설정을 변경할 수 있으면 좋겠습니다. - 향후에 1/2와 1/4인 경우를 비교해 볼 수 있습니다. - random value...

@dongwooklee96 다행입니다. sorted set에 관해 이슈 https://github.com/naver/arcus-memcached/issues/480 에 의견을 남겨 두었으므로 확인해 주기 바랍니다.

@minkikim89 core 부분에서는 parsing 관련 코드 밖에 없겠네요. 이번에 persistence 코드를 보니 bkey range와 eflag filter 다루는 코드가 있어, items 모듈과 core에서도 유사 코드가 있을 거라 생각한 것인데, core에는 없는...

@minkikim89 OK. 본 이슈에 관련한 코드를 확인해 본 후, 수정이 필요 없으면 close 하겠습니다.

@MinWooJin @minkikim89 본 PR은 engine interface를 변경하고 있습니다. 이번에 engine interface를 변경한다면, 보다 general한 engine interface를 가지도록 수정한 후에 본 PR의 내용을 반영하는 것이 좋겠어요. 보다 general하면서 향후 engine interface...

@minkikim89 code 기능의 정확성 위주로 먼저 review 해 주시고, coding style 등은 나중에 review 합시다.

@computerphilosopher stats_prefix 정보를 갱신하는 부분에서 logging하자고 하여 진행된 상태입니다. arcus-memcached 코드를 보면서 logging에 더 적합한 위치나 방법이 있던가요 ?

@computerphilosopher 본 기능의 구현에 있어, 만족해야 할 사항이 있습니다. - stats_lock을 추가로 잡지 않는다. 즉, stats_lock을 잡게 되는 코드에 logging 기능 추가 위 요구 사항을 만족시키면서 적절한 위치와 방안이 있다면,...