-
Notifications
You must be signed in to change notification settings - Fork 23
[JBPM-10187] Sorting resources to avoid deadlock #11
Conversation
jenkins do fdb |
...rools-persistence-api/src/main/java/org/drools/persistence/api/TransactionManagerHelper.java
Outdated
Show resolved
Hide resolved
...rsistence/drools-persistence-jpa/src/main/java/org/drools/persistence/PersistableRunner.java
Show resolved
Hide resolved
…ools/persistence/api/TransactionManagerHelper.java Co-authored-by: Gonzalo Muñoz <[email protected]>
There was a problem hiding this 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
so can you suggest proper import then? @gmunozfe |
Im trying to fix the issue, but Im getting this when forcing the push
|
hmm, what is your push command? @fjtirado 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 |
jenkins retest this please |
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 |
jenkins retest this please i recreated org.drools:drools project on sonarcloud.io so we will see if that is enough to pass it |
SonarCloud Quality Gate failed. 0 Bugs 71.9% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
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. |
[JBPM-10187] Sorting resources to avoid deadlock
[JBPM-10187] Sorting resources to avoid deadlock
JIRA:
link
referenced Pull Requests:
kiegroup/jbpm#2318