naver / arcus-zookeeper

ZooKeeper fork for ARCUS
Apache License 2.0
7 stars 12 forks source link

Add ARCUS patch to prepare arcus-zookeeper 3.5.7 #12

Closed MinWooJin closed 4 years ago

MinWooJin commented 4 years ago

arcus-zookeeper 3.5.7 버전 준비를 위해 기존 ARCUS 패치 추가를 위한 PR 입니다. 다음 PR으로는 아래와 같은 순서로 진행 할 예정입니다.

  1. README PR
  2. Base code 변경으로 인한 error fix PR

@jhpark816 확인 요청 드립니다.

jhpark816 commented 4 years ago

일부 commit은 제외되었는 데, 간단히 설명 부탁해요.

MinWooJin commented 4 years ago

제외된 commit list와 이유는 아래와 같습니다.

jhpark816 commented 4 years ago

하나만 더 질문합니다. 아래 commit은 반드시 필요한가요 ? 새로 추가하는 형태가 되어서요.

MinWooJin commented 4 years ago

해당 commit은 ZOO_LOG_XX macro를 추가했던 commit입니다. ZOO_LOG_XX macro는 arcus-c-client에서 zk관련 로그를 출력할 때 사용하기때문에 반드시 필요합니다.

참고로, 현재 추가한 commit은 ZOO_LOG_XX macro가 mapping된 LOG_XX macro의 수행 함수 argument가 달라지면서 문제가 있는 상태이고, 추후 수정하는 commit을 PR할 예정입니다.