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.

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

Experimentally relax the constraints of the exchange executor. #2153

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

boaks
Copy link
Contributor

@boaks boaks commented Jun 28, 2023

Don't fail, when a null executor is passed in.
Still not supported and may fail later, including hard to find race conditions.

Don't fail, when a null executor is passed in.
Still not supported and may fail later, including hard to find race
conditions.

Signed-off-by: Achim Kraus <achim.kraus@cloudcoap.net>
@boaks boaks force-pushed the relax_exchange_execution branch from 4f55df2 to 58b8024 Compare June 28, 2023 07:38
Copy link

@JimmyBaize JimmyBaize left a comment

Choose a reason for hiding this comment

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

Thank you very much. Very thoughtful. So that I can just pass null in executor.
After the PR is merged, I will try to upgrade the version for verification.

@boaks
Copy link
Contributor Author

boaks commented Jun 28, 2023

After the PR is merged, I will try to upgrade the version for verification.

"upgrade the version" may be not that easy. See MIGRATION_HINTS for a list of details.

@boaks boaks merged commit ec7de43 into eclipse-californium:main Jun 28, 2023
@boaks boaks deleted the relax_exchange_execution branch September 26, 2023 13:40
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.

2 participants