Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-26 Thread via GitHub
aokolnychyi merged PR #11146: URL: https://github.com/apache/iceberg/pull/11146 -- 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...@icebe

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-26 Thread via GitHub
aokolnychyi commented on PR #11146: URL: https://github.com/apache/iceberg/pull/11146#issuecomment-2377400810 Thanks for the hard work, @ajantha-bhat! Great to have this in. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-25 Thread via GitHub
ajantha-bhat commented on PR #11146: URL: https://github.com/apache/iceberg/pull/11146#issuecomment-2376070543 @aokolnychyi: Thanks for the review and guidance. I have addressed the final nits. -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-25 Thread via GitHub
aokolnychyi commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1776149156 ## core/src/main/java/org/apache/iceberg/PartitionStatsUtil.java: ## @@ -0,0 +1,137 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-24 Thread via GitHub
aokolnychyi commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1774082285 ## core/src/jmh/java/org/apache/iceberg/PartitionStatsUtilBenchmark.java: ## @@ -0,0 +1,104 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-24 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1773361634 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-24 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1773357839 ## core/src/main/java/org/apache/iceberg/PartitionStatsUtil.java: ## @@ -0,0 +1,136 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-24 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1773135088 ## core/src/main/java/org/apache/iceberg/PartitionStatsUtil.java: ## @@ -0,0 +1,136 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-24 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1773092664 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
aokolnychyi commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1772724334 ## core/src/main/java/org/apache/iceberg/PartitionStatsUtil.java: ## @@ -0,0 +1,136 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771927587 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771883618 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771881650 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771880686 ## core/src/main/java/org/apache/iceberg/PartitionStatsUtil.java: ## @@ -0,0 +1,150 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771880045 ## core/src/main/java/org/apache/iceberg/BaseScan.java: ## @@ -289,4 +289,21 @@ private static Schema lazyColumnProjection(TableScanContext context, Schema sche

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771865504 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-23 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1771606149 ## core/src/main/java/org/apache/iceberg/PartitionStatsUtil.java: ## @@ -0,0 +1,150 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-20 Thread via GitHub
aokolnychyi commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1769469167 ## core/src/main/java/org/apache/iceberg/BaseScan.java: ## @@ -289,4 +289,21 @@ private static Schema lazyColumnProjection(TableScanContext context, Schema sche

Re: [PR] Core: Add a util to compute partition stats [iceberg]

2024-09-16 Thread via GitHub
ajantha-bhat commented on code in PR #11146: URL: https://github.com/apache/iceberg/pull/11146#discussion_r1761260950 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more