Skip to content

Conversation

@lynnyuan-arch
Copy link
Contributor

No description provided.

@rdblue
Copy link
Contributor

rdblue commented Oct 6, 2017

@LynnYuanInspur, could you please change the PR title? It needs to have a : after the issue ID, and the description could be a bit more clear. For example, "PARQUET-1050: Fix misplaced comments in DataPageHeaderV2."

Thanks!

@rdblue
Copy link
Contributor

rdblue commented Oct 6, 2017

Actually, I backported changes from parquet-mr that makes our merge script a little friendlier. No need to update the title now. I can merge this. Thank you!

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.

3 participants