pingtimeout commented on code in PR #1606: URL: https://github.com/apache/polaris/pull/1606#discussion_r2095063180
########## site/content/in-dev/unreleased/configuring-polaris-for-production.md: ########## @@ -209,13 +207,11 @@ curl -X POST http://localhost:8181/api/catalog/v1/oauth/tokens \ -d "scope=PRINCIPAL_ROLE:ALL" ``` -## Other Configurations - -When deploying Polaris in production, consider adjusting the following configurations: - -#### `polaris.features."SUPPORTED_CATALOG_STORAGE_TYPES"` - -- By default, Polaris catalogs are allowed to be located in local filesystem with the `FILE` storage - type. This should be disabled for production systems. -- Use this configuration to additionally disable any other storage types that will not be in use. +### Disable FILE Storage Type +By default, Polaris allows using the local file system (`FILE`) for catalog storage. This is fine for testing, Review Comment: ~Same, given that #1566 has been merged, this sentence is false, isn't it?~ EDIT: my bad, 1566 has not been merged yet. Let's wait for a couple of hours/days until either of #1532 and #1566 is merged. Both these PRs will remove the need for this doc section, and they are needed for 1.0. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
