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

Siying Dong commented on HIVE-3030:
-----------------------------------

Logic looks good to me. I'll run unit tests now. In the mean time, can you add 
tests to cover those new cases? Cases like escaping '\', and unescaping cases 
like '\\', ,'\\\t' or '\\\t'?
                
> escape more chars for script operator
> -------------------------------------
>
>                 Key: HIVE-3030
>                 URL: https://issues.apache.org/jira/browse/HIVE-3030
>             Project: Hive
>          Issue Type: Bug
>          Components: Query Processor
>            Reporter: Namit Jain
>            Assignee: Namit Jain
>
> Only new line was being escaped.
> The same behavior needs to be done for carriage returns, and tabs

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to