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

Reply via email to