Skip to content

fix: Run MessageBus command request processing in goroutines#5186

Merged
judehung merged 1 commit intoedgexfoundry:mainfrom
FelixTing:issue-5152
Jun 18, 2025
Merged

fix: Run MessageBus command request processing in goroutines#5186
judehung merged 1 commit intoedgexfoundry:mainfrom
FelixTing:issue-5152

Conversation

@FelixTing
Copy link
Member

@FelixTing FelixTing commented Jun 17, 2025

fix: #5152

PR Checklist

Please check if your PR fulfills the following requirements:

  • I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
  • I am not introducing a new dependency (add notes below if you are)
  • I have added unit tests for the new feature or bug fix (if not, why?)
  • I have fully tested (add details below) this the new feature or bug fix (if not, why?)
  • I have opened a PR for the related docs change (if not, why?)

Testing Instructions

  1. Deploy core services, device-modbus, and modbus-simulator.
  2. Add a Modbus device to core-metadata.
  3. Stop modbus-simulator.
  4. While the simulator is offline, asynchronously send 100 command requests.
  5. Restart modbus-simulator.
  6. Verify that core-command service is still able to handle command requests once the simulator is back online.

Signed-off-by: FelixTing <felix@iotechsys.com>
@sonarqubecloud
Copy link

@FelixTing FelixTing requested review from judehung June 17, 2025 11:00
Copy link
Member

@judehung judehung left a comment

Choose a reason for hiding this comment

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

LGTM

@judehung judehung merged commit 82c539f into edgexfoundry:main Jun 18, 2025
7 checks passed
@FelixTing FelixTing deleted the issue-5152 branch June 18, 2025 04:48
FelixTing added a commit to FelixTing/edgex-go that referenced this pull request Oct 15, 2025
FelixTing added a commit to FelixTing/edgex-go that referenced this pull request Oct 15, 2025
FelixTing added a commit to FelixTing/edgex-go that referenced this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Core Command Blocks On Concurrent Readings

2 participants