Skip to content

Conversation

@lw-lin
Copy link
Contributor

@lw-lin lw-lin commented Jan 10, 2016

…odule encoding, column, and hadoop

Codes change now and then, but some corresponding doc comments are left out.

This PR fixes only the doc comments that should have been changed. It should be OK, since none codes are touched.

@rdblue could you take a look please? Cheers.

@rdblue
Copy link
Contributor

rdblue commented Jan 11, 2016

Looks good to me, @proflin. Thanks for fixing docs issues!

Could you add a Parquet JIRA issue for this and add it to this PR's summary? We only merge PRs that start like "PARQUET-NNN: Fix mismatch of javadoc names and method parameters".

@lw-lin lw-lin changed the title Minor: Fix the mismatch of the parameters and their doc comments in m… PARQUET-421: Fix the mismatch of the parameters and their doc comments in m… Jan 12, 2016
@lw-lin lw-lin changed the title PARQUET-421: Fix the mismatch of the parameters and their doc comments in m… PARQUET-421: Fix mismatch of javadoc names and method parameters in m... Jan 12, 2016
@lw-lin
Copy link
Contributor Author

lw-lin commented Jan 12, 2016

@rdblue

JIRA ticket added & this PR's title updated.
Thanks for taking time reviewing this! :-)

@asfgit asfgit closed this in 84b2b74 Jan 12, 2016
@lw-lin lw-lin deleted the Minor--Fix-the-mismatch-of-the-parameters-and-their-doc-comments-in-module-encoding,-column,-and-hadoop branch January 13, 2016 06:09
piyushnarang pushed a commit to piyushnarang/parquet-mr that referenced this pull request Jun 15, 2016
…odule encoding, column, and hadoop

Codes change now and then, but some corresponding doc comments are left out.

This PR fixes only the doc comments that should have been changed. It should be OK, since none codes are touched.

@rdblue could you take a look please? Cheers.

Author: proflin <proflin.me@gmail.com>

Closes apache#307 from proflin/Minor--Fix-the-mismatch-of-the-parameters-and-their-doc-comments-in-module-encoding,-column,-and-hadoop and squashes the following commits:

34c7b01 [proflin] Minor: Fix the mismatch of the parameters and their doc comments in module encoding, column, and hadoop
rdblue pushed a commit to rdblue/parquet-mr that referenced this pull request Jan 6, 2017
…odule encoding, column, and hadoop

Codes change now and then, but some corresponding doc comments are left out.

This PR fixes only the doc comments that should have been changed. It should be OK, since none codes are touched.

@rdblue could you take a look please? Cheers.

Author: proflin <proflin.me@gmail.com>

Closes apache#307 from proflin/Minor--Fix-the-mismatch-of-the-parameters-and-their-doc-comments-in-module-encoding,-column,-and-hadoop and squashes the following commits:

34c7b01 [proflin] Minor: Fix the mismatch of the parameters and their doc comments in module encoding, column, and hadoop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants