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

xds: Rename parseCluster() back to processCluster() for consistency #8802

Merged
merged 1 commit into from
Jan 6, 2022

Commits on Jan 6, 2022

  1. xds: Rename parseCluster() back to processCluster() for consistency

    This is to keep names of the top-level process* functions called from
    handle*Response functions, and returning *Update resources consistent:
    
    - `handleLdsResponse()` -> `LdsUpdate processClientSideListener()`
                               `LdsUpdate processServerSideListener()`
    - `handleCdsResponse()` -> `CdsUpdate processCluster()`
    - `handleRdsResponse()` -> `RdsUpdate processRouteConfiguration()`
    - `handleEdsResponse()` -> `EdsUpdate processClusterLoadAssignment()`
    
    For some reason, processCluster() was renamed to parseCluster() in
    fa4b980.
    sergiitk committed Jan 6, 2022
    Configuration menu
    Copy the full SHA
    21f2acf View commit details
    Browse the repository at this point in the history