INTERNAL: Add sampling method to random get in set

[jeesup0103]

🔗 Related Issue

• https://github.com/jam2in/arcus-works/issues/571

⌨️ What I did

• Set의 random get 동작에서 selection sampling 방법을 추가했습니다.
  • do_set_elem_traverse_sampling()
• Set 전체 크기의 10% 이하의 크기를 요청하면 hash table 방법을 이용합니다.
• Set 전체 크기의 10% 초과의 크기를 요청하면 sampling 방법을 이용합니다.
• count개의 element를 모두 선택한 뒤 shuffle 합니다.
• 요소를 만났을 때 선택 확률 = (남은 필요 요소 개수) / (남은 탐색 요소 개수)

선택 확률 예시

총 10개에서 3개를 요청하는 상황에서 0~9 Index를 탐색하면서 선택합니다.
- 0번 index에서 뽑을 확률 = 3/10

- 1번 index에서 뽑을 확률
   - 0번 index에서 뽑았을 때,     현재 뽑을 확률 = 2/9
   - 0번 index에서 뽑지 않았을 때, 현재 뽑을 확률 = 3/9

...

3개를 다 뽑게 되면 바로 종료합니다.

[jhpark816]

@ing-eoking @namsic
본 이슈는 빠르게 마무리해야 할 것 같고, 리뷰 진행해 주세요.

만약, 늦어진다면,
기존 방식과 random 방식을 선택하는 조건을 if (1)로 수정하여 기존 방식이 선택되도록 해야 합니다.

    if (count >= info->ccnt || count == 0) { /* Return all */
        fcnt = do_set_elem_traverse_dfs(info, info->root, count, delete, elem_array);
    } else { /* Return some */
        fcnt = do_set_elem_traverse_rand(info, count, delete, elem_array);
    }

[jhpark816]

리뷰 완료

[jhpark816]

질문이 있습니다.
확률적으로 선택하게 되는 데, 최종적으로 count 개를 선택한다는 것이 보장되나요?

참고 사항으로, 아래와 같이 괄호를 넣어주면 나을 것 같습니다.

((rand() % *remain) < (count - fcnt))

[ing-eoking]

확률적으로 선택하게 되는 데, 최종적으로 count 개를 선택한다는 것이 보장되나요?

elem을 지나갈 때마다 remain 값은 무조건 1씩 감소하며, 결국 remain 값이 남은 뽑아야 할 개수(count-fcnt)에 가까워지게 됩니다.

remain 값이 count - fcnt 과 같아지면, rand() % remain 의 결과가 항상 remain보다 작은 값을 갖게 되어 elem이 count 개수만큼 될 때까지 100% 확률로 elem이 뽑히게 됩니다.

그래서 remain이 0에 가까워질수록 확률값이 100%까지 증가하기 때문에 count 만큼 뽑히는 것이 보장됩니다.

[jhpark816]

리뷰 완료

[jhpark816]

이 위치에 아래 조건만 있으면 됩니다.

if (fcnt >= count) break;

[jhpark816]

*remain == 0 조건은 없어도 됩니다.

[jhpark816]

여기도 fcnt에 대한 조건만 있으면 됩니다.

[jhpark816]

remain과 count 인자의 순서를 변경하고,
remain을 포인터 타입이 아닌 uint32_t 타입으로 하면 좋겠습니다.
대신, 하위 노드에 대한 traverse가 끝나면, remain은 하위 노드에 있는 element 개수만큼 감소하면 됩니다.

[jhpark816]

remain 타입을 변경하면, 아래와 같이 호출하면 됩니다.

        fcnt = do_set_elem_traverse_sampling(info, info->root, info->ccnt,
                                             count, elem_array);