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

Any interest in using opcache #99

Open
FallingSnow opened this issue Sep 28, 2022 · 3 comments
Open

Any interest in using opcache #99

FallingSnow opened this issue Sep 28, 2022 · 3 comments

Comments

@FallingSnow
Copy link

Would dolibarr run any faster with opcache enabled?

Seems fairly simple to enable.

RUN docker-php-ext-configure opcache --enable-opcache \
    && docker-php-ext-install opcache

Results from https://github.com/cavo789/docker_php_opcache suggest a 30% speed improvement in that benchmark.
@mathieupotier
Copy link
Contributor

Hi, seems interesting.
Quite easy to implement : can you benchmark it and open a pull request ?

Following these guidelines : https://github.com/tuxgasy/docker-dolibarr/blob/master/CONTRIBUTING.md

@FallingSnow
Copy link
Author

Sorry but I don't use dolibarr (though I hope I will in the future). Only came across it in passing and noticed this docker image didn't use opcache. I don't have time to benchmark it at the moment.

@mathieupotier
Copy link
Contributor

I'll probably create a PR soon to add OPCache optims, thanks for suggest.

OdyX added a commit to OdyX/docker-dolibarr that referenced this issue Jan 9, 2023
@OdyX
Add PHP opcache extension
b1b4c5e 
Fixes tuxgasy#99
@OdyX OdyX mentioned this issue Jan 9, 2023
Closed
@mathieupotier mathieupotier mentioned this issue Sep 20, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add PHP opcache extension OdyX/docker-dolibarr
2 participants
@FallingSnow @mathieupotier