[ https://issues.apache.org/jira/browse/PIG-598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12702563#action_12702563 ]
George Mavromatis commented on PIG-598: --------------------------------------- This has been flagged as minor, I think that's a rather serious and elementary one. It takes out the ability to comment/uncomment parameters and creates very bizarre error messages to people who are not aware of it. Can we increase priority? > Parameter substitution ($PARAMETER) should not be performed in comments > ----------------------------------------------------------------------- > > Key: PIG-598 > URL: https://issues.apache.org/jira/browse/PIG-598 > Project: Pig > Issue Type: Bug > Components: impl > Affects Versions: 0.2.0 > Reporter: David Ciemiewicz > Priority: Minor > > Compiling the following code example will generate an error that > $NOT_A_PARAMETER is an Undefined Parameter. > This is problematic as sometimes you want to comment out parts of your code, > including parameters so that you don't have to define them. > This I think it would be really good if parameter substitution was not > performed in comments. > {code} > -- $NOT_A_PARAMETER > {code} > {code} > -bash-3.00$ pig -exectype local -latest comment.pig > USING: /grid/0/gs/pig/current > java.lang.RuntimeException: Undefined parameter : NOT_A_PARAMETER > 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:394) > at org.apache.pig.Main.main(Main.java:296) > {code} -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.