pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy Ryaboy <dvrya...@gmail.com>
Subject Re: Caster interface and byte conversion
Date Sun, 22 Aug 2010 08:40:26 GMT
The current HBase patch on PIG-1205 (patch 7) includes this refactoring.
Please take a look if you have concerns.

Or just if you feel like reviewing the code... :)


On Sat, Aug 21, 2010 at 5:22 PM, Dmitriy Ryaboy <dvryaboy@gmail.com> wrote:

> I just noticed that even though Utf8StorageConverter implements the various
> byte[] toBytes(Obj o) methods, they are not part of the LoadCaster interface
> -- and therefore can't be relied on when using modular Casters, like I am
> trying to do for the HBaseLoader.
> Since we don't want to introduce backwards-incompatible changes, I propose
> adding a ByteCaster interface that defines these methods, and extending
> Utf8StorageConverter to implement them (without actually changing the
> implementation at all).
> That way StoreFuncs that need to convert to bytes can use pluggable
> converters. Objections?
> -D

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message