Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-15 Thread via GitHub
Xuanwo merged PR #771: URL: https://github.com/apache/iceberg-rust/pull/771 -- 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.a

Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-15 Thread via GitHub
Xuanwo commented on PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#issuecomment-2543915869 Thank you @c-thiel for working on this, and thank @Fokko, @liurenjie1024 and @sdd for the review. We have waited for this for so long, let's move! -- This is an automated message from

Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-15 Thread via GitHub
c-thiel commented on PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#issuecomment-2543820634 @liurenjie1024 conflicts resolved. Had to slightly change a function signature: https://github.com/apache/iceberg-rust/pull/771/files#diff-8389535350ef7cddc266dfd18d589a978643da033

Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-15 Thread via GitHub
sdd commented on PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#issuecomment-2543833200 Thanks @c-thiel, those comments are great, looks good to me :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-14 Thread via GitHub
sdd commented on PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#issuecomment-2543368328 Hey all. I'm a bit late to the party on this one, but having taken a first look after not having been closely involved in this, there's something that struck me as being a little confusing

Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-14 Thread via GitHub
sdd commented on code in PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#discussion_r1885410880 ## crates/iceberg/src/spec/partition.rs: ## @@ -55,69 +52,43 @@ impl PartitionField { } } -/// Partition spec that defines how to produce a tuple of partition va

Re: [PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-14 Thread via GitHub
liurenjie1024 commented on PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#issuecomment-2543089808 cc @c-thiel Seems we need to fix conflicts. -- 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] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-11 Thread via GitHub
Fokko commented on PR #771: URL: https://github.com/apache/iceberg-rust/pull/771#issuecomment-2535647234 Gentle ping @liurenjie1024 so we can wrap up the 0.4.0 milestone -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[PR] feat!: Remove `BoundPartitionSpec` [iceberg-rust]

2024-12-09 Thread via GitHub
c-thiel opened a new pull request, #771: URL: https://github.com/apache/iceberg-rust/pull/771 (no comment) -- 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