----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16639/#review31244 -----------------------------------------------------------
I was able to download the patch, but I have issues to apply it - there's an error (unrecognized binary patch at line 165447) and the jars are empty. Andrew, can you maybe submit just the changes without protobuff generated sources and without jars? But instead to send us a zip/tar with the new jars in email. Thanks for the effort! src/org/waveprotocol/box/server/rpc/ServerRpcProvider.java <https://reviews.apache.org/r/16639/#comment59637> Please add Jira issue for this and remove comment. src/org/waveprotocol/box/server/rpc/WebSocketClientRpcChannel.java <https://reviews.apache.org/r/16639/#comment59638> Remove trailing space. src/org/waveprotocol/box/server/waveserver/MemoryPerUserWaveViewHandlerImpl.java <https://reviews.apache.org/r/16639/#comment59639> Fix the comment. src/org/waveprotocol/box/server/waveserver/MemoryPerUserWaveViewHandlerImpl.java <https://reviews.apache.org/r/16639/#comment59640> Why the check was removed? if (explicitPerUserWaveViews.containsKey(user)) { src/org/waveprotocol/box/server/waveserver/Wave.java <https://reviews.apache.org/r/16639/#comment59641> Remove trailing space src/org/waveprotocol/wave/federation/xmpp/RemoteDisco.java <https://reviews.apache.org/r/16639/#comment59642> Remove empty line src/org/waveprotocol/wave/federation/xmpp/XmppDisco.java <https://reviews.apache.org/r/16639/#comment59643> Remove trailing space. - Yuri Zelikov On Jan. 6, 2014, 11:42 a.m., Andrew Kaplanov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16639/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2014, 11:42 a.m.) > > > Review request for wave, Ali Lown, Vicente J. Ruiz Jurado, and Yuri Zelikov. > > > Repository: wave > > > Description > ------- > > I had to disable SocketIO servlet, because library, that support it - > https://github.com/vjrj/Socket.IO-Java - still not been updated to support > Jetty 9. > > > Diffs > ----- > > build.xml 70db95c > src/com/google/wave/api/robot/HttpRobotConnection.java 75ac450 > src/org/waveprotocol/box/common/comms/waveclient-rpc.proto b5fae29 > src/org/waveprotocol/box/server/ServerMain.java b50454d > src/org/waveprotocol/box/server/ServerModule.java 71917c9 > src/org/waveprotocol/box/server/frontend/WaveViewSubscription.java e670dd9 > src/org/waveprotocol/box/server/frontend/WaveletInfo.java faf2fb3 > src/org/waveprotocol/box/server/persistence/file/FileDeltaStore.java > c2b9f11 > src/org/waveprotocol/box/server/persistence/protos/delta-store.proto > 4f08148 > src/org/waveprotocol/box/server/robots/agent/passwd/PasswordRobot.java > 460f724 > src/org/waveprotocol/box/server/robots/dataapi/DataApiOAuthServlet.java > feccb7d > src/org/waveprotocol/box/server/robots/dataapi/DataApiTokenContainer.java > 901b53c > src/org/waveprotocol/box/server/rpc/AbstractWaveSocketIOServlet.java > 0d93b6a > src/org/waveprotocol/box/server/rpc/GadgetProviderServlet.java 1a10d72 > src/org/waveprotocol/box/server/rpc/ProtoCallback.java 24bd21f > src/org/waveprotocol/box/server/rpc/ServerRpcProvider.java 7f65672 > src/org/waveprotocol/box/server/rpc/WebSocketChannelImpl.java PRE-CREATION > src/org/waveprotocol/box/server/rpc/WebSocketClientChannel.java 3265274 > src/org/waveprotocol/box/server/rpc/WebSocketClientRpcChannel.java b067c13 > src/org/waveprotocol/box/server/rpc/WebSocketServerChannel.java 89ab899 > src/org/waveprotocol/box/server/waveserver/DeltaStoreBasedWaveletState.java > 04821f5 > > src/org/waveprotocol/box/server/waveserver/LucenePerUserWaveViewHandlerImpl.java > d61e2e0 > > src/org/waveprotocol/box/server/waveserver/MemoryPerUserWaveViewHandlerImpl.java > 7fa8408 > src/org/waveprotocol/box/server/waveserver/Wave.java b28069e > src/org/waveprotocol/box/server/waveserver/WaveMap.java a5616d2 > src/org/waveprotocol/box/server/waveserver/WaveServerModule.java d706153 > > src/org/waveprotocol/box/server/waveserver/WaveletNotificationDispatcher.java > 92b7b93 > src/org/waveprotocol/examples/robots/echoey/Echoey.java c73fe5e > src/org/waveprotocol/wave/federation/xmpp/RemoteDisco.java 999c8a7 > src/org/waveprotocol/wave/federation/xmpp/XmppDisco.java 5745f91 > src/org/waveprotocol/wave/federation/xmpp/XmppFederationHost.java a666d53 > > test/org/waveprotocol/box/server/robots/dataapi/DataApiOAuthServletTest.java > d625c60 > test/org/waveprotocol/box/server/rpc/RpcTest.java 5a0b07c > test/org/waveprotocol/box/server/rpc/WebSocketChannelTest.java e5ab7b9 > test/org/waveprotocol/wave/federation/xmpp/MockDisco.java 28f25c7 > test/org/waveprotocol/wave/federation/xmpp/XmppDiscoTest.java aad419d > > test/org/waveprotocol/wave/federation/xmpp/XmppFederationHostForDomainTest.java > c47d01f > third_party/runtime/gson/LICENSE PRE-CREATION > third_party/runtime/gson/README.google 6136ac4 > third_party/runtime/gson/gson-1.4.jar > b9c33d0390790ad6ef9549e16e9349f0f7fffca7 > third_party/runtime/gson/gson-2.2.4.jar PRE-CREATION > third_party/runtime/guava/README.google 1bcc427 > third_party/runtime/guava/guava-15.0.jar PRE-CREATION > third_party/runtime/guava/guava-gwt-15.0.jar PRE-CREATION > third_party/runtime/guava/guava-r09-gwt.jar > 5f130e5fb6cbdea46fc444b1e100c7e2a790c3cb > third_party/runtime/guava/guava-r09.jar > f8da8b1c1e81d583dc48d875fd23229f7271e132 > third_party/runtime/jetty/README.google 50e1266 > third_party/runtime/jetty/jetty-annotations-9.1.0.v20131115.jar > PRE-CREATION > third_party/runtime/jetty/jetty-client-8.1.1.v20120215.jar > c0dbff16b1d2200e8eb251dc3ea7cc3686026afd > third_party/runtime/jetty/jetty-client-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-continuation-8.1.1.v20120215.jar > 321153c4eaa3b6799abcd7a282ad335d2e5770a4 > third_party/runtime/jetty/jetty-continuation-9.1.0.v20131115.jar > PRE-CREATION > third_party/runtime/jetty/jetty-http-8.1.1.v20120215.jar > 3a2b3f0cb4a84e9ed6289071c15f5be696a5f733 > third_party/runtime/jetty/jetty-http-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-io-8.1.1.v20120215.jar > d9f67c24e45d29325b836793e7e73ad6e0751dfa > third_party/runtime/jetty/jetty-io-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-proxy-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-security-8.1.1.v20120215.jar > f75dbd1afbfc267031ac1d52fbb30274aa80293f > third_party/runtime/jetty/jetty-security-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-server-8.1.1.v20120215.jar > 0a313af27ee278a0e054d47225ceff2ad839a907 > third_party/runtime/jetty/jetty-server-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-servlet-8.1.1.v20120215.jar > aec7b8f95ab733336a0072999e800d8aa00b6868 > third_party/runtime/jetty/jetty-servlet-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-servlets-8.1.1.v20120215.jar > 9bb787ea5b673d78d88795a75d09e6c3710b820b > third_party/runtime/jetty/jetty-servlets-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-util-8.1.1.v20120215.jar > 096f6cf92bd5b3827d904899c5bae0994919aef5 > third_party/runtime/jetty/jetty-util-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-webapp-8.1.1.v20120215.jar > 0c8aa4af4cb1cf8eeb224436703c170ac000f2cc > third_party/runtime/jetty/jetty-webapp-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/jetty-websocket-8.1.1.v20120215.jar > b398c79ae8428d557962d6c8d844833200abcaab > third_party/runtime/jetty/jetty-xml-8.1.1.v20120215.jar > 9df8fbdf339bb009af06ee457b763e2019245d49 > third_party/runtime/jetty/jetty-xml-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/servlet-api-3.0.jar > b135409682ee2173ee74cc9e9b00469d7fa0a27e > third_party/runtime/jetty/servlet-api-3.1.jar PRE-CREATION > third_party/runtime/jetty/websocket-api-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/websocket-client-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/websocket-common-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/websocket-server-9.1.0.v20131115.jar PRE-CREATION > third_party/runtime/jetty/websocket-servlet-9.1.0.v20131115.jar > PRE-CREATION > third_party/runtime/protobuf/README.google aa7c8c2 > third_party/runtime/protobuf/protobuf-2.3.0-src.jar > ab81e5ec2ac34541b32537e1496d02ba1f129c9e > third_party/runtime/protobuf/protobuf-java-2.3.0.jar > f78b32b05c97c4b27f574849143935dfa8e1e5a5 > third_party/runtime/protobuf/protobuf-java-2.5.0.jar PRE-CREATION > > Diff: https://reviews.apache.org/r/16639/diff/ > > > Testing > ------- > > > Thanks, > > Andrew Kaplanov > >
