[jira] [Updated] (PIG-4213) CSVExcelStorage not quoting texts containing \r (CR) when storing

["Alfonso Nishikawa (JIRA)" <ji...@apache.org>]

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

Alfonso Nishikawa updated PIG-4213:
-----------------------------------
    Attachment: PIG-4213v1.patch

I upload a proposal patch (PIG-4213v1.patch). Tests included (I tried my best). I assume TestLoader does not break lines by CR (\r), but tests are prepared to expose me if I am wrong ;)
I couldn't test since I am not at home and this Celeron from 2005 in front of me can't handle the build...

> CSVExcelStorage not quoting texts containing \r (CR) when storing
> -----------------------------------------------------------------
>
>                 Key: PIG-4213
>                 URL: https://issues.apache.org/jira/browse/PIG-4213
>             Project: Pig
>          Issue Type: Improvement
>          Components: piggybank
>    Affects Versions: 0.12.0
>            Reporter: Alfonso Nishikawa
>            Priority: Trivial
>         Attachments: PIG-4213v1.patch
>
>
> Managing tweets information I found that someone wrote a multiline tweet in Mac OS 9 (or bellow). When exporting the text is not bein quoted so LibreOffice can't import the cell properly (don't try Excel 2007 because it's bugged).
> I suggest including the CR case in the same way as commented in http://svn.apache.org/viewvc/pig/tags/release-0.12.1/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/CSVExcelStorage.java?view=markup#l315



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)