You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@polaris.apache.org by "dimas-b (via GitHub)" <gi...@apache.org> on 2025/11/06 17:28:23 UTC

[PR] fix(docs): Generify S3 index page [polaris]

dimas-b opened a new pull request, #2997:
URL: https://github.com/apache/polaris/pull/2997

   * Remove the mention of "cloud" since not all possible storage options are provided in "cloud".
   
   * Avoid listing specific child pages in the doc test. Rely on Hugo-general index (on the left-hand pane).
   
   <!--
   📝 Describe what changes you're proposing, especially breaking or user-facing changes. 
   📖 See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
   -->
   
   ## Checklist
   - [ ] 🛡️ Don't disclose security issues! (contact security@apache.org)
   - [ ] 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
   - [ ] 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
   - [ ] 💡 Added comments for complex logic
   - [ ] 🧾 Updated `CHANGELOG.md` (if needed)
   - [ ] 📚 Updated documentation in `site/content/in-dev/unreleased` (if needed)
   


-- 
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-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(docs): Generify S3 index page [polaris]

Posted by "dimas-b (via GitHub)" <gi...@apache.org>.

dimas-b merged PR #2997:
URL: https://github.com/apache/polaris/pull/2997


-- 
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-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(docs): Generify S3 index page [polaris]

Posted by "adutra (via GitHub)" <gi...@apache.org>.

adutra commented on code in PR #2997:
URL: https://github.com/apache/polaris/pull/2997#discussion_r2502563997


##########
site/content/in-dev/unreleased/getting-started/creating-a-catalog/s3/_index.md:
##########
@@ -23,16 +23,16 @@ type: docs
 weight: 100
 ---
 
-The following S3 compatible cloud providers can be configured as storage backends for your Polaris catalog:
+Several S3 compatible storage providers can be configured as backends for your Polaris catalog.
+Refer to child pages under this documentation section for some of the possible options. 
 
-- [AWS S3]({{< ref "catalog-aws.md" >}})
-- [MinIO]({{< ref "catalog-minio.md" >}})
-
-For the `polaris catalogs create` [command]({{% ref "../../../command-line-interface#create" %}}) there are few `s3` only options
+For the `polaris catalogs create` [command]({{% ref "../../../command-line-interface#create" %}})
+there are a few s3-only options

Review Comment:
   nit
   ```suggestion
   there are a few s3-only options:
   ```



-- 
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-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org