Remove usage of deprecated functions from the codebase

[kevinjqliu]

Apache Iceberg version

None

Please describe the bug 🐞

Functions that are marked deprecated emit a warning. We should remove the use of deprecated functions from the codebase.

List of deprecated functions
https://github.com/search?q=repo%3Aapache%2Ficeberg-python+deprecated%28+language%3APython&type=code&l=Python

For example, #1318

[ndrluis]

@kevinjqliu Could you explain what you expect? I didn't understand the purpose of this issue. I believe we are supposed to warn the user when they use a deprecated API.

Were you referring to removing the deprecated code from version 0.8.0?

[kevinjqliu]

@ndrluis yea the difference is to not use the deprecated API in pyiceberg's own codebase, which will emit warnings. For example, #1326 removed the use of identifier_to_tuple_without_catalog.

[ndrluis]

Oh, ok. I believe we need to work on #1099. The way we are currently working makes it hard to deprecate because most of our code is public.

[kevinjqliu]

That's true. For the short term, it'll be good to go through the API already marked for deprecation and make sure the codebase itself is not using them

[github-actions]

This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.

[kris-gaudel]

Hi there, I'm interested in contributing for the first time. Is this issue still relevant? If so, can someone assign this to me?

[kevinjqliu]

i did a quick check on the codebase. i think we removed the usage of deprecated functions in pyiceberg's own codebase. so closing this issue