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

Snapshot: Migrate TransportRequestHandler to TransportMasterNodeAction #27165

Merged
merged 10 commits into from
Nov 17, 2017
Merged

Snapshot: Migrate TransportRequestHandler to TransportMasterNodeAction #27165

merged 10 commits into from
Nov 17, 2017

Commits on Oct 30, 2017

  1. Snapshot: Use TransportMasterNodeAction to update 

    Currently, we are using a plain TransportRequestHandler to post snapshot
status messages to the master. However, it doesn't have a robust retry
mechanism as TransportMasterNodeAction. This changes migrate from
TransportRequestHandler to TransportMasterNodeAction. Most of code in
TransportSnapshotUpdateStatusAction is copied from
SnapshotShardsService.

Serializing a MasterNodeRequest requires 8 bytes more than a
TransportRequest. In order to maintain the BWC in a mixed cluster, we
have to serialize/deserialize a MasterNodeRequest as a TransportRequest
without timeout.

Closes elastic#27151
    @dnhatn
    dnhatn committed Oct 30, 2017
    Configuration menu
    Copy the full SHA
    1f6bf50 View commit details
    Browse the repository at this point in the history

Commits on Nov 5, 2017

  1. Revert "Snapshot: Use TransportMasterNodeAction to update" 

    This reverts commit 1f6bf50.
    @dnhatn
    dnhatn committed Nov 5, 2017
    Configuration menu
    Copy the full SHA
    37b8050 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into snapshot-migrate-transport

    @dnhatn
    dnhatn committed Nov 5, 2017
    Configuration menu
    Copy the full SHA
    82a98ef View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2017

  1. Uses two handlers

    @dnhatn
    dnhatn committed Nov 6, 2017
    Configuration menu
    Copy the full SHA
    97c4724 View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2017

  1. Merge branch 'master' into snapshot-migrate-transport

    @dnhatn
    dnhatn committed Nov 14, 2017
    Configuration menu
    Copy the full SHA
    82aedcc View commit details
    Browse the repository at this point in the history

  2. add a disruption test

    @dnhatn
    dnhatn committed Nov 14, 2017
    Configuration menu
    Copy the full SHA
    06292d1 View commit details
    Browse the repository at this point in the history

  3. post update messages to the local node

    @dnhatn
    dnhatn committed Nov 14, 2017
    Configuration menu
    Copy the full SHA
    ea7ec38 View commit details
    Browse the repository at this point in the history

  4. try posting status forever

    @dnhatn
    dnhatn committed Nov 14, 2017
    Configuration menu
    Copy the full SHA
    cb73eea View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2017

  1. do not use sleep

    @dnhatn
    dnhatn committed Nov 15, 2017
    Configuration menu
    Copy the full SHA
    85da45d View commit details
    Browse the repository at this point in the history

Commits on Nov 17, 2017

  1. Merge branch 'master' into snapshot-migrate-transport

    @dnhatn
    dnhatn committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    57daa0d View commit details
    Browse the repository at this point in the history