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.

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

Add javadoc for DocWriteResponse.Builders #23267

Merged
merged 2 commits into from
Feb 21, 2017
Merged

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Feb 20, 2017

Add some comments for DocWriteResponse.Builders and rename the inner classes too.

@@ -348,9 +348,11 @@ protected static void parseInnerToXContent(XContentParser parser, DocWriteRespon
}

/**
* {@link DocWriteResponseBuilder} is used to build {@link DocWriteResponse} objects during XContent parsing.
* Base class of all {@link DocWriteResponse} builders. These {@link DocWriteResponse.Builder} are used during
* xcontent parsing to temporally store the parsed values, then the {@link Builder#build()} method is called to
Copy link
Member

Choose a reason for hiding this comment

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

temporarily

public static class DeleteResponseBuilder extends DocWriteResponse.DocWriteResponseBuilder {
/**
* Builder class for {@link DeleteResponse}. This builder is usually used during xcontent parsing to
* temporally store the parsed values, then the {@link DocWriteResponse.Builder#build()} method is called to
Copy link
Member

Choose a reason for hiding this comment

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

temporarily

public static class IndexResponseBuilder extends DocWriteResponse.DocWriteResponseBuilder {
/**
* Builder class for {@link IndexResponse}. This builder is usually used during xcontent parsing to
* temporally store the parsed values, then the {@link Builder#build()} method is called to
Copy link
Member

Choose a reason for hiding this comment

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

temporarily

public static class UpdateResponseBuilder extends DocWriteResponse.DocWriteResponseBuilder {
/**
* Builder class for {@link UpdateResponse}. This builder is usually used during xcontent parsing to
* temporally store the parsed values, then the {@link DocWriteResponse.Builder#build()} method is called to
Copy link
Member

Choose a reason for hiding this comment

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

temporarily

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 besides the typo I found, thanks!

@tlrx tlrx merged commit c88eb00 into elastic:master Feb 21, 2017
@tlrx tlrx deleted the add-javadoc branch February 21, 2017 09:19
tlrx added a commit that referenced this pull request Feb 21, 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.

None yet

2 participants