[
https://issues.apache.org/jira/browse/DATAFU-34?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13975294#comment-13975294
]
Matthew Hayes commented on DATAFU-34:
-------------------------------------
MapToBag:
* Line 77 in sample code has a less than sign that's not html-encoded.
* Have you confirmed the type of the value that is produced for your tests? I
think it would be good to have tests that verify that if you declared a
map[int] then you get a bag with key string and value int. Also when the type
of the map's value isn't declared, it should be bytearray, and we should
confirm that the bag's value field is a bytearray.
MapToTuple:
* The output schema's tuple doesn't have a type declared.
* Same comments as with MapToBag regarding testing the types.
PutToMap:
* Typo on 73: acutal
* What if the type of the map is map[int] for instance and someone puts a
string value. Does your code catch this? You should add a test case for this.
By the way thanks for writing these :) These look pretty useful.
> Add some UDFS to handle map type
> --------------------------------
>
> Key: DATAFU-34
> URL: https://issues.apache.org/jira/browse/DATAFU-34
> Project: DataFu
> Issue Type: New Feature
> Reporter: jian wang
> Assignee: jian wang
> Attachments: 0001-add-some-UDFs-to-manipulate-map.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.2#6252)
