[jira] Issue Comment Edited: (PIG-267) Don't substitute parameters inside comments

["Pi Song (JIRA)" <ji...@apache.org>]

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

pi_song edited comment on PIG-267 at 6/16/08 3:31 AM:
------------------------------------------------------

Referring to discussion in PIG-58, seems like this is already the expected behavior?

      was (Author: pi_song):
    Referring to discussion to PIG-58, seems like this is already the expected behavior?
  
> Don't substitute parameters inside comments
> -------------------------------------------
>
>                 Key: PIG-267
>                 URL: https://issues.apache.org/jira/browse/PIG-267
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Amir Youssefi
>            Priority: Trivial
>         Attachments: Pig267_ParamFix.patch
>
>
> A script with $x in comments fails because Pig thinks it's an undefined parameter. One approach to address it is to skip substitution for comments. 
> java.lang.RuntimeException: Undefined parameter : x
>         at org.apache.pig.tools.parameters.PreprocessorContext.substitute(PreprocessorContext.java:221)
>         at org.apache.pig.tools.parameters.ParameterSubstitutionPreprocessor.parsePigFile(ParameterSubstitutionPreprocessor.java:106)
>         at org.apache.pig.tools.parameters.ParameterSubstitutionPreprocessor.genSubstitutedFile(ParameterSubstitutionPreprocessor.java:86)
>         at org.apache.pig.Main.runParamPreprocessor(Main.java:382)
>         at org.apache.pig.Main.main(Main.java:284)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.