Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2025-01-09 Thread via GitHub
rahil-c commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2581526854 > This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2025-01-09 Thread via GitHub
github-actions[bot] commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2581488323 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pul

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-12-10 Thread via GitHub
rahil-c commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2532789316 @rdblue @danielcweeks Was wondering if you guys can take a look whenever you get a chance? -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857361617 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java: ## @@ -0,0 +1,142 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857360604 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java: ## @@ -0,0 +1,142 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857260573 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857259376 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857257695 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857256498 ## core/src/test/java/org/apache/iceberg/rest/requests/TestFetchScanTasksRequest.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-25 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1857154984 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = PartitionSpec.

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-21 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1852995146 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-11-21 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1852987342 ## core/src/test/java/org/apache/iceberg/rest/requests/TestFetchScanTasksRequest.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-29 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1821364081 ## core/src/main/java/org/apache/iceberg/ScanTasksIterable.java: ## @@ -120,7 +146,21 @@ private void fetchScanTasks(String planTask) { headers,

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-28 Thread via GitHub
RussellSpitzer commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2442216411 It looks like this won't be ready this week so I'll move it out to the next release milestone. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-22 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1811770900 ## core/src/main/java/org/apache/iceberg/ScanTasksIterable.java: ## @@ -120,7 +146,21 @@ private void fetchScanTasks(String planTask) { header

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-22 Thread via GitHub
rahil-c commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2430491796 @amogh-jahagirdar @rdblue @danielcweeks @nastra Pushed the following commit `Utilize ParallelIterable, and use table prop instead of rest planning`, which addressed some of the plann

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
rahil-c commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2427441270 Please see the following PR: https://github.com/apache/iceberg/pull/11369 which just contains the changes for the model and parser.s -- This is an automated message from the Apache Gi

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1809023652 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = Parti

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1809023652 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = Parti

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1809034227 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1809023652 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = Parti

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808966984 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (AS

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808951841 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponse.java: ## @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808913187 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = PartitionSpec.b

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808909983 ## core/src/main/java/org/apache/iceberg/rest/ResourcePaths.java: ## @@ -125,4 +125,38 @@ public String view(TableIdentifier ident) { public String renameView() {

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808905126 ## core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java: ## @@ -512,6 +521,23 @@ public Table loadTable(SessionContext context, TableIdentifier identifi

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808900734 ## core/src/main/java/org/apache/iceberg/RESTFileScanTaskParser.java: ## @@ -0,0 +1,99 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mor

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808898678 ## core/src/main/java/org/apache/iceberg/RESTFileScanTaskParser.java: ## @@ -0,0 +1,99 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mor

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808893022 ## core/src/main/java/org/apache/iceberg/FetchScanTasksResponseParser.java: ## @@ -0,0 +1,160 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808891083 ## core/src/main/java/org/apache/iceberg/FetchScanTasksResponseParser.java: ## @@ -0,0 +1,160 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808889723 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808887883 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808881928 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808878506 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808878137 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808873572 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponse.java: ## @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808874947 ## core/src/main/java/org/apache/iceberg/rest/responses/PlanTableScanResponse.java: ## @@ -0,0 +1,139 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808871838 ## core/src/main/java/org/apache/iceberg/FetchScanTasksResponseParser.java: ## @@ -0,0 +1,160 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808870856 ## core/src/main/java/org/apache/iceberg/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,177 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808872418 ## core/src/main/java/org/apache/iceberg/PlanTableScanResponseParser.java: ## @@ -0,0 +1,212 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808864754 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponse.java: ## @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808860438 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java: ## @@ -0,0 +1,142 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808855386 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java: ## @@ -0,0 +1,142 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808837117 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808852759 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java: ## @@ -0,0 +1,142 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808851980 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java: ## @@ -0,0 +1,142 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808850772 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808847616 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808846800 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808844659 ## core/src/test/java/org/apache/iceberg/rest/requests/TestFetchScanTasksRequest.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808839428 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequestParser.java: ## @@ -0,0 +1,57 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808735606 ## core/src/test/java/org/apache/iceberg/rest/requests/TestFetchScanTasksRequest.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808740545 ## core/src/test/java/org/apache/iceberg/rest/responses/TestFetchScanTasksResponse.java: ## @@ -0,0 +1,175 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808741712 ## core/src/test/java/org/apache/iceberg/rest/responses/TestPlanTableScanResponse.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-21 Thread via GitHub
nastra commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1808739786 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-20 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1807950377 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = PartitionSpec.

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-19 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1807542082 ## core/src/test/java/org/apache/iceberg/TestBase.java: ## @@ -63,7 +63,7 @@ public class TestBase { public static final PartitionSpec SPEC = Parti

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-17 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1805150599 ## core/src/test/java/org/apache/iceberg/rest/requests/TestPlanTableScanRequest.java: ## @@ -0,0 +1,153 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-16 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1803561481 ## core/src/main/java/org/apache/iceberg/BaseFile.java: ## @@ -434,6 +434,11 @@ public StructLike partition() { return partitionData; } + public v

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-16 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1803556362 ## core/src/main/java/org/apache/iceberg/BaseFile.java: ## @@ -434,6 +434,11 @@ public StructLike partition() { return partitionData; } + public v

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-16 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1803556362 ## core/src/main/java/org/apache/iceberg/BaseFile.java: ## @@ -434,6 +434,11 @@ public StructLike partition() { return partitionData; } + public v

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-15 Thread via GitHub
rdblue commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1802032303 ## core/src/main/java/org/apache/iceberg/BaseFile.java: ## @@ -434,6 +434,11 @@ public StructLike partition() { return partitionData; } + public void setPar

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-15 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1801692407 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-14 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1800245093 ## core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java: ## @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-14 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1800243915 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-14 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1800243915 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-14 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1799822128 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software Foundatio

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
singhpk234 commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798744690 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
singhpk234 commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798744690 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798718031 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798639774 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798639774 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798639774 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798639774 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
singhpk234 commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798637028 ## core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java: ## @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
singhpk234 commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798637028 ## core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java: ## @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798635808 ## core/src/main/java/org/apache/iceberg/RESTTable.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more c

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
singhpk234 commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798632724 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-13 Thread via GitHub
singhpk234 commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1795908515 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1798013229 ## .palantir/revapi.yml: ## @@ -1058,6 +1058,11 @@ acceptedBreaks: new: "method void org.apache.iceberg.encryption.PlaintextEncryptionManager::()" just

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797956652 ## api/src/main/java/org/apache/iceberg/expressions/ResidualEvaluator.java: ## @@ -89,6 +89,12 @@ public static ResidualEvaluator of(PartitionSpec spec, Expression e

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797956652 ## api/src/main/java/org/apache/iceberg/expressions/ResidualEvaluator.java: ## @@ -89,6 +89,12 @@ public static ResidualEvaluator of(PartitionSpec spec, Expression e

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797956080 ## core/src/main/java/org/apache/iceberg/RESTTable.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributo

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797946507 ## core/src/main/java/org/apache/iceberg/ScanTasksIterable.java: ## @@ -0,0 +1,129 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more c

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797931018 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-12 Thread via GitHub
amogh-jahagirdar commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797722022 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797486598 ## core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java: ## @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797443724 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponseParser.java: ## @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software Foundatio

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797318171 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java: ## @@ -0,0 +1,162 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797317407 ## core/src/main/java/org/apache/iceberg/ScanTasksIterable.java: ## @@ -0,0 +1,129 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more c

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797315352 ## core/src/main/java/org/apache/iceberg/RESTPlanningMode.java: ## @@ -0,0 +1,47 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more con

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797313224 ## core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java: ## @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797312608 ## core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java: ## @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797310997 ## core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java: ## @@ -0,0 +1,162 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797310350 ## core/src/main/java/org/apache/iceberg/rest/responses/PlanTableScanResponse.java: ## @@ -0,0 +1,127 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797309737 ## core/src/main/java/org/apache/iceberg/rest/responses/FetchScanTasksResponse.java: ## @@ -0,0 +1,96 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797273923 ## core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java: ## @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797270059 ## .palantir/revapi.yml: ## @@ -1058,6 +1058,11 @@ acceptedBreaks: new: "method void org.apache.iceberg.encryption.PlaintextEncryptionManager::()" just

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797271023 ## core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java: ## @@ -0,0 +1,250 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API, Core: Add scan planning apis to REST Catalog [iceberg]

2024-10-11 Thread via GitHub
rahil-c commented on code in PR #11180: URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797270059 ## .palantir/revapi.yml: ## @@ -1058,6 +1058,11 @@ acceptedBreaks: new: "method void org.apache.iceberg.encryption.PlaintextEncryptionManager::()" just

  1   2   >