n3nash commented on pull request #2424:
URL: https://github.com/apache/hudi/pull/2424#issuecomment-758449461


   @pratyakshsharma Do you have a use-case of deleting fields ? What is the 
reason for supporting deleting fields. Has deleting fields case been tested for 
all types of cases such as upserts ? Generally, the parquet-avro reader will 
throw an exception right now when a smaller schema (schema for which a field 
has been deleted) is used to read a parquet file written with a larger schema. 
Have you tested this scenario ? 
   If not, I suggest we revert this particular change and think of a more 
holistic way to support deletion of fields from schema. 


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


Reply via email to