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
Enhance and add more tests for ResizeRequest #68502
Conversation
@dakrone ,can you take a look at this PR? |
Pinging @elastic/es-core-features (Team:Core/Features) |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
server/src/test/java/org/elasticsearch/action/admin/indices/shrink/ResizeRequestTests.java
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
by converting it to unchecked exceptions, and then use it to create random create index requests.
to cover testEqualsAndHashcode and testSerialization
This is handled for us by AbstractWireTestCase#testEqualsAndHashcode
@gaobinlong thanks again for your patience, and apologies that it took me so long to get back to this. I've added some new commits -- do you think this is still a fair attempt at what you were trying to accomplish? |
Yeah, it looks great! That's really what I wanted! |
@dakrone I don't think it's fair for me to be a reviewer here anymore -- do you mind taking a look at things? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks both of you for iterating on it
This PR relates to #65714 and follows #67705.
The main changes of this PR are:
equals
andhashCode
methods inResizeRequest
.Add the newly added(edit: pulled out into Add max_single_primary_size to ResizeRequest's toXContent #68793)max_single_primary_size
parameter to theXContent
ofResizeRequest
.testEqualsAndHashCode
andtestSerializeRequest
forResizeRequest
.