-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3508/
-----------------------------------------------------------
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
