----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19355/#review40508 -----------------------------------------------------------
Ship it! LGTM PS: Probably we have to use something like this in the future: http://www.eclipse.org/jetty/documentation/current/example-slf4j-multiple-loggers.html - Vicente J. Ruiz Jurado On April 15, 2014, 6:47 p.m., Pablo Ojanguren wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19355/ > ----------------------------------------------------------- > > (Updated April 15, 2014, 6:47 p.m.) > > > Review request for wave, Andrew Kaplanov, Ali Lown, Vicente J. Ruiz Jurado, > and Yuri Zelikov. > > > Bugs: WAVE-405 > https://issues.apache.org/jira/browse/WAVE-405 > > > Repository: wave > > > Description > ------- > > This patch provides a full replacement of the Socket.IO as an alternative to > websockets when they are not available. Atmosphere framework is configured to > use long-polling protocol but additional are available. > > Server and GWT client has been affected. > > NOT all references to Socket.IO has been removed from the source code yet. > And .jar dependencies have been kept also. Is it safe to remove them? > > New dependencies has been included in build.xml, task get-third-party, so > none special process is needed to build this version. > The atmosphe.js client file is also handled as a third-party dep and it's > served from the class path > > > Diffs > ----- > > build.xml 0681b164cf580dd161d110dbf1032337243db79d > src/org/waveprotocol/box/server/rpc/ServerRpcProvider.java > 9b0f2a927bf75b92fb708c3abfdb4666d9cd6e63 > src/org/waveprotocol/box/server/rpc/atmosphere/AtmosphereChannel.java > PRE-CREATION > > src/org/waveprotocol/box/server/rpc/atmosphere/AtmosphereClientInterceptor.java > PRE-CREATION > src/org/waveprotocol/box/server/rpc/atmosphere/GuiceAtmosphereFactory.java > PRE-CREATION > src/org/waveprotocol/box/webclient/WebClient.gwt.xml > 387d0c78206bfca61412f31bdac26ec9a67224c9 > src/org/waveprotocol/box/webclient/client/WaveSocketFactory.java > 4a1788fc6f89fa07cbc41ab99335b25861388d8a > src/org/waveprotocol/box/webclient/client/WaveWebSocketClient.java > 65746d2348bf55a6c6f22b8b4404dfca6c8de302 > > src/org/waveprotocol/box/webclient/client/atmosphere/AtmosphereConnection.java > PRE-CREATION > > src/org/waveprotocol/box/webclient/client/atmosphere/AtmosphereConnectionImpl.java > PRE-CREATION > > src/org/waveprotocol/box/webclient/client/atmosphere/AtmosphereConnectionListener.java > PRE-CREATION > > src/org/waveprotocol/box/webclient/client/atmosphere/AtmosphereConnectionState.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/19355/diff/ > > > Testing > ------- > > Only basic test has been performed so far: wiab server with two users > connected, all in the same dev computer. Using Firefoz 23 with websockets > disabled. > > More test and feedback is needed covering different browsers, work load and > network environments. All configuration values of the atmosphere framework > are set to default ones (thread pool size, buffers size...) so I think they > will need fine tuning. > > > Thanks, > > Pablo Ojanguren > >
