Avro
  1. Avro
  2. AVRO-1056

ReflectData thinks the instance of a java.util.Map as a record type

    Details

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

      MacOSX 10.7.2
      Eclipse 3.7
      JDK1.6

      Description

      the avro's java api org.apache.avro.reflect.ReflectData.isRecord() should add the following code.

      if (datum instanceof Map) return false;

      1. ReflectData.java.patch
        0.5 kB
        Kevin Zhao
      2. AVRO-1056.patch
        1 kB
        Doug Cutting

        Activity

        Hide
        Kevin Zhao added a comment -

        request for code reviewing~

        Show
        Kevin Zhao added a comment - request for code reviewing~
        Hide
        Doug Cutting added a comment -

        Patch looks good to me and existing tests pass with it applied.

        We should also add a test to TestReflect.java that fails without this patch but succeeds with it.

        Show
        Doug Cutting added a comment - Patch looks good to me and existing tests pass with it applied. We should also add a test to TestReflect.java that fails without this patch but succeeds with it.
        Hide
        Doug Cutting added a comment -

        Here's a version of the patch that includes a test.

        I'll commit this soon unless someone objects.

        Show
        Doug Cutting added a comment - Here's a version of the patch that includes a test. I'll commit this soon unless someone objects.
        Hide
        Doug Cutting added a comment -

        I committed this.

        Show
        Doug Cutting added a comment - I committed this.

          People

          • Assignee:
            Kevin Zhao
            Reporter:
            Kevin Zhao
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development