klion26 commented on PR #3678: URL: https://github.com/apache/amoro/pull/3678#issuecomment-3134557019
@MarigWeizhi Thanks for the verification. Seems the result is as expected? 1. If the property was not set, then Amoro will use the default parameter, and the value was not displayed in the ui is as expected, as the user hasn't set it, but maybe we can log the value we used in the log or somewhere 2. If a user sets the property, then Amoro will use the user-set property I prefer to handle the configuration in a single place, so that we can maintain it easily. Please let me know what you think about this? thanks. -- 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]
