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

Add Retryable attribute for FunctionBinding annotation #1036

Draft
wants to merge 13 commits into
base: develop
Choose a base branch
from

Conversation

igdianov
Copy link
Contributor

@igdianov igdianov commented Mar 6, 2023

add FunctionBindingRequestHandlerRetryAdvice class

polish names

add Retryable attribute to FunctionalBinding

polish

Support default retryable configuration for FunctionBinding annotation

@igdianov igdianov self-assigned this Mar 6, 2023
@LorenzoDiGiuseppe LorenzoDiGiuseppe force-pushed the AAE-12537-add-concurrency-test-dev-ldigiuseppe branch from 0fc6df9 to b1f506a Compare March 6, 2023 15:01
@igdianov igdianov force-pushed the AAE-12537-add-concurrency-test-dev-ldigiuseppe branch from b1f506a to 944e6a6 Compare March 6, 2023 17:12
igdianov added 2 commits March 6, 2023 09:13
add FunctionBindingRequestHandlerRetryAdvice class


polish names


add Retryable attribute to FunctionalBinding


polish


Support default retryable configuration for FunctionBinding annotation
@igdianov igdianov force-pushed the AAE-12537-add-concurrency-test-dev-igdianov branch from c346441 to 3db24e2 Compare March 6, 2023 17:14
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@atchertchian
Copy link
Contributor

If this PR shows conflicts, here are instructions to help rebasing this work branch after reformat at 814a919:

git rebase c845f509770e38aa9663a97cb2f9bcd3e5d39e28
git rebase --strategy-option=theirs --exec 'pre-commit run --from-ref HEAD~ --to-ref HEAD' 814a9190ff60af86086e8d111660146569ef6e19
git rebase develop

The last rebase should be omitted if another reformat has happened (similar instructions should be followed for rebase over next reformat).

See more details at Activiti/Activiti#4277.

@LorenzoDiGiuseppe LorenzoDiGiuseppe force-pushed the AAE-12537-add-concurrency-test-dev-ldigiuseppe branch 2 times, most recently from b78918d to 84833cc Compare March 17, 2023 11:55
Base automatically changed from AAE-12537-add-concurrency-test-dev-ldigiuseppe to develop March 17, 2023 13:46
@CLAassistant
Copy link

CLAassistant commented Aug 23, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ igdianov
❌ LorenzoDiGiuseppe
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants