improve meta server to reply ERR_BUSY for config sync if is updating …

imzhenyu / rDSN.dist.service

Service frameworks targeting high availability, reliability, scalability, and consistency for micro services and storages

MIT License

1 stars 3 forks source link

improve meta server to reply ERR_BUSY for config sync if is updating … #2

Closed qinzuoyan closed 8 years ago

qinzuoyan commented 8 years ago

…config to zk, refer to #504

refer to https://github.com/imzhenyu/rDSN/issues/504

qinzuoyan commented 8 years ago

@imzhenyu , is it the step of committing changes to dist.service?

create pull request in rDSN.dist.service
merge the pull request in rDSN.dist.service
in rDSN, run 'git submodule update --remote', and create pull request
merge the pull request in rDSN

imzhenyu commented 8 years ago

@qinzuoyan the travis-ci test seems wired today and I just made some fix at https://github.com/imzhenyu/rDSN/pull/530 . May need to restart the test for this pr again after that pr is merged up to Microsoft/rDSN.

The last two steps for committing changes to dist.service are unnecessary unless you always work from rDSN repo to include this repo. For me, I have the two repos in different directories.

git clone imzhenyu/rDSN
git clone imzhenyu/rDSN.dist.service
cd rDSN
./run.sh build
./run.sh test
./run.sh install
cd ..
cd rDSN.dist.service
dsn.run.sh build
dsn.run.sh test

imzhenyu commented 8 years ago

Seems the failure is caused by ./run.sh stop_zk in .travis.yml. You may remove that and try this pr again.

qinzuoyan commented 8 years ago

@imzhenyu , where is the script dsn.run.sh?

imzhenyu commented 8 years ago

it is under $DSN_ROOT/bin - which is available after rDSN installation.

qinzuoyan commented 8 years ago

@imzhenyu , please review it or merge it.