----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21306/#review50229 -----------------------------------------------------------
For most of these, if the value of the map is a tuple with a schema, that schema ends up getting lost instead of passed alone. Should be able to support,e.g. (B: bag {T: tuple(key:CHARARRAY, value:tuple(v1:int, v2:CHARARRAY)}) datafu-pig/src/main/java/datafu/pig/maps/BagToMap.java <https://reviews.apache.org/r/21306/#comment87875> need to handle schema datafu-pig/src/main/java/datafu/pig/maps/MapToBag.java <https://reviews.apache.org/r/21306/#comment87874> if the mapValue has a schema, that schema should be preserved in the outputSchema - Will Vaughan On June 7, 2014, 3:16 p.m., wang jian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21306/ > ----------------------------------------------------------- > > (Updated June 7, 2014, 3:16 p.m.) > > > Review request for DataFu. > > > Bugs: https://issues.apache.org/jira/browse/DATAFU-34 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/DATAFU-34 > > > Repository: datafu > > > Description > ------- > > Add UDFs to handle map type data. > > Unit tests done. > > Remaining: verify javadocs > > > Diffs > ----- > > datafu-pig/src/main/java/datafu/pig/maps/AbstractHandleByKeys.java > PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/AbstractToMap.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/BagToMap.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/ContainsAllKeys.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/ContainsAnyKey.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/ContainsKeys.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/GetByKeys.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/MapPutHandler.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/MapToBag.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/PutMapToMap.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/RemoveByKeys.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/SchemaValidator.java PRE-CREATION > datafu-pig/src/main/java/datafu/pig/maps/TupleToMap.java PRE-CREATION > datafu-pig/src/test/java/datafu/test/pig/maps/CheckBag.java PRE-CREATION > datafu-pig/src/test/java/datafu/test/pig/maps/CheckMap.java PRE-CREATION > datafu-pig/src/test/java/datafu/test/pig/maps/ContainKeysTests.java > PRE-CREATION > datafu-pig/src/test/java/datafu/test/pig/maps/HandleByKeysMapTests.java > PRE-CREATION > datafu-pig/src/test/java/datafu/test/pig/maps/ToMapTests.java PRE-CREATION > > Diff: https://reviews.apache.org/r/21306/diff/ > > > Testing > ------- > > ues > > > Thanks, > > wang jian > >