[jira] [Commented] (PIG-4734) TOMAP schema inferring breaks some scripts in type checking for bincond

["Rohini Palaniswamy (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/PIG-4734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15306095#comment-15306095 ] 

Rohini Palaniswamy commented on PIG-4734:
-----------------------------------------

Should we relax the type checking for all types? Can we just do it for map? I am not sure what errors relaxing for everything might introduce.

> TOMAP schema inferring breaks some scripts in type checking for bincond
> -----------------------------------------------------------------------
>
>                 Key: PIG-4734
>                 URL: https://issues.apache.org/jira/browse/PIG-4734
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Rohini Palaniswamy
>            Assignee: Daniel Dai
>             Fix For: 0.16.0, 0.15.1
>
>         Attachments: PIG-4734-1.patch, PIG-4734-2.patch
>
>
> PIG-4674 added schema inferring for TOMAP.
> {code}
> FOREACH A GENERATE (val == 'x' ? TOMAP('key', floatfield1) : (val == 'y' ? GenerateFloatMap('key', floatfield2) : NULL)) as floatmap:map[float],
> {code}
> The following line fails with
> {code}
> Two inputs of BinCond must have compatible schemas. left hand side: #675:map(#676:float) right hand side: #801:map
> 	at org.apache.pig.newplan.logical.visitor.TypeCheckingExpVisitor.visit(TypeCheckingExpVisitor.java:616)
> 	... 45 more
> {code}
> GenerateFloatMap is a UDF that returns new HashMap<String, Float>(), but does not have outputSchema() defined. It worked earlier because TOMAP also did not have outputSchema() defined.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)