Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-27 Thread via GitHub
kevinjqliu merged PR #1384: URL: https://github.com/apache/iceberg-python/pull/1384 -- 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...@i

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-27 Thread via GitHub
kevinjqliu commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2504632380 accidentally squashed the commits... redoing in #1384 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-27 Thread via GitHub
kevinjqliu merged PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369 -- 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...@i

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-27 Thread via GitHub
kevinjqliu commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2504540699 Added #1383 -- 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.

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-26 Thread via GitHub
kevinjqliu commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2501494494 thanks @fokko cherry-picked #1373 as well -- 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

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-25 Thread via GitHub
kevinjqliu commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2499059724 changed the base branch to [apache:pyiceberg-0.8.x](https://github.com/apache/iceberg-python/tree/pyiceberg-0.8.x) I want to include #1373 in this too so the instructions

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-25 Thread via GitHub
Fokko commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2499039970 I was grepping for `git tag`, but it is already in there :) https://github.com/kevinjqliu/iceberg-python/blob/main/mkdocs/docs/how-to-release.md#create-a-github-release-note -- Thi

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-25 Thread via GitHub
Fokko commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2499032683 I think we should target this PR [to the `pyiceberg-0.8.x` branch I created earlier this week](https://github.com/apache/iceberg-python/tree/pyiceberg-0.8.x), and then proceed with

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-25 Thread via GitHub
kevinjqliu commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2499014996 I assume a new branch is needed for a patch release, but it seems like we just need a tag. Originally I saw that the iceberg 1.7.1 patch release had its own branch, https://

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-25 Thread via GitHub
Fokko commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2498965096 @kevinjqliu One question, I noticed that this PR targets the `pyiceberg-0.8.1` branch, what's the value of this? I missed this step when reviewing: https://github.com/apache/iceberg-

Re: [PR] [release] Pyiceberg 0.8.1 [iceberg-python]

2024-11-25 Thread via GitHub
Fokko commented on PR #1369: URL: https://github.com/apache/iceberg-python/pull/1369#issuecomment-2498954423 Looks good @kevinjqliu! Thanks for creating that nice overview! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u