Re: [devel] [PATCH 3 of 3] imm: immnd changes for integrating IMM with CLMS [#1640] v2

2017-02-24 Thread Zoran Milinkovic
Hi Neelakanta, Can you fix indentation ? In some places the code looks very messy. I have problem with understanding why protocol52 has been introduced since CLM support is only local per node. If IMMND support CLM integration, then all IMM clients on that node can have this feature. The only w

Re: [devel] [PATCH 3 of 3] imm: immnd changes for integrating IMM with CLMS [#1640] v2

2017-02-24 Thread Hung Nguyen
Hi Neel, 1. Why do we have to keep a list of current member nodes of the cluster? I think all IMMND wants to know is if it's currently a member node. It shouldn't care about membership of other nodes. IMMND_CB.immnd_clm_list is not necessary. The immnd_clm_node_*() functions in immnd_db.c are als

[devel] [PATCH 3 of 3] imm: immnd changes for integrating IMM with CLMS [#1640] v2

2017-02-20 Thread reddy . neelakanta
src/imm/Makefile.am |5 +- src/imm/immnd/ImmModel.cc | 44 +++- src/imm/immnd/ImmModel.h |2 + src/imm/immnd/immnd_cb.h | 18 +++ src/imm/immnd/immnd_clm.c | 200 ++ src/imm/immnd/immnd_db.c | 125 ++