CLEANUP: Refactor includes

[cheesecrust]

🔗 Related Issue

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

⌨️ What I did

• 헤더파일 단독으로 컴파일이 가능하도록 변경
• lqdetect.h의 include 위치를 memcached.h 에서 memcached.c로 변경하였습니다.

[jhpark816]

일부 리뷰

[jhpark816]

bkey_range, eflag_filter, ENGINE_BTREE_ORDER 등은 그대로 유지합시다.
memcached header files을 include하는 코드를 모두 제거하는 것은 이 경우에는 맞지 않는 것 같습니다.

[cheesecrust]

반영하였습니다.

[jhpark816]

@namsic 리뷰 바랍니다.

[namsic]

아래 항목에 대해 의견 주시면 좋겠습니다.

• mc_logger 사용하는 곳이 없는데, 유지하는 것이 좋은가요?

---

• 인자로 logger 전달받아야 한다면, 헤더에서 logger.h를 include하는 것은 아래 관점에서 나쁘지 않아 보입니다.

1. 만약 인자로 고정되지 않은 타입을 받고자 의도한 것이라면 void* 타입을 사용하는 것이 유용합니다.

2. 하지만 lqdetect_init()은 그런 성격의 함수가 아니고, 항상 EXTENTION_LOGGER_DESCRIPTOR를 인자로 받아야 합니다.
   따라서 사용자는 lqdetect.h를 include할 때 항상 extention_logger.h도 함께 include해야만 할 것입니다.

3. 이런 상황에서는 lqdetect.h에서 logger.h를 include하는 것이 더 유용하다고 생각합니다.

   • lqdetect.h에서 logger.h에 의존하고 있는 관계를 명확히 드러냄
   • lqdetect_init()의 인자로 EXTENSION_LOGGER_DESCRIPTOR를 넘긴다는 것을 호출하는 측에서 알 수 있음

[jhpark816]

나중에 사용할 목적으로 mc_logger 유지하도록 합시다.

[jhpark816]

위에 설명된 대로, 기존 인터페이스를 그대로 유지하는 것이 나을 것 같습니다.

int lqdetect_init(EXTENSION_LOGGER_DESCRIPTOR *logger)

참고 사항으로,
EXTENSION_LOGGER_DESCRIPTOR는 include/memcached/extension.h 파일에 있습니다.

[cheesecrust]

mc_logger 를 유지하고 lqdetect.h 에서 memcached/extension.h 를 include 하여 lqdetect_init()의 선언을 기존과 동일하게 유지하도록 수정하겠습니다.