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

Fix activity variable size limit bug #274

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

yinan-symphony
Copy link
Contributor

The activity json string input variable might be too big to be stored in DB (4000 characters limit), e.g. the activity json string contains a variable being resolved at runtime with a more than 4000 characters string value, when camunda persists this new activity json string as variable in DB, it fails with an exception.

This commit fixed this issue by definiting the input activity string in a map instead of string, the map object is stored in bytearray table as BLOB type without limit.

Description

Please put here the intent of your pull request.

Dependencies

List the other pull requests that should be merged before/along this one.

Checklist

  • Referenced a ticket in the PR title or description
  • Filled properly the description and dependencies, if any
  • Unit/Integration tests updated or added
  • Javadoc added or updated
  • Updated the documentation in docs folder

The activity json string input variable might be too big
to be stored in DB (4000 characters limit), e.g. the activity
json string contains a variable being resolved at runtime with
a more than 4000 characters string value, when camunda persists
this new activity json string as variable in DB, it fails with
an exception.

This commit fixed this issue by definiting the input activity
string in a map instead of string, the map object is stored in
bytearray table as BLOB type without limit.
@yinan-symphony yinan-symphony requested a review from ystxn June 5, 2023 08:31
@ystxn ystxn merged commit 60fd1fa into finos:master Jun 5, 2023
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