Remove Python 3.9 support

[Aniketsy]

#2534

Removed python 3.9 support.

Please let me know if my approach or fix needs any improvements . I’m open to feedback and happy to make changes based on suggestions.
Thankyou !

[Fokko]

@Aniketsy Thanks for working on this! Could you update this one as well:

iceberg-python/pyproject.toml

Lines 70 to 73 in 40521c8

	ray = [
	{ version = "==2.10.0", python = "<3.9", optional = true },
	{ version = ">=2.10.0,<=2.44.0", python = ">=3.9", optional = true },
	]

[Aniketsy]

@Fokko Thanks! I will update this.

[Aniketsy]

@Fokko I’ve updated the changes. I had one query, is it okay to work on an issue if no PR has been linked or mentioned for it yet?

[Fokko]

I had one query, is it okay to work on an issue if no PR has been linked or mentioned for it yet?

Yes, for sure. In Iceberg, everyone is free to work on any issue, even when someone else claims it. We often see folks picking up issues but getting into time constraints and not being able to finish the work; therefore, we don't assign it to anyone.

[Fokko]

I think we can simplify this to:

ray = { version = ">=2.10.0,<=2.44.0", optional = true }

[Aniketsy]

Thanks! sure

[Fokko]

I think we can get rid of the brackets [], since the list only has one entry :)

[Aniketsy]

Thanks for correcting me, I will update this

[Aniketsy]

@Fokko I’ve applied the changes based on your suggestions. Please let me know if any further improvements are needed.

[Fokko]

@Aniketsy It looks like we're one poetry lock short? Can you also pull in latest main?

[Aniketsy]

yes, i will update .

[Aniketsy]

@Fokko I’m not sure if I added the poetry.lock correctly. I installed the dependencies and added the file directly , please let me know if I did something wrong.

[kevinjqliu]

there are a few more mentions of 3.9

• .github/workflows/pypi-build-artifacts.yml
• .github/workflows/svn-build-artifacts.yml
• mkdocs/docs/contributing.md

[kevinjqliu]

I’m not sure if I added the poetry.lock correctly

i usually would do make install and then poetry lock

[Aniketsy]

@kevinjqliu I've applied the changes.

[kevinjqliu]

i think we can revert this change.

https://peps.python.org/pep-0585/ is supported by 3.9

[kevinjqliu]

ah looks like this was find/replaced when we removed 3.8

857abd0#diff-c0ad53549c03c8e66a0e3574698ce24dfa722b54ca7a2fdfcb8d62ff0343ab73R205-R206

[kevinjqliu]

i think we can remove this paragraph entirely then

[Aniketsy]

thanks! i will update

[kevinjqliu]

looks like there a conflict, could you rebase this PR with main?

git fetch origin main
git rebase origin/main -Xtheirs

You will likely have to regen the poetry.lock file again

poetry lock

[Aniketsy]

@kevinjqliu I’ve removed the paragraph as suggested. Please take a look at the changes

[kevinjqliu]

LGTM!

lets wait for more folks to chime in on the devlist thread before merging
https://lists.apache.org/thread/z3dhxx2wc0dhvgq9js5ktbqf4ggsn4z8

[kevinjqliu]

oops looks like poetry doesnt like that theres no upperlimit, we can use <4.0 then

[rambleraptor]

This is fantastic! If we do remove Python 3.9, there's a bunch of new typing features in 3.10 that we can now introduce

[rambleraptor]

This looks awesome! Thank you so much for doing this

[Fokko]

@Aniketsy Could you fix the conflicts? Thanks!

[Aniketsy]

@Fokko I have fixed the conflict, please check.

[Fokko]

@Aniketsy it looks like we need to run another poetry lock

[Fokko]

@Aniketsy Please make sure to pull in latest main first 👍

[Aniketsy]

@Fokko i resolved each conflict by accepting incoming changes, did i do something wrong.

[kevinjqliu]

Thanks @Aniketsy i took a try at fixing the conflict

[kevinjqliu]

Thank you for the contribution @Aniketsy and thanks @rambleraptor @Fokko for the review!

I'll update the devlist thread as well