Avro
  1. Avro
  2. AVRO-276

TestGenAvro fails because of charset inconsistencies

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: java
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      On the Mac, I get a failure with TestGenAvro. I traced it down to not explicitly setting the string encoding when reading the expected input. Patch to follow shortly.

      1. AVRO-276.patch.txt
        2 kB
        Philip Zeyliger

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        17m 59s 1 Doug Cutting 31/Dec/09 21:30
        Resolved Resolved Closed Closed
        59d 19h 39m 1 Doug Cutting 01/Mar/10 17:09
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Doug Cutting made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Hadoop Flags [Reviewed]
        Fix Version/s 1.3.0 [ 12314318 ]
        Resolution Fixed [ 1 ]
        Hide
        Doug Cutting added a comment -

        I just committed this. Thanks, Philip.

        Show
        Doug Cutting added a comment - I just committed this. Thanks, Philip.
        Philip Zeyliger made changes -
        Field Original Value New Value
        Attachment AVRO-276.patch.txt [ 12429221 ]
        Hide
        Philip Zeyliger added a comment -

        This is essentially a one-liner, replacing FileReader(...) with InputStreamReader(InputStream(..., utf-8)). I'd also like to sneak in a change to AllTests.launch, which allows this test to run in Eclipse (it introduced a dependency on a new System variable).

        Show
        Philip Zeyliger added a comment - This is essentially a one-liner, replacing FileReader(...) with InputStreamReader(InputStream(..., utf-8)). I'd also like to sneak in a change to AllTests.launch, which allows this test to run in Eclipse (it introduced a dependency on a new System variable).
        Philip Zeyliger created issue -

          People

          • Assignee:
            Philip Zeyliger
            Reporter:
            Philip Zeyliger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development