INTERNAL: Move a default_topkeys variable

[ing-eoking]

🔗 Related Issue

• jam2in/arcus-works#506

⌨️ What I did

• default_topkeys 변수를 memcached.c 에서 topkey.c 로 이동시킵니다.
• memcached.h에서 전역화되어 있는 변수이기에 프로토콜 분리에 문제는 없으나 topkey에 관한 변수이므로 topkey 모듈에 있는 편이 자연스럽다고 생각했습니다.

[jhpark816]

zookeeper 사용하는 응용은 아래와 같이 호출하여 zk handle을 리턴받습니다. 그리고, zk handle을 가지고 zookeeper 연산을 호출합니다.

    zinfo->zh = zookeeper_init(zinfo->ensemble_list, arcus_zk_watcher,
                               arcus_conf.zk_timeout, &zinfo->myid, zinfo, 0);

default_topkeys도 마찬가지로 topkeys 기능 사용에 대한 handle 같은 개념입니다. 따라서, 이는 topkeys 기능을 사용하는 응용인 memcached.c 파일에 있는 것이 정상입니다.

    if (settings.topkeys > 0) {
        default_topkeys = topkeys_init(settings.topkeys);
    }

[ing-eoking]

@jhpark816

현재는 default_topkeys만 사용 중인데, 그럼에도 topkeys_init 함수가 여러 topkeys handler를 반환하도록 구현된 것은, 다양한 핸들러를 사용할 수 있는 상황을 고려한 것인가요?

[jhpark816]

topkeys 기능이 다양한 핸들(handle)을 가질 수 있도록 구현된 것인지를 확인하지 않은 상태이므로, 한번 봐 주세요. 여튼, topkeys 기능의 현재 사용법으로 보면, 핸들을 응용에서 가지고 있는 것이 맞다라는 것입니다.

[ing-eoking]

여튼, topkeys 기능의 현재 사용법으로 보면, 핸들을 응용에서 가지고 있는 것이 맞다라는 것입니다.

기존 로직이 맞는 방향이므로 현재 PR은 닫겠습니다.