Re: [PR] Additional refs update for iceberg 1.9.0 [polaris]

2025-05-03 Thread via GitHub
MonkeyCanCode commented on PR #1491: URL: https://github.com/apache/polaris/pull/1491#issuecomment-2849031945 @adutra @dimas-b This is fixed by explicitly shade avro dependency. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] main: Update docker.io/prom/prometheus Docker tag to v3.3.1 [polaris]

2025-05-03 Thread via GitHub
jbonofre merged PR #1510: URL: https://github.com/apache/polaris/pull/1510 -- 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...@polaris.ap

[PR] Add new committers on website [polaris]

2025-05-03 Thread via GitHub
jbonofre opened a new pull request, #1521: URL: https://github.com/apache/polaris/pull/1521 Update website with new committers. -- 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 commen

Re: [PR] Implement federation to HadoopCatalog [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1466: URL: https://github.com/apache/polaris/pull/1466#discussion_r2072522362 ## polaris-core/src/main/java/org/apache/polaris/core/connection/hadoop/HadoopConnectionConfigInfoDpo.java: ## @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache Soft

Re: [PR] Implement federation to HadoopCatalog [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1466: URL: https://github.com/apache/polaris/pull/1466#discussion_r2072522362 ## polaris-core/src/main/java/org/apache/polaris/core/connection/hadoop/HadoopConnectionConfigInfoDpo.java: ## @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache Soft

Re: [PR] main: Update dependency com.azure:azure-sdk-bom to v1.2.34 [polaris]

2025-05-03 Thread via GitHub
jbonofre merged PR #1490: URL: https://github.com/apache/polaris/pull/1490 -- 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...@polaris.ap

Re: [I] Task handling is incomplete [polaris]

2025-05-03 Thread via GitHub
danielhumanmod commented on issue #774: URL: https://github.com/apache/polaris/issues/774#issuecomment-2849011042 This issue is also mentioned in [#1179](https://github.com/apache/polaris/pull/1179#discussion_r2052747457). I can help patch this by: 1. When a cleanup task fails, persi

Re: [PR] Implement federation to HadoopCatalog [polaris]

2025-05-03 Thread via GitHub
singhpk234 commented on code in PR #1466: URL: https://github.com/apache/polaris/pull/1466#discussion_r2072519291 ## polaris-core/src/main/java/org/apache/polaris/core/connection/hadoop/HadoopConnectionConfigInfoDpo.java: ## @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache Softwa

Re: [PR] Implement pagination for list APIs [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #273: URL: https://github.com/apache/polaris/pull/273#discussion_r2072518910 ## polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageToken.java: ## @@ -0,0 +1,179 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] Implement pagination for list APIs [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #273: URL: https://github.com/apache/polaris/pull/273#discussion_r2072518910 ## polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageToken.java: ## @@ -0,0 +1,179 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] main: Update dependency com.google.cloud:google-cloud-storage-bom to v2.52.1 [polaris]

2025-05-03 Thread via GitHub
jbonofre merged PR #1485: URL: https://github.com/apache/polaris/pull/1485 -- 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...@polaris.ap

Re: [PR] main: Update dependency org.apache.commons:commons-configuration2 to v2.12.0 [polaris]

2025-05-03 Thread via GitHub
jbonofre merged PR #1481: URL: https://github.com/apache/polaris/pull/1481 -- 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...@polaris.ap

Re: [PR] Implement federation to HadoopCatalog [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1466: URL: https://github.com/apache/polaris/pull/1466#discussion_r2072515956 ## polaris-core/src/main/java/org/apache/polaris/core/connection/hadoop/HadoopConnectionConfigInfoDpo.java: ## @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache Soft

Re: [PR] Implement federation to HadoopCatalog [polaris]

2025-05-03 Thread via GitHub
singhpk234 commented on code in PR #1466: URL: https://github.com/apache/polaris/pull/1466#discussion_r2072515021 ## polaris-core/src/main/java/org/apache/polaris/core/connection/hadoop/HadoopConnectionConfigInfoDpo.java: ## @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache Softwa

Re: [PR] Implement pagination for list APIs [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #273: URL: https://github.com/apache/polaris/pull/273#discussion_r2072471853 ## polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/OffsetPageToken.java: ## @@ -0,0 +1,113 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] Implement pagination for list APIs [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #273: URL: https://github.com/apache/polaris/pull/273#discussion_r2072471572 ## polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PolarisPage.java: ## @@ -0,0 +1,42 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] Implement pagination for list APIs [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #273: URL: https://github.com/apache/polaris/pull/273#discussion_r2072471572 ## polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PolarisPage.java: ## @@ -0,0 +1,42 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] Docs: Fix the wrong catalog name in `using polaris` page [polaris]

2025-05-03 Thread via GitHub
eric-maynard merged PR #1471: URL: https://github.com/apache/polaris/pull/1471 -- 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...@polari

Re: [PR] Refactor storage access configuration handling [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1504: URL: https://github.com/apache/polaris/pull/1504#discussion_r2072468033 ## polaris-core/src/main/java/org/apache/polaris/core/storage/StorageAccessProperty.java: ## @@ -18,15 +18,23 @@ */ package org.apache.polaris.core.storage; -

Re: [PR] Refactor storage access configuration handling [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1504: URL: https://github.com/apache/polaris/pull/1504#discussion_r2072468033 ## polaris-core/src/main/java/org/apache/polaris/core/storage/StorageAccessProperty.java: ## @@ -18,15 +18,23 @@ */ package org.apache.polaris.core.storage; -

Re: [PR] Add cleanup support for partition-level statistics files when `DROP TABLE PURGE` [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1508: URL: https://github.com/apache/polaris/pull/1508#discussion_r2072467726 ## service/common/src/main/java/org/apache/polaris/service/task/TableCleanupTaskHandler.java: ## @@ -112,7 +113,6 @@ public boolean handleTask(TaskEntity cleanupTa

Re: [PR] Refactor storage access configuration handling [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1504: URL: https://github.com/apache/polaris/pull/1504#discussion_r2072467857 ## polaris-core/src/main/java/org/apache/polaris/core/storage/IcebergStorageAccessProperty.java: ## Review Comment: That makes sense, I think the new name als

Re: [PR] Add cleanup support for partition-level statistics files when `DROP TABLE PURGE` [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1508: URL: https://github.com/apache/polaris/pull/1508#discussion_r2072467650 ## service/common/src/main/java/org/apache/polaris/service/task/TableCleanupTaskHandler.java: ## @@ -243,12 +243,13 @@ private Stream getMetadataTaskStream( pri

Re: [PR] Add cleanup support for partition-level statistics files when `DROP TABLE PURGE` [polaris]

2025-05-03 Thread via GitHub
eric-maynard commented on code in PR #1508: URL: https://github.com/apache/polaris/pull/1508#discussion_r2072467650 ## service/common/src/main/java/org/apache/polaris/service/task/TableCleanupTaskHandler.java: ## @@ -243,12 +243,13 @@ private Stream getMetadataTaskStream( pri

Re: [PR] Add new committers [polaris-tools]

2025-05-03 Thread via GitHub
eric-maynard merged PR #23: URL: https://github.com/apache/polaris-tools/pull/23 -- 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...@pola

Re: [PR] Add new committers [polaris]

2025-05-03 Thread via GitHub
jbonofre merged PR #1518: URL: https://github.com/apache/polaris/pull/1518 -- 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...@polaris.ap

Re: [PR] Disable`If-None-Match` when fetching tables in benchmark scenarios [polaris-tools]

2025-05-03 Thread via GitHub
pingtimeout commented on PR #22: URL: https://github.com/apache/polaris-tools/pull/22#issuecomment-2848665243 I found it weird that you linked the Gatling mailing list with the solution `.disableCaching` and suspected there was just one commit not pushed :-). No worries. Note that I

Re: [PR] fix overlapping menu item on the nav bar [polaris]

2025-05-03 Thread via GitHub
gfakbar20 closed pull request #1520: fix overlapping menu item on the nav bar URL: https://github.com/apache/polaris/pull/1520 -- 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. T

Re: [PR] Add missing "overrides" to realm-override config examples & tests [polaris]

2025-05-03 Thread via GitHub
adutra commented on PR #1511: URL: https://github.com/apache/polaris/pull/1511#issuecomment-2848617373 > @adutra good point on (2) -- I would be okay changing it back to remove `overrides`... to be honest, I still find the post-Quarkus method of specifying a config's value really arcane. We

[PR] fix overlapping menu item on the nav bar [polaris]

2025-05-03 Thread via GitHub
gfakbar20 opened a new pull request, #1520: URL: https://github.com/apache/polaris/pull/1520 When viewed on our mobile phone, the navigation menu component will overlap and making it impossible to navigate the website. current: ![image](https://github.com/user-attachments

Re: [PR] Fix overlapping navigation menu component on mobile view [polaris]

2025-05-03 Thread via GitHub
gfakbar20 closed pull request #1519: Fix overlapping navigation menu component on mobile view URL: https://github.com/apache/polaris/pull/1519 -- 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 spe

[PR] Fix overlapping navigation menu component on mobile view [polaris]

2025-05-03 Thread via GitHub
gfakbar20 opened a new pull request, #1519: URL: https://github.com/apache/polaris/pull/1519 When viewed on our mobile phone, the navigation menu component will overlap and making it impossible to navigate the website. current ![image](https://github.com/user-attachments/asse