eric-maynard commented on code in PR #1367:
URL: https://github.com/apache/polaris/pull/1367#discussion_r2046070592
##
getting-started/assets/eclipselink/persistence.xml:
##
@@ -32,8 +32,7 @@
org.apache.polaris.jpa.models.ModelSequenceId
NONE
-
+
Rev
singhpk234 commented on issue #1326:
URL: https://github.com/apache/polaris/issues/1326#issuecomment-2807926029
please ref this thread, if anyone wants to know what realm is :
https://lists.apache.org/thread/kvt3gd29hyxlhnwxr9b674s555jlvjjs
--
This is an automated message from the Apache
MonkeyCanCode commented on issue #1359:
URL: https://github.com/apache/polaris/issues/1359#issuecomment-2807912905
@nssalian i think the right fix is to change the URL to
https://raw.githubusercontent.com/apache/polaris/refs/heads/release/0.9.x/spec/rest-catalog-open-api.yaml
instead as `ic
MonkeyCanCode commented on PR #898:
URL: https://github.com/apache/polaris/pull/898#issuecomment-2807897180
> Another conflict :(
Fixed. Yes, this is due to frequent boto3 upgrade.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
singhpk234 commented on code in PR #1287:
URL: https://github.com/apache/polaris/pull/1287#discussion_r2045727903
##
extension/persistence/relational-jdbc/src/main/java/org/apache/polaris/extension/persistence/relational/jdbc/JdbcBasePersistenceImpl.java:
##
@@ -0,0 +1,619 @@
+/
liamzwbao commented on issue #1272:
URL: https://github.com/apache/polaris/issues/1272#issuecomment-2807760904
The ROOT namespace is added
[here](https://github.com/apache/polaris/blob/b94e90269b83d6091bd461ddd7bc7ff87b237882/polaris-core/src/main/java/org/apache/polaris/core/persistence/res
liamzwbao commented on issue #1273:
URL: https://github.com/apache/polaris/issues/1273#issuecomment-2807717442
cc @collado-mike @eric-maynard @RussellSpitzer for more context. Please take
a look when you got a chance, thank you!
--
This is an automated message from the Apache Git Service.
adnanhemani commented on code in PR #1367:
URL: https://github.com/apache/polaris/pull/1367#discussion_r2045649017
##
getting-started/assets/eclipselink/persistence.xml:
##
@@ -32,8 +32,7 @@
org.apache.polaris.jpa.models.ModelSequenceId
NONE
-
+
Revi
travis-bowen commented on code in PR #4:
URL: https://github.com/apache/polaris-tools/pull/4#discussion_r2045574200
##
polaris-synchronizer/api/src/main/java/org/apache/polaris/tools/sync/polaris/service/IcebergCatalogService.java:
##
@@ -0,0 +1,54 @@
+/*
+ * Licensed to the Apa
dimas-b commented on code in PR #1373:
URL: https://github.com/apache/polaris/pull/1373#discussion_r2045473490
##
quarkus/service/src/main/java/org/apache/polaris/service/quarkus/auth/ActiveRolesAugmentor.java:
##
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache Software Foundati
eric-maynard commented on code in PR #1367:
URL: https://github.com/apache/polaris/pull/1367#discussion_r2045489063
##
getting-started/assets/eclipselink/persistence.xml:
##
@@ -32,8 +32,7 @@
org.apache.polaris.jpa.models.ModelSequenceId
NONE
-
+
Rev
adnanhemani commented on PR #1367:
URL: https://github.com/apache/polaris/pull/1367#issuecomment-2807465452
This is the new structure after changes made to address comments:
 {
eric-maynard commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045147504
##
polaris-core/src/main/java/org/apache/polaris/core/entity/PrincipalRoleEntity.java:
##
@@ -38,19 +38,19 @@ public static PrincipalRoleEntity of(PolarisBaseEntit
eric-maynard closed pull request #1295: Make PolarisConfiguration values
optional
URL: https://github.com/apache/polaris/pull/1295
--
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
eric-maynard commented on code in PR #1367:
URL: https://github.com/apache/polaris/pull/1367#discussion_r2045089664
##
site/content/in-dev/unreleased/getting-started/deploying-polaris/_index.md:
##
Review Comment:
I agree that moving the `Deploying Polaris in Production` gu
collado-mike commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045082533
##
spec/polaris-management-service.yml:
##
@@ -1089,6 +1089,10 @@ components:
clientId:
type: string
description: The output-only OA
collado-mike commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045082533
##
spec/polaris-management-service.yml:
##
@@ -1089,6 +1089,10 @@ components:
clientId:
type: string
description: The output-only OA
collado-mike commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045079760
##
integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisManagementServiceIntegrationTest.java:
##
@@ -871,6 +873,42 @@ public void testCreateP
jbonofre merged PR #1370:
URL: https://github.com/apache/polaris/pull/1370
--
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
collado-mike commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045077628
##
quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/ManagementServiceTest.java:
##
@@ -158,4 +173,150 @@ public void
testUpdateCatalogWithDi
eric-maynard commented on code in PR #1368:
URL: https://github.com/apache/polaris/pull/1368#discussion_r2045059803
##
plugins/spark/v3.5/src/main/java/org/apache/polaris/spark/SparkCatalog.java:
##
@@ -161,33 +163,59 @@ public Table createTable(
@Override
public Table a
collado-mike commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045057068
##
polaris-core/src/main/java/org/apache/polaris/core/entity/PrincipalRoleEntity.java:
##
@@ -65,6 +65,13 @@ public Builder(PrincipalRoleEntity original) {
eric-maynard commented on code in PR #1376:
URL: https://github.com/apache/polaris/pull/1376#discussion_r2045053548
##
quarkus/service/src/main/java/org/apache/polaris/service/quarkus/config/QuarkusProducers.java:
##
@@ -159,9 +163,68 @@ public void maybeBootstrap(
MetaSt
collado-mike commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2045050829
##
polaris-core/src/main/java/org/apache/polaris/core/entity/PrincipalRoleEntity.java:
##
@@ -38,19 +38,19 @@ public static PrincipalRoleEntity of(PolarisBaseEntit
eric-maynard commented on code in PR #1374:
URL: https://github.com/apache/polaris/pull/1374#discussion_r2045044472
##
site/content/in-dev/unreleased/entities.md:
##
@@ -53,6 +53,10 @@ Polaris tables are entities that map to [Apache Iceberg
tables](https://iceberg.
For infor
eric-maynard commented on code in PR #1374:
URL: https://github.com/apache/polaris/pull/1374#discussion_r2045043970
##
site/content/in-dev/unreleased/entities.md:
##
@@ -53,6 +53,10 @@ Polaris tables are entities that map to [Apache Iceberg
tables](https://iceberg.
For infor
eric-maynard commented on code in PR #1374:
URL: https://github.com/apache/polaris/pull/1374#discussion_r2045043077
##
site/content/in-dev/unreleased/access-control.md:
##
@@ -124,6 +124,19 @@ To grant the full set of privileges (drop, list, read,
write, etc.) on an object
| T
wolflex888 commented on code in PR #1164:
URL: https://github.com/apache/polaris/pull/1164#discussion_r2045040810
##
service/common/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalogAdapter.java:
##
@@ -398,9 +398,18 @@ public Response loadTable(
dennishuo commented on code in PR #1305:
URL: https://github.com/apache/polaris/pull/1305#discussion_r2043654728
##
polaris-core/src/main/java/org/apache/polaris/core/entity/CatalogEntity.java:
##
@@ -190,6 +202,21 @@ public Catalog.TypeEnum getCatalogType() {
.orElse(n
eric-maynard commented on code in PR #4:
URL: https://github.com/apache/polaris-tools/pull/4#discussion_r2042599636
##
polaris-synchronizer/README.md:
##
@@ -0,0 +1,162 @@
+# Objective
+
+To provide users of [Apache Polaris
(Incubating)](https://github.com/apache/polaris) a too
dimas-b commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2043032833
##
service/common/src/main/java/org/apache/polaris/service/admin/PolarisAdminService.java:
##
@@ -771,6 +771,9 @@ public PrincipalWithCredentials
createPrincipal(Polar
eric-maynard commented on code in PR #4:
URL: https://github.com/apache/polaris-tools/pull/4#discussion_r2042606826
##
polaris-synchronizer/api/src/main/java/org/apache/polaris/tools/sync/polaris/PolarisSynchronizer.java:
##
@@ -0,0 +1,1135 @@
+/*
+ * Licensed to the Apache Soft
snazy merged PR #1229:
URL: https://github.com/apache/polaris/pull/1229
--
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.apach
dennishuo commented on code in PR #1305:
URL: https://github.com/apache/polaris/pull/1305#discussion_r2041544641
##
polaris-core/src/main/java/org/apache/polaris/core/secrets/UserSecretReference.java:
##
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
fivetran-arjuncnair closed pull request #1369: Adding fivetran-cd for CICD
URL: https://github.com/apache/polaris/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 u
adutra commented on PR #1367:
URL: https://github.com/apache/polaris/pull/1367#issuecomment-2804833229
@adnanhemani thanks for this proposal. Looking at the screenshot you shared,
I notice that we now have 4 subsections named "Deploying Polaris on..." but
also one separate section named "De
nssalian commented on issue #564:
URL: https://github.com/apache/polaris/issues/564#issuecomment-2806754290
Thanks @flyrain. I'll put in a fix. I think the `gcpCredentialsSupplier`
could be improved as well.
--
This is an automated message from the Apache Git Service.
To respond to the me
snazy opened a new pull request, #1376:
URL: https://github.com/apache/polaris/pull/1376
Log explicit messages around auto-bootstrapping and unnecessary/left-over
secrets that are (still) available.
--
This is an automated message from the Apache Git Service.
To respond to the
jbonofre merged PR #1372:
URL: https://github.com/apache/polaris/pull/1372
--
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
snazy merged PR #1375:
URL: https://github.com/apache/polaris/pull/1375
--
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.apach
singhpk234 commented on code in PR #1287:
URL: https://github.com/apache/polaris/pull/1287#discussion_r2043169115
##
extension/persistence/relational-jdbc/src/main/java/org/apache/polaris/extension/persistence/relational/jdbc/DatasourceOperations.java:
##
@@ -0,0 +1,178 @@
+/*
+
adutra commented on issue #1345:
URL: https://github.com/apache/polaris/issues/1345#issuecomment-2802472657
FYI I am working on a fix for this.
--
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
singhpk234 commented on code in PR #1287:
URL: https://github.com/apache/polaris/pull/1287#discussion_r2043239193
##
extension/persistence/relational-jdbc/src/main/java/org/apache/polaris/extension/persistence/relational/jdbc/JdbcCrudQueryGenerator.java:
##
@@ -0,0 +1,241 @@
+/*
gh-yzou commented on code in PR #1368:
URL: https://github.com/apache/polaris/pull/1368#discussion_r2043282845
##
plugins/spark/v3.5/src/main/java/org/apache/polaris/spark/PolarisRESTCatalog.java:
##
@@ -138,12 +152,51 @@ public void close() throws IOException {
@Override
wolflex888 commented on code in PR #1164:
URL: https://github.com/apache/polaris/pull/1164#discussion_r2044987142
##
service/common/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalogAdapter.java:
##
@@ -398,9 +398,18 @@ public Response loadTable(
eric-maynard commented on code in PR #1368:
URL: https://github.com/apache/polaris/pull/1368#discussion_r2043195402
##
plugins/spark/v3.5/src/main/java/org/apache/polaris/spark/PolarisRESTCatalog.java:
##
@@ -138,12 +152,51 @@ public void close() throws IOException {
@Overr
eric-maynard commented on code in PR #1356:
URL: https://github.com/apache/polaris/pull/1356#discussion_r2042581926
##
polaris-core/src/main/java/org/apache/polaris/core/persistence/cache/EntityCache.java:
##
@@ -76,15 +81,22 @@ public EntityCache(@Nonnull PolarisMetaStoreManage
snazy commented on code in PR #1292:
URL: https://github.com/apache/polaris/pull/1292#discussion_r2042214542
##
integration-tests/src/main/resources/META-INF/LICENSE:
##
@@ -0,0 +1,213 @@
+ Apache License
+ Version 2.0, J
wolflex888 commented on code in PR #1164:
URL: https://github.com/apache/polaris/pull/1164#discussion_r2044987142
##
service/common/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalogAdapter.java:
##
@@ -398,9 +398,18 @@ public Response loadTable(
fivetran-ashokborra commented on issue #480:
URL: https://github.com/apache/polaris/issues/480#issuecomment-2806283561
@jbonofre @MonkeyCanCode As per the TODO comment
[here](https://github.com/apache/polaris/blob/main/polaris-core/src/main/java/org/apache/polaris/core/storage/aws/AwsCredent
renovate-bot opened a new pull request, #1375:
URL: https://github.com/apache/polaris/pull/1375
This PR contains the following updates:
| Package | Update | Change |
|---|---|---|
| docker.io/prom/prometheus | minor | `v3.2.1` -> `v3.3.0` |
---
### Configuration
gfakbar20 opened a new pull request, #1374:
URL: https://github.com/apache/polaris/pull/1374
This pr is made [to update access control
documentation](https://github.com/apache/polaris/issues/1324). Based on sync
meeting notes, There are new implementations related to generic tables.
-
jbonofre merged PR #1292:
URL: https://github.com/apache/polaris/pull/1292
--
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
renovate-bot opened a new pull request, #1372:
URL: https://github.com/apache/polaris/pull/1372
This PR contains the following updates:
| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[io.projectreactor.netty:reactor-netty-http](https://r
fivetran-edknight commented on issue #480:
URL: https://github.com/apache/polaris/issues/480#issuecomment-2804418285
Did the suggestion solve the issue?
--
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
snazy commented on PR #898:
URL: https://github.com/apache/polaris/pull/898#issuecomment-2804598378
Another conflict :(
--
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 u
gh-yzou opened a new pull request, #1368:
URL: https://github.com/apache/polaris/pull/1368
Add support for following table operations:
1) listTables
2) renameTables
3) purgeTables
4) alterTable
5) invalidate table
Integration test will come in follow up PRs
--
Thi
singhpk234 opened a new pull request, #1371:
URL: https://github.com/apache/polaris/pull/1371
### About the change
This is part 3 of Bigger change https://github.com/apache/polaris/issues/649
since part 2 is yet to be merged, and based on decision on part2 on how we
wanna model
63 matches
Mail list logo
