INTERNAL: use groupingKeys in asyncGet(s)Bulk

[oliviarla]

🔗 Related Issue

• CLEANUP: tidy MemcachedClient codes #800 (comment)

⌨️ What I did

addKeyToMap() 메소드를 호출하지 않고,
호출 쪽에서 해당 로직을 직접 가지는 것이 좋을 것 같은 데,
확인하고 동의한다면 PR 보내 주세요.

• asyncGetBulk, asyncGetsBulk에서 노드에 따라 키를 분배할 때 groupingKeys 메서드를 사용하도록 변경합니다.
• 이를 통해 일관적인 내부 동작을 가질 수 있습니다.
• addKeyToMap, getWholeChunkSize 메서드는 제거하였습니다.

[jhpark816]

리뷰 완료

[jhpark816]

MemcachedClient 클래스로 옮겼으므로, 아래 코드는 없어도 됩니다.

MemcachedConnection conn = getMemcachedConnection();

[jhpark816]

아래와 같은 코드이면 좋겠습니다.

        op = node.enabledMGetsOp() ? opFact.mgets(keyList, cb)
                                   : opFact.gets(keyList, cb);

[jhpark816]

여기는 질문입니다.

node == null 상황에서 operation 객체를 만들어 connection에 추가할 필요가 있나요?

[oliviarla]

현재는 connection에서 node가 null일 경우 op를 cancel 시키고 있습니다.
여러 노드에 다양한 mget op를 보내놓고 node가 null일 경우가 발생하면 cancel로 인해 예외가 발생하기 때문에, get/mget 요청이 서버에서 처리되었지만 결과값을 얻을 수 없는 상황이 발생할 수 있습니다.
이를 막으려면 node가 null인 상황이 발생할 때 기존 ops list에 존재하는 모든 op를 cancel시키고 이 op도 cancel시키고 예외를 던지면 될 것 같습니다.

[jhpark816]

node가 null 이라는 의미는 현재 남아 있는 node가 없다는 것이므로,
모든 key들이 null node에 mapping 될 것 같고, 생성하는 모든 op가 생성될 것 같습니다.
맞지 않는 부분이 있으면, 알려주세요.