Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Eliminate Dispatch Semaphore in MQTT internal code #4210

Closed
brennanMKE opened this issue Jul 8, 2022 · 1 comment
Closed

Eliminate Dispatch Semaphore in MQTT internal code #4210

brennanMKE opened this issue Jul 8, 2022 · 1 comment
Assignees
@brennanMKE
Labels
concurrency Issues with concurrency problems iot Issues related to the IoT SDK

Comments

@brennanMKE
Copy link
Contributor

Which AWS Services is the feature request for?

AWSIoT

Is your feature request related to a problem? Please describe.

Dispatch Semaphore is being used in 3 places to protect state in stored properties. Serial behavior can be achieved with a Dispatch Queue instead which won't experience some of the problems which have been caused by the use of semaphores.

Describe the solution you'd like

In every block of code where serial behavior is necessary, a block for a Dispatch Queue will be used.

Related

#2016
@brennanMKE brennanMKE added the iot Issues related to the IoT SDK label Jul 8, 2022
@brennanMKE brennanMKE self-assigned this Jul 8, 2022
@brennanMKE brennanMKE mentioned this issue Jul 8, 2022
Merged
6 tasks
@brennanMKE brennanMKE added the concurrency Issues with concurrency problems label Jul 15, 2022
@brennanMKE
Copy link
Contributor Author

PR is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brennanMKE brennanMKE

Labels
concurrency Issues with concurrency problems iot Issues related to the IoT SDK
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@brennanMKE