pvary opened a new pull request #2078: URL: https://github.com/apache/iceberg/pull/2078
During the testing of the write path @lcspinter run more extensive tests and identified an issue that fields which are supposed to use the WriteObjectInspector does not work. The root cause was that the we both have to use the Hive generated ObjectInspectors to get the primitive object data from the Hive data model, and then use the IcebergObjectInspectors to convert them to the Iceberg data model. The proposed solution creates an ObjectInspectorPair for every field which contains both the Hive and the Iceberg ObjectInspectors, and uses them when the conversion is needed. Testing is done with the original code with changing the ObjectInspectors used. Also we hit #2053 so I had to disable the case sensitivity tests until #2053 is committed. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
