Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

[JBPM-10187] Sorting resources to avoid deadlock #11

Merged
merged 5 commits into from
Nov 6, 2023
Merged

Conversation

fjtirado
Copy link
Contributor

@fjtirado fjtirado commented Oct 26, 2023

JIRA:

link

referenced Pull Requests:
kiegroup/jbpm#2318

@fjtirado fjtirado requested a review from gmunozfe October 26, 2023 15:08
@fjtirado fjtirado changed the title Jbpm 10187 [JBPM-10187] Sorting resources to avoid deadlock Oct 26, 2023
@fjtirado fjtirado requested a review from mareknovotny October 26, 2023 15:08
@fjtirado fjtirado added backport-7.67.x Generate backport PR for 7.67.x branch backport-7.67.x-blue Generate backport PR for 7.67.x-blue branch labels Oct 26, 2023
@mareknovotny
Copy link
Member

jenkins do fdb

…ools/persistence/api/TransactionManagerHelper.java

Co-authored-by: Gonzalo Muñoz <[email protected]>
@fjtirado fjtirado requested a review from gmunozfe October 27, 2023 13:56
Copy link
Member

@gmunozfe gmunozfe left a comment

Choose a reason for hiding this comment

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

@fjtirado Sorry, I didn't include the import in my comment and compilation fails:
16:03:42 [ERROR] /home/jenkins/workspace/KIE/7.x/pullrequest/drools-7.x.pr/bc/kiegroup_drools/drools-persistence/drools-persistence-api/src/main/java/org/drools/persistence/api/TransactionManagerHelper.java:[64,61] cannot find symbol
16:03:42 symbol: variable Comparator
16:03:42 location: class org.drools.persistence.api.TransactionManagerHelper

@mareknovotny
Copy link
Member

so can you suggest proper import then? @gmunozfe

@fjtirado
Copy link
Contributor Author

fjtirado commented Nov 2, 2023

so can you suggest proper import then? @gmunozfe

@mareknovotny @gmunozfe

Im trying to fix the issue, but Im getting this when forcing the push


Repository https://github.com/kiegroup/drools.git

pre-receive hook declined

@mareknovotny
Copy link
Member

mareknovotny commented Nov 2, 2023

hmm, what is your push command? @fjtirado
I would expect as you pushed this branch before it should work for you, but i will check if by any chance this branch is not by protected pattern guarded

UPDATE: no, this branch is not guarded by protected branch pattern

@fjtirado
Copy link
Contributor Author

fjtirado commented Nov 2, 2023

hmm, what is your push command? @fjtirado I would expect as you pushed this branch before it should work for you, but i will check if by any chance this branch is not by protected pattern guarded

UPDATE: no, this branch is not guarded by protected branch pattern

I edited directly in the PR, I was messing up with branches from my IDE, sorry about that

@mareknovotny
Copy link
Member

jenkins retest this please

@mareknovotny
Copy link
Member

there is some weird sonarcloud analysis error not related to this change, i am not able to reproduce it locally yet. My local build fails on h2 persistence module due Caused by: org.h2.jdbc.JdbcSQLException: Feature not supported: "-ifNotExists" [50100-197]
but there were removed org.drools:drools project in Sonarcloud due Apache migration of repository, but now it should be recreated for 7.x branch. But this is not clear as not fixed. I am trying to sort this as it affect all Drools PR here in this repository

@mareknovotny
Copy link
Member

jenkins retest this please

i recreated org.drools:drools project on sonarcloud.io so we will see if that is enough to pass it

Copy link

sonarqubecloud bot commented Nov 3, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

71.9% 71.9% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@fjtirado fjtirado merged commit 934f572 into 7.x Nov 6, 2023
github-actions bot pushed a commit that referenced this pull request Nov 6, 2023
[JBPM-10187] Sorting resources to avoid deadlock
github-actions bot pushed a commit that referenced this pull request Nov 6, 2023
[JBPM-10187] Sorting resources to avoid deadlock
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-7.67.x Generate backport PR for 7.67.x branch backport-7.67.x-blue Generate backport PR for 7.67.x-blue branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants