snazy commented on code in PR #2131:
URL: https://github.com/apache/polaris/pull/2131#discussion_r2215052076
##
persistence/nosql/idgen/impl/src/main/resources/META-INF/beans.xml:
##
@@ -0,0 +1,24 @@
+
+
+https://jakarta.ee/xml/ns/jakartaee";
+ xmlns:xsi="http://www.w3.org/2001
snazy commented on code in PR #2131:
URL: https://github.com/apache/polaris/pull/2131#discussion_r2215036515
##
persistence/nosql/idgen/api/src/main/java/org/apache/polaris/ids/api/MonotonicClock.java:
##
@@ -0,0 +1,69 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
snazy commented on code in PR #2127:
URL: https://github.com/apache/polaris/pull/2127#discussion_r2215022981
##
client/python/cli/command/catalogs.py:
##
@@ -64,6 +64,9 @@ class CatalogsCommand(Command):
remove_properties: List[str]
hadoop_warehouse: str
iceberg_r
snazy merged PR #2130:
URL: https://github.com/apache/polaris/pull/2130
--
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
snazy merged PR #2128:
URL: https://github.com/apache/polaris/pull/2128
--
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
snazy merged PR #2129:
URL: https://github.com/apache/polaris/pull/2129
--
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
jbonofre commented on issue #586:
URL: https://github.com/apache/polaris/issues/586#issuecomment-3086767715
I worked on a proposal about this. I will share the proposal on the mailing
list very soon.
--
This is an automated message from the Apache Git Service.
To respond to the message, p
dimas-b commented on code in PR #2131:
URL: https://github.com/apache/polaris/pull/2131#discussion_r2214750693
##
persistence/nosql/idgen/api/src/main/java/org/apache/polaris/ids/api/SnowflakeIdGenerator.java:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Foundatio
adam-christian-software opened a new pull request, #2131:
URL: https://github.com/apache/polaris/pull/2131
# Motivation
This implementation starts the series of PRs to implement the NoSQL work
presented in
https://docs.google.com/document/d/1POUWe0xMZOBoaJ6Rgiw35ziEoc6OEYCiW7Zk6bR9H6M/ed
dimas-b merged PR #2126:
URL: https://github.com/apache/polaris/pull/2126
--
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 #2126:
URL: https://github.com/apache/polaris/pull/2126#issuecomment-3086374431
Re: S3 tests, I'm planning to run then in CI on MinIO (once the backend is
fully ready)
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
renovate-bot opened a new pull request, #2130:
URL: https://github.com/apache/polaris/pull/2130
This PR contains the following updates:
| Package | Change | Age | Confidence |
|---|---|---|---|
| [io.smallrye.config:smallrye-config-core](https://smallrye.io)
([source](https://re
renovate-bot opened a new pull request, #2129:
URL: https://github.com/apache/polaris/pull/2129
This PR contains the following updates:
| Package | Change | Age | Confidence |
|---|---|---|---|
| [boto3](https://redirect.github.com/boto/boto3) | `==1.39.7` -> `==1.39.8`
|
[
has merged? At least it shouldn't be a 1.0 blocker now.
@snazy should we close this one as now t
gh-yzou commented on PR #2122:
URL: https://github.com/apache/polaris/pull/2122#issuecomment-3085108696
@dimas-b Thanks! Could you help take a look somtime?
--
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
eric-maynard commented on code in PR #2127:
URL: https://github.com/apache/polaris/pull/2127#discussion_r2213874271
##
client/python/cli/command/catalogs.py:
##
@@ -64,6 +64,9 @@ class CatalogsCommand(Command):
remove_properties: List[str]
hadoop_warehouse: str
ic
MonkeyCanCode commented on PR #2049:
URL: https://github.com/apache/polaris/pull/2049#issuecomment-3084803394
@eric-maynard / @HonahX may I have a review here? This will get us a proper
packaging for polaris client as well as to help us to remove dependency on the
bash script (and use it on
eric-maynard commented on code in PR #2127:
URL: https://github.com/apache/polaris/pull/2127#discussion_r2213866090
##
client/python/cli/command/catalogs.py:
##
@@ -64,6 +64,9 @@ class CatalogsCommand(Command):
remove_properties: List[str]
hadoop_warehouse: str
ic
MonkeyCanCode commented on code in PR #2104:
URL: https://github.com/apache/polaris/pull/2104#discussion_r2213222792
##
helm/polaris/README.md:
##
@@ -189,21 +189,24 @@ ct install --namespace polaris --charts ./helm/polaris
|-|--|-|-|
| advancedConf
MonkeyCanCode commented on code in PR #2127:
URL: https://github.com/apache/polaris/pull/2127#discussion_r2213848168
##
client/python/cli/command/catalogs.py:
##
@@ -64,6 +64,9 @@ class CatalogsCommand(Command):
remove_properties: List[str]
hadoop_warehouse: str
i
singhpk234 commented on PR #2048:
URL: https://github.com/apache/polaris/pull/2048#issuecomment-3084708025
@snazy @adutra thank you for sharing your feedbacks, i want to walk you all
through what my thought process was :
Why Iceberg Expressions and not SQL
Iceberg expresions are p
renovate-bot opened a new pull request, #2128:
URL: https://github.com/apache/polaris/pull/2128
This PR contains the following updates:
| Package | Change | Age | Confidence |
|---|---|---|---|
| io.quarkus | `3.24.3` -> `3.24.4` |
[![age](https://developer.mend.io/api/mc/badges
eric-maynard merged PR #2121:
URL: https://github.com/apache/polaris/pull/2121
--
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
eric-maynard commented on code in PR #2127:
URL: https://github.com/apache/polaris/pull/2127#discussion_r2213791278
##
client/python/cli/constants.py:
##
@@ -222,6 +225,11 @@ class Create:
EXTERNAL_ID = "(Only for S3) The external ID to use when
connecting to S3"
eric-maynard commented on code in PR #2127:
URL: https://github.com/apache/polaris/pull/2127#discussion_r2213790497
##
client/python/cli/command/catalogs.py:
##
@@ -64,6 +64,9 @@ class CatalogsCommand(Command):
remove_properties: List[str]
hadoop_warehouse: str
ic
eric-maynard commented on code in PR #1965:
URL: https://github.com/apache/polaris/pull/1965#discussion_r2213781651
##
service/common/src/main/java/org/apache/polaris/service/events/AwsCloudWatchEventListener.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Fou
eric-maynard commented on code in PR #1965:
URL: https://github.com/apache/polaris/pull/1965#discussion_r2213779987
##
service/common/src/main/java/org/apache/polaris/service/events/AwsCloudWatchEventListener.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Fou
eric-maynard commented on code in PR #1965:
URL: https://github.com/apache/polaris/pull/1965#discussion_r2213775761
##
service/common/src/main/java/org/apache/polaris/service/events/AwsCloudWatchEventListener.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Fou
eric-maynard commented on code in PR #1938:
URL: https://github.com/apache/polaris/pull/1938#discussion_r2213771840
##
polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageTokenUtil.java:
##
@@ -0,0 +1,267 @@
+/*
+ * Licensed to the Apache Software Found
adutra commented on PR #2048:
URL: https://github.com/apache/polaris/pull/2048#issuecomment-3084574973
> Row-level filter expressions benefit a lot from CASE, CAST, COALESCE etc.
Not sure whether it's worth to not have those available to users.
Indeed, the ability to express something
MonkeyCanCode commented on code in PR #2104:
URL: https://github.com/apache/polaris/pull/2104#discussion_r2213222792
##
helm/polaris/README.md:
##
@@ -189,21 +189,24 @@ ct install --namespace polaris --charts ./helm/polaris
|-|--|-|-|
| advancedConf
snazy opened a new pull request, #2127:
URL: https://github.com/apache/polaris/pull/2127
Amends #1913 and #2012
--
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
snazy merged PR #2106:
URL: https://github.com/apache/polaris/pull/2106
--
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
snazy commented on code in PR #1965:
URL: https://github.com/apache/polaris/pull/1965#discussion_r2212830643
##
service/common/src/main/java/org/apache/polaris/service/events/AwsCloudWatchEventListener.java:
##
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache Software Foundation
snazy commented on code in PR #2048:
URL: https://github.com/apache/polaris/pull/2048#discussion_r2212550907
##
polaris-core/src/main/java/org/apache/polaris/core/policy/content/IcebergExpressionListDeserializer.java:
##
@@ -0,0 +1,49 @@
+/*
+ * Licensed to the Apache Software F
adutra commented on code in PR #2104:
URL: https://github.com/apache/polaris/pull/2104#discussion_r2212609682
##
helm/polaris/tests/configmap_test.yaml:
##
@@ -336,3 +427,33 @@ tests:
asserts:
- matchRegex: { path: 'data["application.properties"]', pattern:
"polaris
adutra commented on code in PR #2104:
URL: https://github.com/apache/polaris/pull/2104#discussion_r2212610906
##
helm/polaris/README.md:
##
@@ -189,21 +189,24 @@ ct install --namespace polaris --charts ./helm/polaris
|-|--|-|-|
| advancedConfig | ob
adutra commented on issue #1327:
URL: https://github.com/apache/polaris/issues/1327#issuecomment-3083040643
This has been partially addressed by #2013, although that PR does not cover
how to setup the IDP for Polaris (users, groups, roles, etc.) or how to
integrate with common IDPs (Okta, A
snazy commented on PR #2048:
URL: https://github.com/apache/polaris/pull/2048#issuecomment-3083016697
(Replying to the later editor PR description)
> Note: In last Apache Iceberg Community Sync all were generally alligned
that using iceberg expression with extending its support for refere
adutra commented on code in PR #2104:
URL: https://github.com/apache/polaris/pull/2104#discussion_r2212553207
##
helm/polaris/README.md:
##
@@ -189,21 +189,24 @@ ct install --namespace polaris --charts ./helm/polaris
|-|--|-|-|
| advancedConfig | ob
snazy commented on code in PR #1904:
URL: https://github.com/apache/polaris/pull/1904#discussion_r2212355993
##
service/common/src/main/java/org/apache/polaris/service/events/AfterCatalogGetEvent.java:
##
@@ -0,0 +1,25 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
43 matches
Mail list logo
