----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47501/#review133665 -----------------------------------------------------------
Ship it! Ship It! - Kirk Lund On May 17, 2016, 11:49 p.m., Jianxia Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47501/ > ----------------------------------------------------------- > > (Updated May 17, 2016, 11:49 p.m.) > > > Review request for geode, Bruce Schuchardt, Hitesh Khamesra, Kirk Lund, and > Udo Kohlmeyer. > > > Repository: geode > > > Description > ------- > > The locator status file writer and reader are out of sync. The reader may > read the empty file before writer writes to it. So write the status to a temp > file, then rename it. So that the reader read the status file that has > content in it. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/internal/process/ControllableProcess.java > f459aed > > Diff: https://reviews.apache.org/r/47501/diff/ > > > Testing > ------- > > > Thanks, > > Jianxia Chen > >
