Avro
  1. Avro
  2. AVRO-375

Initializing uninizialized encoders fail

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: java
    • Labels:
      None

      Description

      If you construct an encoder with null as OutputStream and try to call init() with a valid OutputStream later, the code throws NullPointerException.

      1. AVRO-375.patch
        2 kB
        Thiruvalluvan M. G.
      2. AVRO-375-test.patch
        1 kB
        Thiruvalluvan M. G.

        Issue Links

          Activity

          Thiruvalluvan M. G. created issue -
          Hide
          Thiruvalluvan M. G. added a comment -

          The test patch adds unit-tests that catch the problem and the actual patch fixes the problem.

          Show
          Thiruvalluvan M. G. added a comment - The test patch adds unit-tests that catch the problem and the actual patch fixes the problem.
          Thiruvalluvan M. G. made changes -
          Field Original Value New Value
          Attachment AVRO-375.patch [ 12431273 ]
          Attachment AVRO-375-test.patch [ 12431274 ]
          Thiruvalluvan M. G. made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Thiruvalluvan M. G. added a comment -

          Committed revision 903335.

          Show
          Thiruvalluvan M. G. added a comment - Committed revision 903335.
          Thiruvalluvan M. G. made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Philip Zeyliger added a comment -

          Thiru,

          Did you forget to commit the test as well? Please also check that it has ASF license (the attached patch doesn't).

          Thanks!

          – Philip

          Show
          Philip Zeyliger added a comment - Thiru, Did you forget to commit the test as well? Please also check that it has ASF license (the attached patch doesn't). Thanks! – Philip
          Hide
          Thiruvalluvan M. G. added a comment -

          Thanks Philip for pointing it out. I added the tests with ASF licence. Committed revision 903351.

          Show
          Thiruvalluvan M. G. added a comment - Thanks Philip for pointing it out. I added the tests with ASF licence. Committed revision 903351.
          Thiruvalluvan M. G. made changes -
          Link This issue relates to AVRO-385 [ AVRO-385 ]
          Doug Cutting made changes -
          Fix Version/s 1.3.0 [ 12314318 ]
          Doug Cutting made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          5m 29s 1 Thiruvalluvan M. G. 25/Jan/10 03:24
          Patch Available Patch Available Resolved Resolved
          1d 14h 13m 1 Thiruvalluvan M. G. 26/Jan/10 17:37
          Resolved Resolved Closed Closed
          33d 23h 31m 1 Doug Cutting 01/Mar/10 17:09

            People

            • Assignee:
              Thiruvalluvan M. G.
              Reporter:
              Thiruvalluvan M. G.
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development