[GitHub] [iceberg] rdblue commented on pull request #2717: fix inconsistent case type of field name

[GitBox <gi...@apache.org>]

rdblue commented on pull request #2717:
URL: https://github.com/apache/iceberg/pull/2717#issuecomment-869867090


   This is a correctness problem. The contract for name-based methods is to match name exactly. Without a proposal for changing to case insensitive, I don't think this is a good idea. I'm going to close this PR. If you'd like to pursue this, I recommend creating a case insensitive record implementation or discussing what the behavior should be in a wider context. Simply changing the behavior is not a good idea.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org