sclee01 commented on PR #1905: URL: https://github.com/apache/polaris/pull/1905#issuecomment-2983821577
Thanks for your comment, @snazy . You're absolutely right that `default-realm` is more generic and descriptive. I originally used `default-realm,root,s3cr3t`, but updated it to `POLARIS,root,secret` following reviewer feedback — particularly the point that "POLARIS" is set as the default realm in Polaris configuration. Would you be okay with updating the PR summary to reflect this, rather than reverting the change? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org