Skip to content

Conversation

@original-brownbear
Copy link
Contributor

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

}

@Override
public Bucket lockRetentionPolicy(final BucketInfo bucket, final BucketTargetOption... options) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

API change in the dependency made overriding this necessary.
Since we don't use it yet, I added just a dummy override.

Copy link
Contributor

@alpar-t alpar-t left a comment

Choose a reason for hiding this comment

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

LGTM

@original-brownbear
Copy link
Contributor Author

@atorok thanks!

@imrimt
Copy link

imrimt commented Feb 5, 2019

I have a follow up question that I mention here. Would someone be able to take a look?

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.

5 participants