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

Fix #16411, added ORM SessionCache AKA FirstLevelCache #16450

Open

rudiservo wants to merge 1 commit into phalcon:5.0.x from rudiservo:i16411

Contributor

rudiservo commented Oct 2, 2023

Hello!

Type: bug fix | new feature | code quality
Link to issue: [NFR]: ORM: Model Thread Cache, AKA First Level Cache or L1 cache #16411

In raising this pull request, I confirm the following:

I have read and understood the Contributing Guidelines
I have checked that another pull request for this purpose does not exist
I wrote some tests for this PR
I have updated the relevant CHANGELOG
I have created a PR for the documentation about this change

Small description of change:

Thanks

rudiservo force-pushed the i16411 branch 2 times, most recently from 7e21423 to ddbcbe2 Compare

October 2, 2023 12:18

Contributor Author

rudiservo commented Oct 2, 2023

SqLite tests fail, the tests are fixed in one of the PR I previously made, I will rebase this once they are approved.
feel free to check and comment.


          Fix phalcon#16411, added ORM SessionCache AKA FirstLevelCache

4ee909f

rudiservo force-pushed the i16411 branch from ddbcbe2 to 4ee909f Compare

October 9, 2023 12:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment