Review Board 1.7.22


FLUME-978: Context interface is too basic requiring boilerplate user code

Review Request #3968 - Created Feb. 20, 2012 and submitted

Brock Noland
flime-728
FLUME-978
Reviewers
Flume
flume-git
1) Changing backing structure of Context from object to String
2) Add getBoolean, getInteger, getLong to Context
3) Remove get(key, clazz)
4) Add javadoc to public methods of Context
5) Change backing map of Context to synchronized map since it seems this class could be accessed by multiple threads.
Unit tests added for all new methods and some exiting methods.

All unit tests pass.
Review request changed
Updated (Feb. 29, 2012, 3:51 a.m.)
Updated based on review.
Ship it!
Posted (Feb. 29, 2012, 4:52 a.m.)
+1