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.

Sign up for GitHub

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

Jump to bottom

Respect UDP maximum packet size when sending batched points #185

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Respect UDP maximum packet size when sending batched points #185

wants to merge 14 commits into from

Commits on Mar 21, 2023

  1. Resolve UDP endpoint using the non-deprecated method and catch except… 

    …ion for unsuccessful call
    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    7d7ce20 View commit details
    Browse the repository at this point in the history

  2. Add unit test for UDP transport

    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    1cf6890 View commit details
    Browse the repository at this point in the history

  3. Catch invalid URI port and protocol values in UriParser, add unit-test

    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    8838ee0 View commit details
    Browse the repository at this point in the history

  4. Make it impossible to construct the UDP or TCP transports without spe… 

    …cifying a valid port number
    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    371a58d View commit details
    Browse the repository at this point in the history

  5. Add system tests for UDP transport

    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    9a07eab View commit details
    Browse the repository at this point in the history

  6. Allow Transport implementations to specify the maximum message size t… 

    …hey can transmit using their send method
    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    865c8af View commit details
    Browse the repository at this point in the history

  7. Respect the transport's maximum message size when transmitting point(s)

    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    72585fc View commit details
    Browse the repository at this point in the history

  8. Refactor the TransmitBatch function to make the logic easier to under… 

    …stand
    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    5f741d0 View commit details
    Browse the repository at this point in the history

  9. When transmitting a point batch, skip any untransmittable points and … 

    …raise an exception when all other points have been sent. Also, ensure that the connection's point batch is definitely cleared even if there is a transmission error.
    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    238b6a0 View commit details
    Browse the repository at this point in the history

  10. Extend the wait within systemtest for Influx to flush UDP data to mea… 

    …surements
    @simonaldrich
    simonaldrich committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    bfb35ae View commit details
    Browse the repository at this point in the history

Commits on Mar 22, 2023

  1. Apply clang-format, fix CI build errors under various compilers

    @simonaldrich
    simonaldrich committed Mar 22, 2023
    Configuration menu
    Copy the full SHA
    071bb61 View commit details
    Browse the repository at this point in the history

  2. Make the UDP max message size the minimum of the socket's send buffer… 

    … size or the UDP theoretical limit (for MacOS)
    @simonaldrich
    simonaldrich committed Mar 22, 2023
    Configuration menu
    Copy the full SHA
    306e05d View commit details
    Browse the repository at this point in the history

  3. Wrap all calls to std::numeric_limits::(min/max) in parentheses to pr… 

    …event unwanted macro expansion in MSVC

see https://stackoverflow.com/questions/27442885/syntax-error-with-stdnumeric-limitsmax
    @simonaldrich
    simonaldrich committed Mar 22, 2023
    Configuration menu
    Copy the full SHA
    598988d View commit details
    Browse the repository at this point in the history

  4. Always perform an "apt update" before an "apt install" and allow 3 re… 

    …try attempts at installing packages

This is a workaround for intermittent 404 errors for the Azure apt mirror
    @simonaldrich
    simonaldrich committed Mar 22, 2023
    Configuration menu
    Copy the full SHA
    312e383 View commit details
    Browse the repository at this point in the history