flyrain commented on PR #168:
URL: https://github.com/apache/polaris/pull/168#issuecomment-2302748683
cc @collado-mike @dennishuo @aihuaxu
--
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
snazy opened a new pull request, #173:
URL: https://github.com/apache/polaris/pull/173
(no comment)
--
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:
snazy commented on PR #173:
URL: https://github.com/apache/polaris/pull/173#issuecomment-2302862635
This is based on #170
/cc @jbonofre
--
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
jbonofre commented on PR #173:
URL: https://github.com/apache/polaris/pull/173#issuecomment-2302867850
Cool. Let me take a look. Thanks !
--
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 speci
snazy commented on PR #141:
URL: https://github.com/apache/polaris/pull/141#issuecomment-2302880391
Hi @bbrewington,
do you have any idea why [this CI job
fails](https://github.com/apache/polaris/actions/runs/10496260045/job/29076506165)?
--
This is an automated message from the Ap
flyrain commented on PR #141:
URL: https://github.com/apache/polaris/pull/141#issuecomment-2302885581
```
ERROR: 4 dead links found!
[✖] https://polaris.apache.org/ → Status: 404
[✖]
https://polaris.apache.org/index.html#tag/polaris-management-service_other →
Status: 404
[✖] htt
annafil commented on code in PR #169:
URL: https://github.com/apache/polaris/pull/169#discussion_r1725714732
##
.github/CODEOWNERS:
##
@@ -14,4 +14,4 @@
# limitations under the License.
#
-* @polaris-catalog/polaris
+* @jbonofre @anoopj @RussellSpitzer @adutra @laurentgo @sn
jbonofre commented on PR #169:
URL: https://github.com/apache/polaris/pull/169#issuecomment-2302974740
@annafil GitHub says that `CODEOWNERS` contains error. Do you have the same
message in your GitHub ?
--
This is an automated message from the Apache Git Service.
To respond to the messag
jbonofre closed pull request #170: [WIP] Update resources for ASF landing
URL: https://github.com/apache/polaris/pull/170
--
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 uns
jbonofre commented on PR #174:
URL: https://github.com/apache/polaris/pull/174#issuecomment-2302988010
@snazy do you mind to review ? I would like to merge at least `.asf.yml`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
jbonofre opened a new pull request, #174:
URL: https://github.com/apache/polaris/pull/174
(no comment)
--
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-mai
jbonofre commented on PR #170:
URL: https://github.com/apache/polaris/pull/170#issuecomment-2302987298
Superseeded by https://github.com/apache/polaris/pull/174
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
rdblue commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1725777124
##
.asf.yml:
##
@@ -0,0 +1,57 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distri
jbonofre commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1725778283
##
.asf.yml:
##
@@ -0,0 +1,57 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# dist
flyrain commented on PR #168:
URL: https://github.com/apache/polaris/pull/168#issuecomment-2303064851
cc @snazy @RussellSpitzer
--
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 comme
rdblue commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1725799896
##
NOTICE:
##
@@ -1,5 +1,5 @@
-Polaris
-Copyright 2024 Snowflake Computing Inc.
+Apache Polaris (incubating)
+Copyright 2024 The Apache Software Foundation
Review Comment
flyrain commented on PR #169:
URL: https://github.com/apache/polaris/pull/169#issuecomment-2303085891
I can see the same error message, but the format looks look to me. Here is
the doc about the code owner format.
--
This is an automated message from the Apache Git Service.
To respond to
jbonofre commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1725803755
##
NOTICE:
##
@@ -1,5 +1,5 @@
-Polaris
-Copyright 2024 Snowflake Computing Inc.
+Apache Polaris (incubating)
+Copyright 2024 The Apache Software Foundation
Review Comme
findepi commented on PR #162:
URL: https://github.com/apache/polaris/pull/162#issuecomment-2303088664
thank you all!
--
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 unsu
flyrain commented on PR #169:
URL: https://github.com/apache/polaris/pull/169#issuecomment-2303087066
```
Unknown owner on line 17: make sure @jbonofre exists and has write access to
the repository
* @jbonofre @ashvina @anoopj @RussellSpitzer @snazy @vvcephei @hivivo
@takidau @jackye1
jbonofre commented on PR #174:
URL: https://github.com/apache/polaris/pull/174#issuecomment-2303089057
@rdblue that's right, I talked with @takidau.
--
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 t
annafil commented on PR #169:
URL: https://github.com/apache/polaris/pull/169#issuecomment-2303090204
I think this is happening because GitHub checks that these folks have
permissions at the repo level. @jbonofre how long does the LDAP stuff take to
propagate? To resolve this error, each of
jbonofre commented on PR #169:
URL: https://github.com/apache/polaris/pull/169#issuecomment-2303096674
@annafil it should be done already. Let me check.
--
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
findepi commented on PR #159:
URL: https://github.com/apache/polaris/pull/159#issuecomment-2303098867
Rebased to resolve conflicts
--
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
mayankvadariya commented on issue #156:
URL: https://github.com/apache/polaris/issues/156#issuecomment-2303111725
Possibly `BaseMetastoreCatalog#registerTable` requires overridden in
`BasePolarisCatalog` to have `BasePolarisCatalog#refreshIOWithCredentials` or
`refreshCredentials` considere
MonkeyCanCode commented on issue #144:
URL: https://github.com/apache/polaris/issues/144#issuecomment-2303253984
@sfc-gh-adlee are you still seeing this issue? I am able to run those
locally. If the issue is reproducible, I can debug to see if there are any
diffs.
--
This is an automate
sfc-gh-adlee commented on issue #144:
URL: https://github.com/apache/polaris/issues/144#issuecomment-2303266755
Hello Apache/Polaris
1) If I use the default realm principal client ID and client secret, i
don't hit the 500 error. I was able to go until the step "create or replace
justinmclean opened a new pull request, #175:
URL: https://github.com/apache/polaris/pull/175
# Description
Fix name and trademark as per ASF policy
Fixes # N/A
## Type of change
Please delete options that are not relevant.
- [X] This change requires a docum
annafil commented on PR #169:
URL: https://github.com/apache/polaris/pull/169#issuecomment-2303302134
It might also help to re-run the checks on the last commit, it might trigger
checking the codeowners file for validity
--
This is an automated message from the Apache Git Service.
To resp
MonkeyCanCode commented on issue #144:
URL: https://github.com/apache/polaris/issues/144#issuecomment-2303508717
> Hello Apache/Polaris 1) If I use the default realm principal client ID and
client secret, i don't hit the 500 error. I was able to go until the step
"create or replace table" I
cgpoh commented on PR #135:
URL: https://github.com/apache/polaris/pull/135#issuecomment-2303527560
Thanks @MonkeyCanCode, LGTM!
--
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 comme
MonkeyCanCode commented on PR #135:
URL: https://github.com/apache/polaris/pull/135#issuecomment-2303537821
> Thanks @MonkeyCanCode, LGTM!
Thanks for review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
dennishuo commented on code in PR #77:
URL: https://github.com/apache/polaris/pull/77#discussion_r1726146291
##
polaris-core/src/main/java/io/polaris/core/storage/azure/AzureCredentialsStorageIntegration.java:
##
@@ -88,6 +89,40 @@ public EnumMap
getSubscopedCreds(
String
kevinjqliu commented on code in PR #141:
URL: https://github.com/apache/polaris/pull/141#discussion_r1726148135
##
.github/workflows/check-md-link.yml:
##
@@ -0,0 +1,43 @@
+#
+# Copyright (c) 2024 Snowflake Computing Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the
kevinjqliu commented on PR #141:
URL: https://github.com/apache/polaris/pull/141#issuecomment-2303544487
Those links are not accessible to me currently.
Apache Polaris website is currently under
http://polaris.incubator.apache.org/ but that's also not accessible
See https://project
kevinjqliu commented on issue #139:
URL: https://github.com/apache/polaris/issues/139#issuecomment-2303545491
As a follow up, enable this github action for all MD files and for all PRs
See comment:
https://github.com/apache/polaris/pull/141#discussion_r1726148135
--
This is an automate
dennishuo commented on issue #146:
URL: https://github.com/apache/polaris/issues/146#issuecomment-2303565592
This was definitely an unfortunate oversight; good catch!
Luckily, since the bug wasn't that the request handlers actually expected to
parse `true` from the String, just that t
MonkeyCanCode commented on PR #176:
URL: https://github.com/apache/polaris/pull/176#issuecomment-2303616282
@flyrain @eric-maynard current main is not working due to this...this will
fix the spark demo.
--
This is an automated message from the Apache Git Service.
To respond to the message
MonkeyCanCode opened a new pull request, #176:
URL: https://github.com/apache/polaris/pull/176
# Description
For `docker-compose-jupyter.yml`, this is the demo for how to use interact
with Iceberg with Spark when using Polaris as catalog and Jupyter notebook as
UI. Currently it is se
MonkeyCanCode commented on issue #144:
URL: https://github.com/apache/polaris/issues/144#issuecomment-2303630952
@sfc-gh-adlee So what I refers to as working was
https://github.com/apache/polaris/blob/main/docker-compose-jupyter.yml
However, I did found one issue there with the demo c
flyrain commented on code in PR #176:
URL: https://github.com/apache/polaris/pull/176#discussion_r1726362667
##
docker-compose-jupyter.yml:
##
@@ -44,6 +44,8 @@ services:
condition: service_healthy
environment:
AWS_REGION: us-west-2
+ AWS_ACCESS_KEY_ID:
flyrain commented on code in PR #176:
URL: https://github.com/apache/polaris/pull/176#discussion_r1726362667
##
docker-compose-jupyter.yml:
##
@@ -44,6 +44,8 @@ services:
condition: service_healthy
environment:
AWS_REGION: us-west-2
+ AWS_ACCESS_KEY_ID:
MonkeyCanCode commented on code in PR #176:
URL: https://github.com/apache/polaris/pull/176#discussion_r1726367401
##
docker-compose-jupyter.yml:
##
@@ -44,6 +44,8 @@ services:
condition: service_healthy
environment:
AWS_REGION: us-west-2
+ AWS_ACCESS_K
flyrain commented on code in PR #176:
URL: https://github.com/apache/polaris/pull/176#discussion_r1726372139
##
docker-compose-jupyter.yml:
##
@@ -44,6 +44,8 @@ services:
condition: service_healthy
environment:
AWS_REGION: us-west-2
+ AWS_ACCESS_KEY_ID:
MonkeyCanCode commented on code in PR #176:
URL: https://github.com/apache/polaris/pull/176#discussion_r1726382824
##
docker-compose-jupyter.yml:
##
@@ -44,6 +44,8 @@ services:
condition: service_healthy
environment:
AWS_REGION: us-west-2
+ AWS_ACCESS_K
snazy commented on PR #141:
URL: https://github.com/apache/polaris/pull/141#issuecomment-2303866694
Is there a way to let the tool only print errors?
--
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
MonkeyCanCode commented on code in PR #176:
URL: https://github.com/apache/polaris/pull/176#discussion_r1726410980
##
docker-compose-jupyter.yml:
##
@@ -44,6 +44,8 @@ services:
condition: service_healthy
environment:
AWS_REGION: us-west-2
+ AWS_ACCESS_K
MonkeyCanCode closed pull request #176: Fix spark demo
URL: https://github.com/apache/polaris/pull/176
--
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: d
snazy commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1726410756
##
.asf.yml:
##
@@ -0,0 +1,57 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distrib
snazy commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1726419763
##
README.md:
##
@@ -93,8 +87,14 @@ curl -i -X POST -H "Authorization: Bearer $PRINCIPAL_TOKEN"
-H 'Accept: applicat
```
From here, you can use Spark to create namespa
jbonofre commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1726424682
##
README.md:
##
@@ -1,51 +1,45 @@
-
+# Apache Polaris (incubating)
-Polaris
snazy commented on PR #177:
URL: https://github.com/apache/polaris/pull/177#issuecomment-2303898903
/cc @jbonofre
--
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 unsubs
snazy opened a new pull request, #177:
URL: https://github.com/apache/polaris/pull/177
Based on #174, with the PR comments applied.
--
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 co
jbonofre commented on PR #177:
URL: https://github.com/apache/polaris/pull/177#issuecomment-2303900799
Why creating a new one whereas we have the other PR ? I'm creating the
mailing lists btw.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
snazy opened a new pull request, #178:
URL: https://github.com/apache/polaris/pull/178
The project web site is not up yet, so disabling link checking for it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
snazy opened a new pull request, #179:
URL: https://github.com/apache/polaris/pull/179
(no comment)
--
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:
snazy commented on PR #177:
URL: https://github.com/apache/polaris/pull/177#issuecomment-2303909952
I don't mind using your PR - just that there's more stuff in there
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
jbonofre commented on PR #175:
URL: https://github.com/apache/polaris/pull/175#issuecomment-2303931771
@justinmclean I'm not sure Polaris trademark has been registered already. Do
we need it now ? I think we register for graduation right ?
--
This is an automated message from the Apache G
snazy opened a new pull request, #180:
URL: https://github.com/apache/polaris/pull/180
(no comment)
--
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:
snazy commented on PR #181:
URL: https://github.com/apache/polaris/pull/181#issuecomment-2303992161
/cc @kevinjqliu
--
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 unsub
justinmclean commented on PR #175:
URL: https://github.com/apache/polaris/pull/175#issuecomment-2303993465
That's why I used the TM symbol rather than the R symbol, as it is not,
AFAIK, a registered trademark.
--
This is an automated message from the Apache Git Service.
To respond to the
snazy opened a new pull request, #181:
URL: https://github.com/apache/polaris/pull/181
The `Check Markdown links` CI job currently lists all checked links, even
successful checks. Only listing errors makes it easier to find mistakes.
--
This is an automated message from the Apache Git
justinmclean commented on PR #175:
URL: https://github.com/apache/polaris/pull/175#issuecomment-2303998997
Basically a TM says it's the ASF's brand with no need to register it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
snazy commented on PR #180:
URL: https://github.com/apache/polaris/pull/180#issuecomment-2304001030
> It should be rebase on the "new" `README.md` when merged, but good to me.
Definitely - and I want to add some "textual links" once all that's in.
--
This is an automated message fro
jbonofre commented on PR #175:
URL: https://github.com/apache/polaris/pull/175#issuecomment-2304003626
@justinmclean ok, thanks for clarifying. I mixed TM and R symbols.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
snazy commented on PR #175:
URL: https://github.com/apache/polaris/pull/175#issuecomment-2304008788
> Basically a TM says it's the ASF's brand with no need to register it.
Agree, AFAIK yes, `TM` says it's a trademark, just unregistered.
--
This is an automated message from the Apach
jbonofre commented on PR #177:
URL: https://github.com/apache/polaris/pull/177#issuecomment-2304016980
@snazy let me remove `.asf.yml` from my other PR and use yours. We need to
wait the mailing lists created (request submitted) before merging (ok, we can't
merge anything for now 😄 ).
--
jbonofre commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1726542951
##
.asf.yml:
##
@@ -0,0 +1,57 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# dist
jbonofre commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1726543741
##
.asf.yml:
##
@@ -0,0 +1,57 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# dist
jbonofre commented on code in PR #174:
URL: https://github.com/apache/polaris/pull/174#discussion_r1726544718
##
.asf.yml:
##
@@ -0,0 +1,57 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# dist
annafil opened a new pull request, #1:
URL: https://github.com/apache/polaris-site/pull/1
The sidebar was being generated in a random order. This PR creates a fixed
list that is consistent with the current docs UX on polaris.io.
I also noticed that Docusaurus didn't like the license
jbonofre merged PR #1:
URL: https://github.com/apache/polaris-site/pull/1
--
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: dev-unsubscr...@polaris.apache
72 matches
Mail list logo
