incubator-wave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yuri Zelikov" <vega...@gmail.com>
Subject Re: Review Request: Refactors the SearchService related classes to allow easier integration of alternative search provider
Date Tue, 22 Nov 2011 19:13:17 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2882/
-----------------------------------------------------------

(Updated 2011-11-22 19:13:17.845982)


Review request for wave, Michael MacFadden, Tad Glines, and vjrj.


Summary
-------

Refactors the SearchService related classes to allow easier integration of alternative search
provider. Basically, the code responsible for digest generation was decoupled from SearchService
and moved into SearchProvider.


Diffs
-----

  src/com/google/wave/api/SearchResult.java fea106b 
  src/com/google/wave/api/impl/GsonFactory.java 615a182 
  src/org/waveprotocol/box/server/robots/operations/SearchService.java b235232 
  src/org/waveprotocol/box/server/waveserver/SearchProvider.java a2d55be 
  src/org/waveprotocol/box/server/waveserver/WaveDigester.java PRE-CREATION 
  src/org/waveprotocol/box/server/waveserver/WaveMap.java 1192103 
  test/org/waveprotocol/box/server/robots/operations/SearchServiceTest.java 5fb5e19 
  test/org/waveprotocol/box/server/robots/operations/TestingWaveletData.java PRE-CREATION

  test/org/waveprotocol/box/server/waveserver/WaveDigesterTest.java PRE-CREATION 
  test/org/waveprotocol/box/server/waveserver/WaveMapTest.java 82daafc 
  test/org/waveprotocol/box/server/waveserver/WaveServerTest.java 1147578 

Diff: https://reviews.apache.org/r/2882/diff


Testing
-------


Thanks,

Yuri


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message