-
Notifications
You must be signed in to change notification settings - Fork 17
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
[WIP] PHP SDK integration #485
base: main
Are you sure you want to change the base?
Conversation
…detection and loading
…ry/successful_query feature
…ue and options based on an attribute
…erun RoadRunner; add `query/timeout_due_to_no_active_workers` feature
…el_try_cancel` and `retry_on_error`
… converter after the json proto converter
Looks like we need to register the UPD: |
# Conflicts: # .github/workflows/ci.yaml
# Conflicts: # .github/workflows/ci.yaml
# Conflicts: # cmd/run.go # harness/php/.rr.yaml # harness/php/composer.json # sdkbuild/php.go
About this option:
Adding this test requires adjustments to the framework because RR needs to be restarted with special worker settings. |
Ready for review. |
What was changed
Added runner preparation:
Added runner that runs RoadRunner and starts PHP features in a single run.
Implemented features:
3 / 5
0 / 6
(not supported)3 / 13
0 / 2
1 / 1
7 / 7
0 / 1
(waiting SDK feature)1 / 1
0 / 15
(all cases are empty)5 / 5
0 / 1
(not required)4 / 5
(our cron doesn't support seconds. Should it support seconds?)0 / 2
(all cases are empty)6 / 7
0 / 1
(OTEL package is not released yet)0 / 0
0 / 0
11 / 11
WorkflowStub::getUpdateHandle()
method sdk-php#458Checklist
How to run
Windows: