+1 Brock. I don't see any problem in removing the get(String, Class) method.
Thanks, Arvind On Sun, Feb 19, 2012 at 4:41 PM, juhani connolly < [email protected]> wrote: > I feel the same way about Context... There is a lot of repeated code that > could be altogether avoided to make simpler, less bulky code. > > > On 02/20/2012 08:42 AM, Brock Noland wrote: > >> Hi, >> >> In FLUME-978, I make the case that the Context object should provide a >> few convenience methods and store types internally the same way the >> data will be store externally (properties file) as Strings. >> >> I am looking for comment on this before writing a patch as I think the >> method: >> >> get(String key, Class<? extends T> clazz) >> >> will need to be removed and this would touch a large number of lines of >> code. >> >> Cheers, >> Brock >> >> >
