----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3508/#review4645 -----------------------------------------------------------
Any update? I want this patch to be committed so I can proceed. If there will be no more comments during next 24 hours I ll consider it as "LGTM" :) - Yuri On 2012-01-19 20:49:33, Yuri Zelikov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3508/ > ----------------------------------------------------------- > > (Updated 2012-01-19 20:49:33) > > > Review request for wave, Michael MacFadden and Lennard de Rijk. > > > Summary > ------- > > The goal of this patch is to decouple the search implementation > (SearchProvider) from the waves accessing logic (WaveMap). It should allow > for alternative implementations of search providers - not dependent on > concrete implementation of waves loading and access. The biggest problem with > current implementation is that it requires to load all waves into memory on > the server start up. An alternative (not memory based) search implementation > would allow to load the waves lazily and evict later. > > > This addresses bug WAVE-325. > https://issues.apache.org/jira/browse/WAVE-325 > > > Diffs > ----- > > src/org/waveprotocol/box/server/waveserver/MemorySearchProvider.java > PRE-CREATION > src/org/waveprotocol/box/server/waveserver/PerUserWaveViewSubscriber.java > PRE-CREATION > src/org/waveprotocol/box/server/waveserver/QueryHelper.java PRE-CREATION > src/org/waveprotocol/box/server/waveserver/TokenQueryType.java PRE-CREATION > src/org/waveprotocol/box/server/waveserver/Wave.java PRE-CREATION > src/org/waveprotocol/box/server/waveserver/WaveMap.java 42eb62b > src/org/waveprotocol/box/server/waveserver/WaveServerModule.java 4892dc3 > src/org/waveprotocol/box/server/waveserver/WaveletContainerImpl.java > ca05900 > test/org/waveprotocol/box/server/waveserver/MemorySearchProviderTest.java > PRE-CREATION > > test/org/waveprotocol/box/server/waveserver/PerUserWaveViewSubscriberTest.java > PRE-CREATION > test/org/waveprotocol/box/server/waveserver/WaveDigesterTest.java 9a49ab2 > test/org/waveprotocol/box/server/waveserver/WaveMapTest.java 17ba0df > > Diff: https://reviews.apache.org/r/3508/diff > > > Testing > ------- > > > Thanks, > > Yuri > >
