-
Notifications
You must be signed in to change notification settings - Fork 844
TS-4627: support TSRemapOSResponse in ts_lua #779
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
Conversation
|
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/398/ for details. |
|
Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/294/ for details. |
| "TS_LUA_SRVSTATE_CONNECTION_ERROR", "TS_LUA_SRVSTATE_INACTIVE_TIMEOUT", | ||
| "TS_LUA_SRVSTATE_OPEN_RAW_ERROR", "TS_LUA_SRVSTATE_PARSE_ERROR", | ||
| "TS_LUA_SRVSTATE_TRANSACTION_COMPLETE", "TS_LUA_SRVSTATE_CONGEST_CONTROL_CONGESTED_ON_F", | ||
| "TS_LUA_SRVSTATE_CONGEST_CONTROL_CONGESTED_ON_M"}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a ',' before the closing brace to encourage ``clang-format` to format this right.
|
@jpeach , I have done the cleanup according to the comments. please take a look. thanks. |
|
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/436/ for details. |
|
Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/329/ for details. |
| "TS_LUA_SRVSTATE_PARSE_ERROR", | ||
| "TS_LUA_SRVSTATE_TRANSACTION_COMPLETE", | ||
| "TS_LUA_SRVSTATE_CONGEST_CONTROL_CONGESTED_ON_F", | ||
| "TS_LUA_SRVSTATE_CONGEST_CONTROL_CONGESTED_ON_M"}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try putting a ',' after the last element to get clang-format to be sensible.
|
@jpeach cleanup is done accordingly. pls review. |
|
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/446/ for details. |
|
FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/447/ for details. |
|
Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/340/ for details. |
|
Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/341/ for details. |
|
Loods good 👍 |
This cherry-picks two PRs from Vinith to 10.0.x from 9.1.x: * apache#779 * apache#784
…e#779) * Add SO_INCOMING_CPU support to net.sock_option_flag_in (apache#11366) * Add SO_INCOMING_CPU support to net.sock_option_flag_in * Fix FreeBSD Build (cherry picked from commit 266b666) Conflicts: include/iocore/net/NetVCOptions.h iocore/eventsystem/I_EThread.h iocore/net/Connection.cc * Fix building without HWLOC (apache#11435) Fixes apache#11434. (cherry picked from commit 44a41cb) --------- Co-authored-by: Masaori Koshiba <masaori@apache.org> Co-authored-by: JosiahWI <41302989+JosiahWI@users.noreply.github.com>
@jpeach pls take a look. Thanks.
I want to add enhancement to ts_lua so I can write script to take advantage of the TSRemapOSResponse() related functionality.
Basically it would allow me to do something when origin server fails to connect (such as DNS error, timeout, etc)