Skip to content

[AMORO-3490] we got sync optimizer groups failed error in ams-error.log#3491

Merged
zhoujinsong merged 1 commit intoapache:masterfrom
Aireed:issue_3490
Mar 28, 2025
Merged

[AMORO-3490] we got sync optimizer groups failed error in ams-error.log#3491
zhoujinsong merged 1 commit intoapache:masterfrom
Aireed:issue_3490

Conversation

@Aireed
Copy link
Contributor

@Aireed Aireed commented Mar 28, 2025

Why are the changes needed?

Close #3490 .

Brief change log

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@Aireed Aireed requested a review from zhoujinsong March 28, 2025 08:10
@github-actions github-actions bot added the module:ams-server Ams server module label Mar 28, 2025
Copy link
Contributor

@zhoujinsong zhoujinsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@zhoujinsong zhoujinsong merged commit f78069d into apache:master Mar 28, 2025
4 checks passed
@Aireed Aireed deleted the issue_3490 branch May 15, 2025 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-server Ams server module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: we got Sync optimizer groups failed

2 participants