Copilot commented on code in PR #708: URL: https://github.com/apache/ranger/pull/708#discussion_r2450405699
########## dev-support/ranger-docker/scripts/ranger-tagsync-install.properties: ########## @@ -25,9 +25,10 @@ TAG_DEST_RANGER_ENDPOINT = http://ranger:6080 # SSL config file name for HTTPS messages to tag destination - Ranger TAG_DEST_RANGER_SSL_CONFIG_FILENAME = Review Comment: [nitpick] Enabling Atlas source by default in the Docker configuration may cause unexpected behavior for existing setups. Consider documenting this change prominently or providing a comment explaining why this default was changed from 'false' to 'true'. ```suggestion # NOTE: The default value for TAG_SOURCE_ATLAS_ENABLED has been changed from 'false' to 'true' in this Docker configuration. # This enables Atlas as a tag source by default, which may cause unexpected behavior for existing setups that do not use Atlas. # Please review and adjust this setting as needed for your environment. ``` -- 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]
