HonahX commented on PR #922: URL: https://github.com/apache/iceberg-python/pull/922#issuecomment-2227281955
> I wonder if we can treat the old property names (i.e. aws_access_key_id) as an alias for the fallback config (client.access-key-id) versus fully deprecating it in 0.8.0 @kevinjqliu Thanks for raising this! I chose to deprecate `aws_access_key_id` because I would like to take this chance to formalize the property names in pyiceberg (iceberg), to use `<service name>.` prefix and hyphens to connect words. Another reason is that I think it is better to ultimately having one property name for a specific functionality to avoid confusion. Please let me know if these reasons for the deprecation make sense. We could also consider postponing the deprecation to a later release if more discussion is needed or extending the deprecation notice period to 0.9.0 or later to give users more time to adjust. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
