Avro
  1. Avro
  2. AVRO-558

Java server serializing error union schema branch number incorrectly for single-branch unions

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: java
    • Labels:
      None

      Description

      In my testing of HBASE-2400, I am seeing a branch index of 1 turn up when 0 should have been encoded for messages with a single branch in their union of error schemas. See attached protocol definition, Java server, and Python client which reproduces the error.

        Issue Links

          Activity

          Jeff Hammerbacher created issue -
          Jeff Hammerbacher made changes -
          Field Original Value New Value
          Link This issue is related to AVRO-559 [ AVRO-559 ]
          Show
          Jeff Hammerbacher added a comment - Protocol definition: http://github.com/hammer/avro-debug/blob/trunk/debug/src/fake.avpr Fake server: http://github.com/hammer/avro-debug/blob/trunk/debug/src/FakeServer.java Fake client: http://github.com/hammer/avro-debug/blob/trunk/debug/src/fake_client.py
          Hide
          Jeff Hammerbacher added a comment -

          Something may be borked in SpecificResponder#writeError for single-branch unions, which are admittedly a funny case, but I can't debug that craziness.

          Show
          Jeff Hammerbacher added a comment - Something may be borked in SpecificResponder#writeError for single-branch unions, which are admittedly a funny case, but I can't debug that craziness.
          Hide
          Jeff Hammerbacher added a comment -
          Show
          Jeff Hammerbacher added a comment - See AVRO-447
          Jeff Hammerbacher made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Won't Fix [ 2 ]
          Jeff Hammerbacher made changes -
          Link This issue relates to AVRO-447 [ AVRO-447 ]
          Doug Cutting made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          6h 42m 1 Jeff Hammerbacher 03/Jun/10 18:53
          Resolved Resolved Closed Closed
          83d 1h 8m 1 Doug Cutting 25/Aug/10 20:01

            People

            • Assignee:
              Unassigned
              Reporter:
              Jeff Hammerbacher
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development