INTERNAL: Remove AuthThread and do auth on IO Thread

[uhm0311]

🔗 Related Issue

• 기존에 AuthThread가 IO Thread와 별도로 동작하면서 auth 연산을 요청했다.
• 멀티 쓰레딩 특성으로 인해 auth가 연관된 작업을 할 때 동시성 제어를 계속 생각해야 하는 단점이 있다.
• AuthThread를 제거하고 IO Thread가 스스로 auth 연산을 수행하도록 하면 이러한 단점을 해소할 수 있다.

⌨️ What I did

• AuthThread와 관련 코드를 제거합니다.
• MemcachedConnection 클래스에 prepareAuthentication() 메소드를 추가하여 auth 관련 연산을 수행할 수 있도록 합니다.
• prepareAuthentication() 메소드에 auth 로직이 모두 포함되어 있어서, 연결이 완료될 때마다 prepareAuthentication()를 1회만 호출하면 됩니다.

[jhpark816]

일부 리뷰

[jhpark816]

MemcacheClient는 더 이상 ConnectionObserver를 구현할 필요가 없어 보입니다.

[jhpark816]

@uhm0311
질문입니다.
writeQ는 empty 상태일 것 같은 데, 아닐 수도 있나요?

예전 구현에서도 inputQueue가 empty일 것 같은 데요.
inputQueue에 op가 있을 수 있음을 가정하고 있습니다.
이렇게 할 필요가 있는 지 궁금합니다.

[uhm0311]

Queue가 비어 있는 것은 확실합니다.
기존 코드를 거의 그대로 사용하려고 했는데, 비어 있으니 그냥 마지막에 연산을 추가하도록 할까요?

[jhpark816]

마지막 질문입니다.
authOperations에 남은 op가 complete되지 않았다면, cancel하는 로직인데요. 어떤 경우를 대비하기 위함인가요?

혹시, 기존 auth thread monitor에 위의 로직이 있어서 이를 동일하게 구현해 둔 것인가요?

[uhm0311]

혹시, 기존 auth thread monitor에 위의 로직이 있어서 이를 동일하게 구현해 둔 것인가요?

동일하게 구현하려고 했습니다.
제가 보기에는 이 조건문 안으로 들어올 수 있는 경우는 존재하지 않습니다.
이 코드를 제거하도록 할까요?

[jhpark816]

예. 제거하시죠.

[jhpark816]

@oliviarla @brido4125
수정 코드를 확인해 두기 바랍니다.