You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pig.apache.org by "Cheolsoo Park (JIRA)" <ji...@apache.org> on 2014/02/08 05:02:19 UTC

[jira] [Updated] (PIG-3670) Fix assert in Pig script

     [ https://issues.apache.org/jira/browse/PIG-3670?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Cheolsoo Park updated PIG-3670:
-------------------------------

    Attachment: PIG-3670-2.patch

+1. Thank you for fixing this.

Your patch didn't apply cleanly to trunk, so I rebased it. In addition, I replaced the temporary file in the test with ByteArrayInputStream. Since PigServer.registerScript() accepts InputStream, it seems better to avoid to create a temporary file. Let me know if you think otherwise.

> Fix assert in Pig script
> ------------------------
>
>                 Key: PIG-3670
>                 URL: https://issues.apache.org/jira/browse/PIG-3670
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: 0.13.0
>
>         Attachments: PIG-3670-1.patch, PIG-3670-2.patch
>
>
> PIG-3367 introduce "assert" keyword. However, it can only be used in Java embedding, when use registerQuery. In Pig script or Grunt shell, GruntParser will complain.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)