Review Board 2.0.8

FLUME 136: Port Configuration Server to Avro

Review Request #319 - Created July 13, 2010 and submitted

Patrick Wendell
Henry, jon, phunt
These are some preliminary files for the flume conversion to Avro. This review consists of three types of files

1) Avro source files and build targets
2) FlumeConfigData, a class capable of translating itself to/from Avro and Flume data types.
3) Avro versions of existing client/server implementations.

Having the translation logic within the intermediary class (FlumeConfigData) may not be the best way to do it. However, there isn't really an obvious place to put this logic. We could put it \*just\* in the client or server implementations, but then it's awkward to import translating logic from a server to a client or vice-versa. I'm open to suggestions though\!

This upgrade is in two github commits on my github:
Avro port: eb751f9038d8098c9971d868da07949b33c49f35
See for an end-to-end test of client-server communication with Avro.
Review request changed
Updated (Aug. 18, 2010, 5:19 a.m.)

Change Summary:

I addressed everything in Jon's review comments. Also one small bugfix. Pull this from my github:\_admin\_rpc

(DONE) Licenses on certain files.
(DONE) DECOMMISSIONED state in avro rpc.

(DONE) I'm still not a fan of having init(xxx) instead of having the parameters plugged into the constructor.  

Not blockers:
(DONE where applicable) Prefer @override on interface implementations
(null is actually correct - see prior comments) return Void.TYPE instead of return null for things that return Void type.


Revision 5 (+3452 -1720)

Show changes

Posted (Aug. 18, 2010, 5:23 a.m.)
Responding to Jon's review comments.
Ship it!
Posted (Aug. 18, 2010, 9:33 a.m.)

One nit (no need for review) - is SingleMasterRPC still in the patch or is it removed (and just showing up in review board review?)  If not used, please remove\!
  1. Just an artifact of reviewboard. Should be gone if pulled in from github commit listed here. (reminder: 473b428dd1d8e48f2f68)
  2. can you change the status of this to submitted? Thanks\!