[
https://issues.apache.org/jira/browse/PIG-85?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12597521#action_12597521
]
Olga Natkovich commented on PIG-85:
-----------------------------------
Hi Pi,
Thanks for submitting the patch. Couple of comments:
(1) To keep QueryParser.jjt code consistent I think it would make sense to use
unquote rather than StringUtils.unescapeInputString since that's what the rest
of the code does
(2) I think we should keep track of where we are at in the initial textLine
rather than reallocating it every time we find the delimiter to keep the code
more efficient.
> Unable to specify CTRL-A as a delimiter for the PigStorage function
> -------------------------------------------------------------------
>
> Key: PIG-85
> URL: https://issues.apache.org/jira/browse/PIG-85
> Project: Pig
> Issue Type: Bug
> Reporter: Anand Murugappan
> Attachments: PIG_85_escaping_parameters.patch
>
>
> A PIG command like -
> store abc into 'abc' using PigStorage('\x01');
> does not recognize hat the user is requesting the data to by ^A separated.
> Instead the data that is stored is literally separated by the string '\x01'.
> Neither does punching in ^A directly through the editor, nor do any other
> strings like \u0001 help.
> Using a ^A directly through the editor complains about it being an invalid
> XML character and bails out.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
