voonhous commented on PR #17669:
URL: https://github.com/apache/hudi/pull/17669#issuecomment-3685598271

   > Hi @voonhous! Thank you for reply. You are right, nested field will see 
much performance improve this way. And we can put this in utility class, but 
which one should I put it in? I thought HoodieAvroUtils, but I see you are 
working on its removal now.
   
   `HoodieAvroUtils` makes sense. We can add it in there first as it's a direct 
manipulation on a Avro GenericRecord. While we are actively removing it, 
`HoodieSchema` is still delegating to it. So, full removal will not happen so 
quickly. 
   
   As of now, i feel the most apt and suitable area to place it.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to