iemejia commented on pull request #1635:
URL: https://github.com/apache/hive/pull/1635#issuecomment-721984455


   @sunchao great to see some of the changes merged. I cannot open the link. I 
got a 404 error now (before it was a sockets error who aborted the access 
eagerly).
   
   I am not sure if I understand. I upgraded Avro to the latest stable release 
(1.10.0) but I expect this will trigger the failures previously discussed in 
the ticket (on ill defined schemas on avro_deserialize_map_null.q, 
parquet_map_null.q) and fixed by 
[AVRO-2817](https://issues.apache.org/jira/browse/AVRO-2817) but that require 
of the 1.10.1 release.
   
   Great to see #1621 merged, is there already a PR to cherry-pick that one 
into `branch-2.3`? Do you want me to do that?
   I suppose however we might require also the changes on `TypeInfoToSchema` 
from this PR too to make it also forward compatible because the Field 
constructor with JsonNode as a parameter is not available after Avro 1.9.
   


----------------------------------------------------------------
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]

Reply via email to