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

Run used-defined function in provided scheduler rather than in pc-pool thread #794

Conversation

yevheniisemenov
Copy link
Contributor

@yevheniisemenov yevheniisemenov commented Jun 10, 2024

This contribution fixes the issue described here #793

Checklist

  • Documentation (if applicable)
  • Changelog

@yevheniisemenov yevheniisemenov requested a review from a team as a code owner June 10, 2024 11:56
Copy link

cla-assistant bot commented Jun 10, 2024

CLA assistant check
All committers have signed the CLA.

@yevheniisemenov yevheniisemenov changed the title Run used-refined function in provided scheduler rather than in pc-pool thread Run used-defined function in provided scheduler rather than in pc-pool thread Jun 10, 2024
@ajax-semenov-y ajax-semenov-y force-pushed the run-user-defined-react-function-in-provided-scheduler branch from 41d0ca2 to fd5969e Compare June 10, 2024 14:07
@rkolesnev
Copy link
Contributor

/sem-approve

@rkolesnev
Copy link
Contributor

Hi @yevheniisemenov - thanks for the fix.
Can you please do couple of housekeeping updates to the PR

  • update the license header in changed files to 2024 - or alternatively run mvn license:format.
  • update changelog and regenerate Readme - for changelog its Changelog.md in the root.
  • run mvn asciidoc-template:build to incorporate the updated changelog into generated Readme.

Cheers

@rkolesnev
Copy link
Contributor

I've merged this change to a branch and added required Changelog / Readme / license header updates there.
Superseded by #798.

@rkolesnev rkolesnev closed this Jun 11, 2024
@yevheniisemenov
Copy link
Contributor Author

@rkolesnev, thanks for the review!

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