puchengy commented on PR #7299:
URL: https://github.com/apache/iceberg/pull/7299#issuecomment-1507422241

   > Thank you for your contributions!
   > 
   > I have a question regarding the overall solution for the issue addressed 
in this PR. Please forgive me if this is a silly one. I recall that the 
namemapping JSON object allows for specifying multiple names for a field. Would 
it be possible to solve the case sensitivity issue by specifying alternate 
field names in the namemapping? Are there any real-world scenarios that would 
prevent us from modifying the namemapping and necessitate the solution 
presented in this PR? I look forward to hearing your insights on this matter.
   
   The issue we encountered happens when building iceberg table from hive table 
where hive table already has schema name case mismatch between its parquet file 
and table schema. While your solution should work, but overall I feel it adds 
more complexity given that case insensitivity support is already there for 
other places in iceberg. 


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to