kpm1985 commented on issue #1004: FLUO-1000 OracleServer race conditions URL: https://github.com/apache/fluo/pull/1004#issuecomment-359927572 Hi Keith, About the errors that show during my verify. My original solution I sit and check curator state inside of the function takeLadership and wait until its ready. I tried moving this check to the start method and I ran Maven verify on this a few times and to my surprise I kept getting some exceptions on the state of that curator framework again. So what I did is I left it in that method that is called by leader selector and when the check is in that method with a couple other checks like this synchronized on the leader variable then the errors are not present. This made me wonder after tracing the flow of what I thought was going on in Oracle server, well what the heck is happening somebody else is screwing with something in the background thread somewhere so, I still didn't really understand what was going on but I left the check inside of the i, I still didn't really understand what was going on but I left the check inside of the is take leadership method. My only thought is possibly multiple instances but I thought those were private variables so not exactly sure what's going on with that. Anyways I really appreciate you responding and looking into this with me this is the help I totally needed. I'm on break with the phone so I apologize for not highlighting code and doing other things that you can do on a computer. On Jan 23, 2018 8:42 AM, "Keith Turner" <[email protected]> wrote: > Maybe we don't need to verify that the illegal state came from curator. We > could make the check stronger, something like the following. > > } catch (IllegalStateException e) { > if(curatorFramework.getState() == STOPPED && !started) { > // the oracle server had been stopped, do not care much about errors... lets suppress the exception > log.debug(...); log a debug message that this happened > } else { > throw e; > } > } > > ? > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/fluo/pull/1004#issuecomment-359852270>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/Acg-LI5CI5WUf9Avp_CQOuOa08pi5s8Cks5tNgv9gaJpZM4Rh_58> > . >
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
