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

Small unrelated improvements #897

Merged
merged 5 commits into from
Nov 7, 2023
Merged

Small unrelated improvements #897

merged 5 commits into from
Nov 7, 2023

Conversation

DrDet
Copy link
Contributor

@DrDet DrDet commented Sep 11, 2023

  • don't fetch net events right before script termination (relates to Fix curl resumable epoll related crash #893)
  • added total allocation metrics: kphp_memory_script_allocated_total, kphp_memory_script_allocations_count
  • removed maximum script timeout limit
  • make disable-sql option deprecated and turned on unless SQL port is given

@DrDet DrDet added the runtime Feature related to runtime label Sep 11, 2023
@DrDet DrDet self-assigned this Sep 11, 2023
@DrDet DrDet force-pushed the dvaksman/runtime-improvements branch from efe9ab0 to ae44f7c Compare November 2, 2023 10:30
@DrDet DrDet changed the title WIP: Improvements Small unrelated improvements Nov 2, 2023
@DrDet DrDet added the small fix When it's not an huge enhancement label Nov 2, 2023
@DrDet DrDet added this to the next milestone Nov 2, 2023
@DrDet DrDet force-pushed the dvaksman/runtime-improvements branch from 7ad4e68 to 2d4cb8d Compare November 2, 2023 17:16
@DrDet DrDet merged commit adaba6d into master Nov 7, 2023
5 checks passed
@DrDet DrDet deleted the dvaksman/runtime-improvements branch November 7, 2023 17:25
astrophysik pushed a commit that referenced this pull request Nov 9, 2023
- don't fetch net events right before script termination (relates to #893)
- added total allocation metrics: `kphp_memory_script_allocated_total`, `kphp_memory_script_allocations_count`
- removed maximum script timeout limit
- make `disable-sql` option deprecated and turned on unless SQL port is given
astrophysik pushed a commit that referenced this pull request Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
runtime Feature related to runtime small fix When it's not an huge enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants