[GitHub] [spark] dongjoon-hyun commented on pull request #31133: [SPARK-26836][SQL] Supporting Avro schema evolution for partitioned Hive tables

2021-02-05 Thread GitBox
dongjoon-hyun commented on pull request #31133: URL: https://github.com/apache/spark/pull/31133#issuecomment-773564889 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [spark] dongjoon-hyun commented on pull request #31133: [SPARK-26836][SQL] Supporting Avro schema evolution for partitioned Hive tables

2021-02-04 Thread GitBox
dongjoon-hyun commented on pull request #31133: URL: https://github.com/apache/spark/pull/31133#issuecomment-773567152 Thank you for updating, @attilapiros . But, I'm still not sure about the other properties in those namespace. For `avro.schema.literal`, your test case provides the valu

[GitHub] [spark] dongjoon-hyun commented on pull request #31133: [SPARK-26836][SQL] Supporting Avro schema evolution for partitioned Hive tables

2021-02-04 Thread GitBox
dongjoon-hyun commented on pull request #31133: URL: https://github.com/apache/spark/pull/31133#issuecomment-773564889 @xkrogen . That another instance explicitly uses `AvroSerdeUtils.AvroTableProperties.SCHEMA_LITERAL.getPropName()`, not other property. My suggestion, `it seems that we on

[GitHub] [spark] dongjoon-hyun commented on pull request #31133: [SPARK-26836][SQL] Supporting Avro schema evolution for partitioned Hive tables

2021-02-02 Thread GitBox
dongjoon-hyun commented on pull request #31133: URL: https://github.com/apache/spark/pull/31133#issuecomment-771470866 Sure, I'll review right now, @attilapiros . This is an automated message from the Apache Git Service. To r

[GitHub] [spark] dongjoon-hyun commented on pull request #31133: [SPARK-26836][SQL] Supporting Avro schema evolution for partitioned Hive tables

2021-02-02 Thread GitBox
dongjoon-hyun commented on pull request #31133: URL: https://github.com/apache/spark/pull/31133#issuecomment-771470866 Sure, I'll review right now, @attilapiros . This is an automated message from the Apache Git Service. To r