----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48086/#review135684 -----------------------------------------------------------
Not sure how important it will be but maybe a test for an invalid function argument (such as a bad location or a null location?) geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/directory/DumpDirectoryFiles.java (line 74) <https://reviews.apache.org/r/48086/#comment200702> Maybe check to see if index is null first? - Jason Huynh On May 31, 2016, 7:16 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48086/ > ----------------------------------------------------------- > > (Updated May 31, 2016, 7:16 p.m.) > > > Review request for geode, Jason Huynh, nabarun nag, and xiaojian zhou. > > > Repository: geode > > > Description > ------- > > Adding a function that will dump all of the files for the lucene index > to disk, for examination with external tools like Luke. > > > Diffs > ----- > > > geode-junit/src/main/java/com/gemstone/gemfire/test/junit/rules/DiskDirRule.java > 184619fe2332038b80d4769e88968023f6c55d63 > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/LuceneServiceImpl.java > 1f011a6b8650f98fd42fef71bd1593080fe91379 > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/directory/DumpDirectoryFiles.java > PRE-CREATION > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/filesystem/File.java > 2937af54631ed593ca9a5213dbcc6c964895731d > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/filesystem/FileSystem.java > fdafcbecfc95ec4bb9e165d7073ebbcd783ca8ae > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepository.java > f1e63e096208724b935649f102f37a0be3245822 > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepositoryImpl.java > 3dbbc9415e176cdc8f0d9d02dc48aa93ddd20d3d > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/directory/DumpDirectoryFilesIntegrationTest.java > PRE-CREATION > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/directory/DumpDirectoryFilesJUnitTest.java > PRE-CREATION > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/filesystem/FileSystemJUnitTest.java > 1d936ca216bf06550e6ce552bdefa999a3c25a31 > > Diff: https://reviews.apache.org/r/48086/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >
