weizhouapache commented on PR #13380:
URL: https://github.com/apache/cloudstack/pull/13380#issuecomment-4657294318

   code lgtm
   
   I just wonder if the sql is in correct uprgade file.
   if we release 4.23.0 now, the changes will not apply to 4.23.0 users.
   I think we should avoid sql changes of minor releases, especially when a new 
major version is released.


-- 
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]

Reply via email to