rangareddy commented on issue #17259:
URL: https://github.com/apache/hudi/issues/17259#issuecomment-5365806977

   This issue was reviewed as part of the JIRA-migrated backlog triage 
(HUDI-7602).
   
   **Findings: needs a reproduction against a current build before it can be 
dispositioned.**
   
   The report is concrete and actionable as written: two struct fields whose 
nested member has the same name (colName) but different decimal precision - 
decimal(11,2) and decimal(10,2) - make the writer throw on the second insert.
   
   I have deliberately not called this one from the source. The surrounding 
code has been substantially rewritten since it was filed - table versions 8 and 
9, the `BufferedRecordMerger` family replacing payload-based merging, the 
file-group-reader migration, and the `hudi-common` package reorganisations in 
#19193 / #19195. With that much churn, inferring either "still broken" or 
"silently fixed" would be a guess rather than a finding, and both errors are 
costly here.
   
   The good news is that reproducing it is cheap: the SQL in the description is 
fully self-contained (create table with two structs, then two inserts) and runs 
in spark-sql without any external dependency.
   
   There is also no commit anywhere in the tree referencing HUDI-7602, so if 
this was fixed it was incidental to another change rather than deliberate - 
which is another reason to verify rather than assume.
   
   Keeping this open.
   


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