CurtHagenlocher commented on PR #36797: URL: https://github.com/apache/arrow/pull/36797#issuecomment-1708382984
> Hmm, it seems C# producing unions fails in the integration test, though I thought this had worked before, but it clearly can't have. There may be more than one problem, but the obvious one is the exception thrown when comparing the data generated from the JSON template with the checked-in data. The template for union test data contains duplicate names -- two fields named "struct" and two fields named "dense". One of each field is marked as nullable in the JSON and one as non-nullable. But the data file that's been checked-in shows all four fields as being non-nullable. I'm not sure what to make of this. I don't see code in the Java or C++ implementations which special-case anything about nullability -- either in reading from the JSON template or in the implementation classes themselves. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
