Uploaded image for project: 'Pig'
  1. Pig
  2. PIG-4213

CSVExcelStorage not quoting texts containing \r (CR) when storing

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • 0.12.0
    • 0.15.0
    • piggybank
    • None
    • Reviewed

    Description

      Managing tweets information I found that someone wrote a multiline tweet in Mac OS 9 (or bellow). When exporting the text, it is not being 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

      Attachments

        1. PIG-4213v3.patch
          9 kB
          alfonso.nishikawa
        2. PIG-4213v2.patch
          9 kB
          alfonso.nishikawa
        3. PIG-4213v1.patch
          6 kB
          alfonso.nishikawa
        4. PIG-4213_tests_updated_CR_before.patch
          4 kB
          alfonso.nishikawa

        Activity

          People

            alfonsonishikawa Alfonso Nishikawa Muñumer
            alfonsonishikawa Alfonso Nishikawa Muñumer
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: