Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

@DaveCTurner DaveCTurner commented Sep 26, 2018

Mainly this fixes a warning by replacing the unchecked new ActionRunnable
with the checked new ActionRunnable<Response>, and it also fixes the line
length violations in this class.

Mainly this fixes a warning by replacing the unchecked `new ActionListener`
with the checked `new ActionListener<Response>`, and it also fixes the line
length violations in this class.
@DaveCTurner DaveCTurner added >non-issue :Core/Infra/Core Core issues without another label v7.0.0 v6.5.0 labels Sep 26, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

Copy link
Member

@javanna javanna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DaveCTurner
Copy link
Contributor Author

@elasticmachine test this please

@DaveCTurner DaveCTurner merged commit 2aff005 into elastic:master Oct 2, 2018
@DaveCTurner DaveCTurner deleted the 2018-09-26-clean-up-TransportMasterNodeAction branch October 2, 2018 02:17
DaveCTurner added a commit that referenced this pull request Oct 2, 2018
Mainly this fixes a warning by replacing the unchecked `new ActionListener`
with the checked `new ActionListener<Response>`, and it also fixes the line
length violations in this class.
bleskes added a commit that referenced this pull request Oct 2, 2018
kcm pushed a commit that referenced this pull request Oct 30, 2018
Mainly this fixes a warning by replacing the unchecked `new ActionListener`
with the checked `new ActionListener<Response>`, and it also fixes the line
length violations in this class.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants