Skip to content

Conversation

@tlrx
Copy link
Member

@tlrx tlrx commented May 16, 2017

This method is not used and not tested. But it existence forces implementations of the interface to implement it even if it's unused.

This method is not used and not tested. While it exists it forces
implementations of the interface to implement it while it's unused.
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

@tlrx tlrx merged commit 96dac4c into elastic:master May 18, 2017
tlrx added a commit that referenced this pull request May 18, 2017
This method is not used and not tested. While it exists it forces
implementations of the interface to implement it while it's unused.
@tlrx tlrx deleted the remove-sig-term-compare-method branch May 18, 2017 13:22
tlrx added a commit that referenced this pull request May 19, 2017
javanna pushed a commit to javanna/elasticsearch that referenced this pull request May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants