-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
merge from main into dev/4.3.x #2207
Conversation
merge from dev/4.2.x into main
* adaption for obcloud * set account name when async database * do not save password if user chooses not to remember the password * add ut test * patch ODC 413bp10 bug fix:adaption for OBInstanceType * modfiy ConnectionPasswordCache expiration policy * response to reivew * response to review * delete ConnectionStatus.NOPASSWORD * fix ut test
…#2158) * fix failed to mirgrate parti data * fix parti plan migrate failed
* add close after cancel job * return true * fix log when pod deleted * yaobin_taskframework_fix_cancel_log * fix log forward * try to get log from localhost * fix destory executor * format code * format code * fix for pr comment * fix for pr comment * fix log
… memory (#2174) * add node memory * format code * add unit in comment * fix for pr comment * fix for pr comment
…ons (#2184) * fix failed to listTables when lack mysql schema permissions * upgrade db-browser version
… ticket rejection (#2185) * only enqueue nonnull event label * only enqueue nonnull event label * remove transactional annotation
* add some log * add some log * add some log * add some log * add some log
* fix security * rm
…2198) * add Configuration Consumer for UserConfigService * fix autowired ConnectionAdapter bean
merge from dev/4.2.x into main
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What type of PR is this?
type-builds
What this PR does / why we need it:
merge from main into dev/4.3.x
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Additional documentation e.g., usage docs, etc.: