Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.11, 0.10.1
    • Component/s: internal-udfs
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      TOBAG only reports an output schema beyond {{

      {(NULL)}

      }} when all input field schemas match deeply, including field schema aliases. This seems wrong to me. Shouldn't it just require recursive type equality?

      For relevant code, see:

      http://svn.apache.org/viewvc/pig/tags/release-0.9.2/src/org/apache/pig/builtin/TOBAG.java?view=markup#l142

          private boolean nullEquals(Schema currentSchema, Schema newSchema) {
              if(currentSchema == null){
                  if(newSchema != null){
                      return false;
                  }
                  return true;
              }
              return currentSchema.equals(newSchema);
          }
      

      The included patch modifies the return line to use Schema.equals(currentSchema, newSchema, false, true) to avoid alias matching requirement.

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Unassigned
            Reporter:
            Andy Schlaikjer
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development