cloud-fan commented on a change in pull request #24405: [SPARK-27506][SQL] Allow deserialization of Avro data using compatible schemas URL: https://github.com/apache/spark/pull/24405#discussion_r313836701
########## File path: external/avro/src/main/scala/org/apache/spark/sql/avro/AvroDataToCatalyst.scala ########## @@ -53,7 +54,9 @@ case class AvroDataToCatalyst( @transient private lazy val avroSchema = new Schema.Parser().parse(jsonFormatSchema) - @transient private lazy val reader = new GenericDatumReader[Any](avroSchema) Review comment: do you mean the reader must use a schema that matches the avro encoded binary, but we can use a compatible schema in `AvroDeserializer`, which can do schema evolution? ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
