Skip to content

Conversation

@vickenty
Copy link
Contributor

@vickenty vickenty commented Dec 21, 2023

What does this PR do?

Update payload size when updating socket path

Description of the Change

When initialize sets socket_path, thus switching from udp to uds transport, update max_payload_size to use payloads of optimal size for the transport.

Alternate Designs

Possible Drawbacks

Verification Process

Additional Notes

Release Notes

Review checklist (to be filled by reviewers)

  • Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

@vickenty vickenty added the changelog/Fixed Fixed features results into a bug fix version bump label Jan 4, 2024
@vickenty vickenty marked this pull request as ready for review January 4, 2024 11:56
@vickenty vickenty requested a review from a team as a code owner January 4, 2024 11:56
@vickenty vickenty requested a review from a team January 4, 2024 11:56
@vickenty vickenty merged commit 1e2cfcf into master Jan 4, 2024
@vickenty vickenty deleted the vickenty/bufsz branch January 4, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/Fixed Fixed features results into a bug fix version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants