[
https://issues.apache.org/jira/browse/DATAFU-83?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15609317#comment-15609317
]
Eyal Allweil commented on DATAFU-83:
------------------------------------
Hi [~sonalit],
I don't see your review board request. Can you check that it's associated with
the DataFu group, or attach your updated patch? This seems like a bug worth
fixing, even if Pig already has its own [IN
operator|https://pig.apache.org/docs/r0.14.0/basic.html#boolops].
> InUDF does not validate that types are compatible
> -------------------------------------------------
>
> Key: DATAFU-83
> URL: https://issues.apache.org/jira/browse/DATAFU-83
> Project: DataFu
> Issue Type: Improvement
> Reporter: Matthew Hayes
> Priority: Minor
> Attachments: DATAFU-83.patch
>
>
> See the example below. The input data is a long, but ints are provided to
> match against. Because it uses the Java equals to compare and these are
> different types, this will never match, which can lead to confusing results.
> I believe it should at least throw an error.
> {code}
> define I datafu.pig.util.InUDF();
>
> data = LOAD 'input' AS (B: bag {T: tuple(v:LONG)});
>
> data2 = FOREACH data {
> C = FILTER B By I(v, 1,2,3);
> GENERATE C;
> }
>
> describe data2;
>
> STORE data2 INTO 'output';
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
