search

SollmoStudio / beyond

Beyond: The Scalable Game Server Framework
http://www.beyondframework.com
Apache License 2.0
25 stars 9 forks source link

Intergrate ZooKeeper configuration file with Beyond configuration file. #196

Closed sgkim126 closed 9 years ago

sgkim126 commented 9 years ago

Current ZooKeeper configuration is difficult because there are many config files to use ZooKeeper. This patch intergrates zookeeper configuration files with the beyond framework file.

hatashiro commented 9 years ago

The concept looks very good. Could you update the PR as it may conflict with my patches? After that I'll look into it more specifically.

sgkim126 commented 9 years ago

@noraesae, I reupdated PR.

sgkim126 commented 9 years ago

@noraesae PR updated. I removed BeyondConfiguration.zooKeeperConfigPath, and modified to use config.configuration function to remove duplicated code.

hatashiro commented 9 years ago

LGTM