INTERNAL: Add AdminServer property in zoo_sample.cfg

naver / arcus-zookeeper

ZooKeeper fork for ARCUS

Apache License 2.0

7 stars 12 forks source link

Closed namsic closed 6 months ago

namsic commented 6 months ago

zoo_sample.cfg를 기반으로 zoo.cfg를 작성하는 경우 http를 통해 서버 명령을 입력받는 AdminServer가 비활성화됩니다.

jhpark816 commented 6 months ago

만약 default가 enabled 이었다면, 그대로 enable 시켜두는 것이 좋을 것 같습니다. 어떤가요?

namsic commented 6 months ago

만약 default가 enabled 이었다면, 그대로 enable 시켜두는 것이 좋을 것 같습니다. 어떤가요?

default(enabled) 설정 기준으로 주석만 추가하는 형태로 변경했습니다.
하나의 장비에 ZK를 여럿 구동하는 경우 여러 서버 중 처음 구동하는 서버가 8080 port를 점유하게 되는데, (나머지 서버들의 구동이 실패하는 것은 아님) 이 부분이 조금 부자연스러워 보이기는 합니다.

jhpark816 commented 6 months ago

commit message 수정 필요

Disable AdminServer ... => Add AdminServer property ...

namsic commented 6 months ago

commit message 및 PR title 변경했습니다.

INTERNAL: Add AdminServer property in zoo_sample.cfg