----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57864/#review169911 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/internal/cache/OverflowOplog.java Line 756 (original), 766 (patched) <https://reviews.apache.org/r/57864/#comment242619> I'm ok with you changing this method to set bbArray[0] and then null it out BUT I think if you do this then you also need to change "consumeWriteBuf" and "createCrf" to not change bbArray. - Darrel Schneider On March 22, 2017, 6:45 p.m., anilkumar gingade wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57864/ > ----------------------------------------------------------- > > (Updated March 22, 2017, 6:45 p.m.) > > > Review request for geode, Darrel Schneider and Eric Shu. > > > Repository: geode > > > Description > ------- > > GEODE-2472: > The Oplog and OverflowOplog flush(OplogFile olf, ByteBuffer b1, ByteBuffer > b2) method doesn't check the results of the channel.write() call. > > It could so happen partial byte array is written to the channel. The check is > added to make sure the bytes are completely written to the file channel. > > > Diffs > ----- > > geode-core/src/main/java/org/apache/geode/internal/cache/Oplog.java 27f41a2 > geode-core/src/main/java/org/apache/geode/internal/cache/OverflowOplog.java > c7f8e0d > > geode-core/src/test/java/org/apache/geode/internal/cache/OplogFlushTest.java > d24e66d > > > Diff: https://reviews.apache.org/r/57864/diff/1/ > > > Testing > ------- > > Run new unit tests added. > precheckin in progress. > > > Thanks, > > anilkumar gingade > >
