eric-maynard commented on code in PR #2227:
URL: https://github.com/apache/polaris/pull/2227#discussion_r2249138722
##
site/content/in-dev/unreleased/getting-started/minio.md:
##
@@ -0,0 +1,114 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
eric-maynard commented on code in PR #2227:
URL: https://github.com/apache/polaris/pull/2227#discussion_r2249138416
##
site/content/in-dev/unreleased/getting-started/minio.md:
##
@@ -0,0 +1,114 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
eric-maynard commented on code in PR #2227:
URL: https://github.com/apache/polaris/pull/2227#discussion_r2249138576
##
site/content/in-dev/unreleased/getting-started/minio.md:
##
@@ -0,0 +1,114 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
eric-maynard commented on code in PR #2227:
URL: https://github.com/apache/polaris/pull/2227#discussion_r2249138161
##
site/content/in-dev/unreleased/getting-started/minio.md:
##
@@ -0,0 +1,114 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
eric-maynard commented on code in PR #2227:
URL: https://github.com/apache/polaris/pull/2227#discussion_r2249137947
##
site/content/in-dev/unreleased/getting-started/minio.md:
##
@@ -0,0 +1,114 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more c
eric-maynard commented on code in PR #2149:
URL: https://github.com/apache/polaris/pull/2149#discussion_r2249133209
##
polaris-core/src/main/java/org/apache/polaris/core/storage/StorageUtil.java:
##
@@ -62,4 +68,50 @@ public class StorageUtil {
public static @Nonnull String g
gh-yzou commented on code in PR #2239:
URL: https://github.com/apache/polaris/pull/2239#discussion_r2249031204
##
plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/SparkCatalog.java:
##
@@ -182,6 +188,9 @@ public Table alterTable(Identifier ident, TableChange...
c
gh-yzou commented on code in PR #2239:
URL: https://github.com/apache/polaris/pull/2239#discussion_r2249026117
##
plugins/spark/README.md:
##
@@ -83,6 +86,9 @@ bin/spark-shell \
--conf spark.sql.catalog.polaris.credential="root:secret" \
--conf spark.sql.catalog.polaris.scope=
flyrain commented on code in PR #2239:
URL: https://github.com/apache/polaris/pull/2239#discussion_r2249024219
##
plugins/spark/README.md:
##
@@ -60,6 +60,9 @@ bin/spark-shell \
--conf spark.sql.catalog..credential="root:secret" \
--conf spark.sql.catalog..scope='PRINCIPAL_ROL
flyrain commented on code in PR #2239:
URL: https://github.com/apache/polaris/pull/2239#discussion_r2249023400
##
plugins/spark/README.md:
##
@@ -123,4 +132,4 @@ Following describes the current functionality limitations
of the Polaris Spark c
2) Create a Delta table without ex
dimas-b commented on issue #2207:
URL: https://github.com/apache/polaris/issues/2207#issuecomment-3146010146
I can look into this in +1 week (unless someone else picks it up)... I'll
mark this issue for 1.1.0 proactively, but inclusion into the release is not
guaranteed :)
--
This is an
dimas-b closed issue #2207: Malformed subscoped credentials error with external
s3
URL: https://github.com/apache/polaris/issues/2207
--
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 com
dimas-b commented on PR #2213:
URL: https://github.com/apache/polaris/pull/2213#issuecomment-3146007013
@flyrain : feel free to revert
--
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
flyrain commented on PR #2213:
URL: https://github.com/apache/polaris/pull/2213#issuecomment-3146005631
As a spec change, we will need to conclude the dev ML discussion or vote
before merging. I don't think there is a consensus in the discussion thread,
https://lists.apache.org/thread/9nggn
dimas-b commented on code in PR #2236:
URL: https://github.com/apache/polaris/pull/2236#discussion_r2249009104
##
polaris-core/src/main/java/org/apache/polaris/core/storage/aws/AwsStorageConfigurationInfo.java:
##
@@ -18,95 +18,37 @@
*/
package org.apache.polaris.core.storage
renovate-bot opened a new pull request, #2240:
URL: https://github.com/apache/polaris/pull/2240
This PR contains the following updates:
| Package | Change | Age | Confidence |
|---|---|---|---|
| [boto3](https://redirect.github.com/boto/boto3) | `==1.40.0` -> `==1.40.1`
|
[![ag
jackye1995 opened a new pull request, #2239:
URL: https://github.com/apache/polaris/pull/2239
Integrate Lance format with the generic tables API and allow it to be used
in the Spark connector
--
This is an automated message from the Apache Git Service.
To respond to the messag
snazy commented on code in PR #2149:
URL: https://github.com/apache/polaris/pull/2149#discussion_r2248651812
##
polaris-core/src/main/java/org/apache/polaris/core/storage/StorageUtil.java:
##
@@ -62,4 +68,50 @@ public class StorageUtil {
public static @Nonnull String getBucke
snazy commented on code in PR #2149:
URL: https://github.com/apache/polaris/pull/2149#discussion_r2248651812
##
polaris-core/src/main/java/org/apache/polaris/core/storage/StorageUtil.java:
##
@@ -62,4 +68,50 @@ public class StorageUtil {
public static @Nonnull String getBucke
forking-renovate[bot] commented on PR #2226:
URL: https://github.com/apache/polaris/pull/2226#issuecomment-3145457441
### Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not
recognize the last commit author and assumes somebody else may have
snazy merged PR #2218:
URL: https://github.com/apache/polaris/pull/2218
--
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 #2219:
URL: https://github.com/apache/polaris/pull/2219#discussion_r2248582752
##
persistence/relational-jdbc/src/main/java/org/apache/polaris/persistence/relational/jdbc/JdbcBasePersistenceImpl.java:
##
@@ -183,11 +183,17 @@ private void persi
singhpk234 commented on code in PR #2219:
URL: https://github.com/apache/polaris/pull/2219#discussion_r2248582752
##
persistence/relational-jdbc/src/main/java/org/apache/polaris/persistence/relational/jdbc/JdbcBasePersistenceImpl.java:
##
@@ -183,11 +183,17 @@ private void persi
creepysta opened a new issue, #2238:
URL: https://github.com/apache/polaris/issues/2238
### Describe the bug
I'm trying to configure polaris 1.0.0 to work with external Identity
Provider following - https://github.com/apache/polaris/pull/1397
Looks like when bootstrap, `java -j
singhpk234 commented on PR #2048:
URL: https://github.com/apache/polaris/pull/2048#issuecomment-3145299844
Could you clarify which of the points above are directly related to this pr?
Or is this a general objection to any work on FGAC until the above questions
have all been resolved?
eric-maynard commented on PR #1904:
URL: https://github.com/apache/polaris/pull/1904#issuecomment-3145168186
@snazy are there still changes you'd like to see here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
snazy commented on code in PR #2236:
URL: https://github.com/apache/polaris/pull/2236#discussion_r2248415949
##
polaris-core/src/main/java/org/apache/polaris/core/storage/aws/AwsStorageConfigurationInfo.java:
##
@@ -18,95 +18,37 @@
*/
package org.apache.polaris.core.storage.a
snazy commented on code in PR #2237:
URL: https://github.com/apache/polaris/pull/2237#discussion_r2248381123
##
api/management-model/build.gradle.kts:
##
@@ -50,13 +50,13 @@ val generatedOpenApiSrcDir =
project.layout.buildDirectory.dir("generated-openap
openApiGenerate {
/
snazy opened a new pull request, #2237:
URL: https://github.com/apache/polaris/pull/2237
* `shadowPub.kt` the change removes a special case that doesn't apply to
Polaris
* `api/...` build scripts - changes due to nullable type handling
(`Property` vs `Property` - latter is ... weird)
snazy commented on PR #2226:
URL: https://github.com/apache/polaris/pull/2226#issuecomment-3145093271
Quarkus issue https://github.com/quarkusio/quarkus/issues/49294
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
snazy commented on PR #2226:
URL: https://github.com/apache/polaris/pull/2226#issuecomment-3145072307
Oops...
```
Caused by: org.gradle.api.InvalidUserCodeException: Cannot mutate the
hierarchy of configuration ':polaris-core:compilePlusRuntime' after the
configuration was consume
metadaddy commented on issue #32:
URL: https://github.com/apache/polaris/issues/32#issuecomment-3145017911
@dimas-b No, it's purely a cloud-based service, like S3. You can sign up for
an account, with 10 GB of free storage, no credit card required, or I can
create a test bucket and provide
dimas-b commented on issue #32:
URL: https://github.com/apache/polaris/issues/32#issuecomment-3144996855
@metadaddy : Does B2 have an OSS version runnable in Docker (for testing)?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
metadaddy commented on issue #32:
URL: https://github.com/apache/polaris/issues/32#issuecomment-3144991873
Hi @adutra - I work at Backblaze; our [B2 cloud object
store](https://www.backblaze.com/cloud-storage) is S3-compatible, but we do not
currently have an STS, so I'm keen to move this a
dimas-b commented on code in PR #2236:
URL: https://github.com/apache/polaris/pull/2236#discussion_r2248282283
##
polaris-core/src/main/java/org/apache/polaris/core/storage/aws/AwsStorageConfigurationInfo.java:
##
@@ -18,95 +18,37 @@
*/
package org.apache.polaris.core.storage
snazy commented on code in PR #2235:
URL: https://github.com/apache/polaris/pull/2235#discussion_r2248277534
##
service/common/src/main/java/org/apache/polaris/service/storage/PolarisStorageIntegrationProviderImpl.java:
##
@@ -81,27 +85,32 @@ public PolarisStorageIntegrationProv
snazy opened a new pull request, #2236:
URL: https://github.com/apache/polaris/pull/2236
This change eventually enables usage of the `*StorageConfigurationInfo` in
the `StorageCredentialCacheKey` due to the then memoized hash-code values, to
eliminate a couple of JSON re-serializations.
-
dimas-b commented on code in PR #2235:
URL: https://github.com/apache/polaris/pull/2235#discussion_r2248262082
##
service/common/src/main/java/org/apache/polaris/service/storage/PolarisStorageIntegrationProviderImpl.java:
##
@@ -81,27 +85,32 @@ public PolarisStorageIntegrationPr
dimas-b commented on code in PR #2229:
URL: https://github.com/apache/polaris/pull/2229#discussion_r2248180699
##
polaris-core/src/main/java/org/apache/polaris/core/persistence/BaseMetaStoreManager.java:
##
@@ -53,6 +56,16 @@ public static PolarisStorageConfigurationInfo
extrac
snazy commented on code in PR #2229:
URL: https://github.com/apache/polaris/pull/2229#discussion_r2248149172
##
polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/PolarisTestMetaStoreManager.java:
##
@@ -2903,9 +2900,10 @@ void testPolicyMappingCleanup() {
dimas-b commented on code in PR #2219:
URL: https://github.com/apache/polaris/pull/2219#discussion_r2248146061
##
persistence/relational-jdbc/src/main/java/org/apache/polaris/persistence/relational/jdbc/JdbcBasePersistenceImpl.java:
##
@@ -183,11 +183,17 @@ private void persistE
snazy opened a new pull request, #2235:
URL: https://github.com/apache/polaris/pull/2235
Instances of `PolarisStorageIntegration` are created for a particular
`PolarisStorageConfigurationInfo`, the same value is then passed into
`PSI.getSubscopedCreds()`.
This change removes the conf
XN137 commented on PR #2210:
URL: https://github.com/apache/polaris/pull/2210#issuecomment-3144642258
rebased again after trivial conflict in `AbstractIcebergCatalogViewTest.java`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
adutra commented on PR #2233:
URL: https://github.com/apache/polaris/pull/2233#issuecomment-3144571544
The failing test does not fail to me locally:
`testCreateGcpCredentialsFromStaticToken` – I am investigating.
--
This is an automated message from the Apache Git Service.
To respond to t
dimas-b merged PR #2216:
URL: https://github.com/apache/polaris/pull/2216
--
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.apa
dimas-b merged PR #2213:
URL: https://github.com/apache/polaris/pull/2213
--
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.apa
dimas-b commented on PR #2227:
URL: https://github.com/apache/polaris/pull/2227#issuecomment-3144544264
A docker-compose file would be nice, but I wanted to have a prominent doc
for people building something from scratch. I've seen a few messages where the
catalog creation step appeared to
snazy opened a new pull request, #2234:
URL: https://github.com/apache/polaris/pull/2234
All `PolarisStorageIntegration` requite only the `RealmConfig`, not the
whole `CallContext`. This makes it easier for the new tasks impleemntations
(both proposals).
--
This is an automated message f
adutra commented on issue #32:
URL: https://github.com/apache/polaris/issues/32#issuecomment-3144394618
FYI I'm going to take a stab at request signing.
--
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
adutra opened a new pull request, #2233:
URL: https://github.com/apache/polaris/pull/2233
As discussed in the ML here is a PR that merges polaris-service-common into
polaris-runtime-service.
99% of the changes are file moves or renamings:
- d15c8265a113358696a57c2d873bf97b008e0
lastranget commented on issue #2207:
URL: https://github.com/apache/polaris/issues/2207#issuecomment-3144303412
Yeah, that would be great. We can provide local credentials to our clients.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
adutra commented on PR #2196:
URL: https://github.com/apache/polaris/pull/2196#issuecomment-3144136407
@sclee01 I went ahead and started the thread:
https://lists.apache.org/thread/6c7lzgy3675jrrtbrrxsm8l0xd1fo1mj
Please chime in! 🙏
--
This is an automated message from the Apache
snazy commented on code in PR #2219:
URL: https://github.com/apache/polaris/pull/2219#discussion_r2247639330
##
persistence/relational-jdbc/src/main/java/org/apache/polaris/persistence/relational/jdbc/JdbcBasePersistenceImpl.java:
##
@@ -183,11 +183,17 @@ private void persistEnt
snazy commented on code in PR #2213:
URL: https://github.com/apache/polaris/pull/2213#discussion_r2247621486
##
spec/polaris-management-service.yml:
##
@@ -1050,12 +1050,23 @@ components:
example: "us-east-2"
endpoint:
type: string
-
snazy merged PR #2203:
URL: https://github.com/apache/polaris/pull/2203
--
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
adutra merged PR #2231:
URL: https://github.com/apache/polaris/pull/2231
--
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.apac
adutra merged PR #2232:
URL: https://github.com/apache/polaris/pull/2232
--
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.apac
adutra commented on PR #2227:
URL: https://github.com/apache/polaris/pull/2227#issuecomment-3143937666
Nice addition, although I'm wondering: wouldn't it be better to create a
docker-compose getting-started example in the directory below?
https://github.com/apache/polaris/tree/main/g
adutra opened a new pull request, #2232:
URL: https://github.com/apache/polaris/pull/2232
--
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: issue
XN137 commented on issue #463:
URL: https://github.com/apache/polaris/issues/463#issuecomment-3143925166
https://github.com/apache/polaris/pull/2203 removes the thread local
`CallContext.CURRENT_CONTEXT`
which is probably biggest offender in the codebase as
`PolarisEclipseLinkMetaSto
adutra opened a new pull request, #2231:
URL: https://github.com/apache/polaris/pull/2231
--
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: issue
snazy merged PR #2224:
URL: https://github.com/apache/polaris/pull/2224
--
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
XN137 commented on code in PR #2230:
URL: https://github.com/apache/polaris/pull/2230#discussion_r2247439127
##
runtime/service/src/test/java/org/apache/polaris/service/quarkus/admin/ManagementServiceTest.java:
##
@@ -302,34 +300,19 @@ public void
testCatalogNotReturnedWhenDele
XN137 opened a new pull request, #2230:
URL: https://github.com/apache/polaris/pull/2230
this takes less code and is more flexible in the future
--
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 th
XN137 commented on PR #2214:
URL: https://github.com/apache/polaris/pull/2214#issuecomment-3143837912
closing as it will become obsolete anyway with other ongoing PRs most likely
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
XN137 closed pull request #2214: Call
MetaStoreManagerFactory.getOrCreateSession once in QuarkusProducers
URL: https://github.com/apache/polaris/pull/2214
--
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 g
XN137 opened a new pull request, #2229:
URL: https://github.com/apache/polaris/pull/2229
unfinished WIP
--
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-ma
pingtimeout commented on PR #2156:
URL: https://github.com/apache/polaris/pull/2156#issuecomment-3143651894
I updated the code to include the current changes being discussed in the
Helm PR and to integrate code review feedback. I am going to update the PR
description with output example so
pingtimeout commented on code in PR #2156:
URL: https://github.com/apache/polaris/pull/2156#discussion_r2247271426
##
releasey/README.md:
##
@@ -0,0 +1,131 @@
+
+
+# Polaris Release Automation Scripts
+
+This directory contains scripts to automate the Apache Polaris release
pro
pingtimeout commented on code in PR #2156:
URL: https://github.com/apache/polaris/pull/2156#discussion_r2247269683
##
releasey/04-build-and-test.sh:
##
@@ -0,0 +1,118 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
pingtimeout commented on code in PR #2156:
URL: https://github.com/apache/polaris/pull/2156#discussion_r2247268142
##
releasey/libs/_files.sh:
##
@@ -0,0 +1,50 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreeme
eric-maynard opened a new pull request, #2228:
URL: https://github.com/apache/polaris/pull/2228
After #2049, some users reported errors when running the CLI:
```
./polaris
./polaris: line 71: exec: polaris: not found
```
I was able to reproduce this and found that reverting
72 matches
Mail list logo
