FIX: Fix do_command_dupcheck method that didn't show 0 at same key

[cheesecrust]

🔗 Related Issue

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

⌨️ What I did

• 이미 접근한 키에 대해서 전체에 접근하는 연산 (ex: sop get arcus 0)을 나중에 실행시키면 lqdetect show 에서 빠지는 버그를 수정하였습니다.

[jhpark816]

lqdetect 명령어들이 공통적으로 사용하는 로직을 밖으로 빼내어 간소화 시켰습니다.

버그 수정만 포함시키고, 위의 수정은 제외하는 것이 좋을 것 같은데요. 의견이 어떤가요?

[cheesecrust]

버그 수정만 포함시키고, 위의 수정은 제외하는 것이 좋을 것 같은데요. 의견이 어떤가요?

버그 수정만을 포함하도록 수정 했습니다.

[namsic]

전체 조회 (sop get skey 0) 명령을 먼저 보내는가 나중에 보내는가 하는 문제는 차치하고,
현재 develop branch와 PR 모두 아래와 같이 동작합니다.

lqdetect start 1
        long query detection started.

sop get skey1 0
sop get skey2 0
sop get skey1 1
sop get skey2 1

lqdetect show
sop get : 4
12:16:52.262987 127.0.0.1 <1> sop get skey1 0
12:17:07.816425 127.0.0.1 <1> sop get skey2 0
12:17:11.983276 127.0.0.1 <1> sop get skey1 1

위 결과를 보면, sop get skey2 1 연산은 출력되지 않았는데, dupcheck 연산이 아래와 같은 로직을 갖기 때문입니다.

• count가 0인 경우
  • (count는 항상 동일하므로) 대상 key가 달라지면 false
• count가 1 이상인 경우
  • count 값이 같으면 key가 달라도 true

즉 sop get skey1 0과 sop get skey2 0은 모두 기록하고,
sop get skey1 5와 sop get skey2 5는 중복 처리하여 하나만 기록하는 것입니다.

@jhpark816
제 생각에는 count가 0인지 아닌지와 무관하게 아래 둘 중 하나로 통일해야 할 것 같은데, 혹시 현재 동작이 의도된 설계인가요?

1. count가 같더라도 key가 다르면 모두 기록
2. count가 같으면 key가 다르더라도 하나만 기록

[namsic]

테스트 관련 리뷰만 우선 작성합니다.

[jhpark816]

제 생각에는 count가 0인지 아닌지와 무관하게 아래 둘 중 하나로 통일해야 할 것 같은데, 혹시 현재 동작이 의도된 설계인가요?

의도한 설계입니다.

• count = 0이면, 해당 set 크기에 따라 long query 여부가 결정됩니다. 따라서, 해당 key를 기록합니다.
• count > 0이면, 해당 요청 자체에 따라 long query 여부가 결정됩니다. 따라서, key 보다는 count > N 이었다는 질의를 기록합니다.

[jhpark816]

리뷰 완료

[cheesecrust]

long key에 대해서 저장할때는 변형되어 저장되므로 dupcheck 할때도 long key에 대해서는 변형된 key 를 기준으로 변경해야합니다.
따라서 해당 로직을 추가하였습니다.

[namsic]

count가 0일 때만 의미가 있는 동작이죠?

[jhpark816]

count == 0인 경우에만 의미가 있는 동작이 맞습니다.

대략 아래와 같이 구현하는 것이 좋겠습니다.

• do_command_dupcheck() => is_command_duplicated()
• 호출하는 쪽에서 do_command_dupcheck() 호출 제거

do_lqdetect_write()
{
    if (keylen > 250) {
        keylen = snprintf(keybuf, sizeof(keybuf), "%.*s...%.*s",
                          124, key, 123, (key+(keylen - 123)));
        keyptr = keybuf;
    }
    
    if (is_command_duplicated(keyptr, keylen, cmd, arg) != true) {
        gettimeofday(&val, NULL);
        ptm = localtime(&val.tv_sec);

        . . .

        lqdetect.arg[cmd][nsaved] = *arg;
        buffer->nsaved += 1;
    }
}

[namsic]

count가 0일 때만 의미가 있는 동작이죠?

[jhpark816]

리뷰 완료

[jhpark816]

count == 0인 경우에만 의미가 있는 동작이 맞습니다.

대략 아래와 같이 구현하는 것이 좋겠습니다.

• do_command_dupcheck() => is_command_duplicated()
• 호출하는 쪽에서 do_command_dupcheck() 호출 제거

do_lqdetect_write()
{
    if (keylen > 250) {
        keylen = snprintf(keybuf, sizeof(keybuf), "%.*s...%.*s",
                          124, key, 123, (key+(keylen - 123)));
        keyptr = keybuf;
    }
    
    if (is_command_duplicated(keyptr, keylen, cmd, arg) != true) {
        gettimeofday(&val, NULL);
        ptm = localtime(&val.tv_sec);

        . . .

        lqdetect.arg[cmd][nsaved] = *arg;
        buffer->nsaved += 1;
    }
}