Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
0.10.0
-
None
-
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:
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.