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

Lps 81028 #58945

Closed
wants to merge 3 commits into from
Closed

Lps 81028 #58945

wants to merge 3 commits into from

Conversation

shuyangzhou
Copy link

No description provided.

… anything when doing this because we release the cache before caching the result since we put into the entity cache first and the ThreadLocal is static in BlockingCache to avoid deadlock conditions. See any generated persistence cacheResult() method. We also haven't been using the correct property in FinderCacheImpl since LPS-56513.
…se the BlockingCache anymore for the finder cache and these values are not being cleared by the persistences.
@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests.

Comment "ci:test" to run the full PR Tester for this pull.

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 1 minute 14 seconds 90 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: ddbe175576f60069b392547f582e523a61693b75

Sender Branch:

Branch Name: LPS-81028
Branch GIT ID: 4b26c84e01717988ae23adbce4686d572a246c15

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 36 out of 49 jobs passed in 1 hour 48 minutes 36 seconds 329 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: ddbe175576f60069b392547f582e523a61693b75

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 421cf6b41ca874678b98a9c05baecd0ae98a0b73

36 out of 49 jobs PASSED

13 Failed Jobs:

36 Successful Jobs:
For more details click here.

Failures unique to this pull:

  1. ...

For upstream results, click here.

@brianchandotcom
Copy link
Owner

Merged. Thx.

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.

4 participants