Changed secondValue to use BeanProperty to test that works too
Added support for unpack(To) -> '* as per:

https://groups.google.com/forum/?hl=en&fromgroups=#!topic/cascading-user/49OkfPAotx8

Involved updating ReflectionSetter's arity() method and adding a
getResultFields() method to TupleUnpacker (and overriding the default
implementation in ReflectionTupleUnpacker). Without these, Fields.ALL would
cause problems for Cascading downstream.
Half way through updates
Added tests. Working nicely except that the order of fields is unpredictable
Tests are passing but shouldn't be
Simplified - ReflectionSetter now doesn't have to be aware of Fields.ALL differences
Switched to Oscar's idea of passing Fields around. Much more sensible. Also added in ReflectionUtils (mostly still stubbed).
Refactoring to a single helper method (expandIfAll) which does the only Fields.ALL check. I think this is the joint.
Use a lazy val so we generate allFields once only
Fixed comment
Move fieldMap below methodMap, where it was before I started editing
Removed unnecessary imports
Changed the FatContainer tests to sink to Tsv & manually checking each entry in the resulting TupleEntry
Merge pull request #295 from snowplow/develop

Added support for unpack(To) -> '* to support fat (>22 field) rows