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

[JBPM-9817] Adding IDX_TaskEvent_processInstance #1973

Merged
merged 1 commit into from Jul 30, 2021
Merged

[JBPM-9817] Adding IDX_TaskEvent_processInstance #1973

merged 1 commit into from Jul 30, 2021

Conversation

MichalCervenansky
Copy link
Member

@MichalCervenansky MichalCervenansky commented Jul 14, 2021 •
edited

Adding IDX_TaskEvent_processInstance to db schema and the TaskEvent class definition to showcase preventing sequential scan resulting in timeout in JBPM performance tests
JIRA: link

@elguardian
Copy link
Member

Jenkins retest this please

@MichalCervenansky
Copy link
Member Author

Jenkins retest this please

@elguardian
Copy link
Member

good work @MichalCervenansky
Do you mind to do the upgrades scripts for community as well ? I can see ddl, upgrades in pam... but not in jbpm ?

@elguardian elguardian self-requested a review July 29, 2021 10:22
elguardian
elguardian approved these changes Jul 29, 2021
View reviewed changes
Copy link
Member

@elguardian elguardian left a comment

Choose a reason for hiding this comment

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

LGTM. maybe as an ok it should be 7.57-to-7.58

but it is fine as it is. waiting for CI green and merging after that.

@MichalCervenansky MichalCervenansky changed the title WIP: JBPM-9817: Adding IDX_TaskEvent_processInstance in postgresql JBPM-9817: Adding IDX_TaskEvent_processInstance Jul 29, 2021
@MichalCervenansky
Copy link
Member Author

Pushed it once again and renamed the PR

@elguardian elguardian changed the title JBPM-9817: Adding IDX_TaskEvent_processInstance [JBPM-9817] Adding IDX_TaskEvent_processInstance Jul 29, 2021
@sonarcloud
Copy link

sonarcloud bot commented Jul 29, 2021

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
No Duplication information No Duplication information

@elguardian elguardian merged commit 643fb10 into kiegroup:main Jul 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elguardian elguardian elguardian approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@MichalCervenansky @elguardian