> On Feb. 27, 2017, 10:29 p.m., Barry Oglesby wrote: > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/results/LuceneGetPageFunction.java, > > line 75 > > <https://reviews.apache.org/r/57057/diff/1/?file=1647835#file1647835line75> > > > > I'm not sure if you want to deal with this, but the getValue call will > > deserialize the value on the server, then it will be reserializeed to send > > to the client.
Good point. I think I should fix that, but I'll do it as a separate commit so that I can get this fix in before doing that optimization. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57057/#review166946 ----------------------------------------------------------- On Feb. 25, 2017, 1:05 a.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57057/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2017, 1:05 a.m.) > > > Review request for geode, Barry Oglesby, Jason Huynh, nabarun nag, and > xiaojian zhou. > > > Repository: geode > > > Description > ------- > > Instead of using getAll, fetch the values of a lucene query using a > function that calls getEntry. We can then avoid invoking the cache > loader. > > > Diffs > ----- > > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneQueryImpl.java > b50996be8eeb2677537ba7756e33fc199601b3fc > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneServiceImpl.java > cf7b2c9a3e0a78417fa192ceee57441e005536ea > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/PageableLuceneQueryResultsImpl.java > 5c5d8250cf9c31f0d29964fe5f12f3bae779500d > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/results/LuceneGetPageFunction.java > PRE-CREATION > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/results/MapResultCollector.java > PRE-CREATION > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/LuceneQueriesIntegrationTest.java > 01bdca40a59422dc0700ff22e867db7717a374ea > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/PaginationDUnitTest.java > cfde4f23305e3e4c6419268d32b04e2c9db4f9db > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/LuceneQueryImplJUnitTest.java > 9f826d586479e9e56b9e7130a9f23e7f706693a7 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/PageableLuceneQueryResultsImplJUnitTest.java > 35a4c9169714ff766a655255d471cb639b7a0404 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/results/LuceneGetPageFunctionTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/57057/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >
