Skip to content

PARQUET-253: Fixes Javadoc of AvroSchemaConverter#173

Closed
liancheng wants to merge 1 commit intoapache:masterfrom
liancheng:avro-schema-converter-comment-fix
Closed

PARQUET-253: Fixes Javadoc of AvroSchemaConverter#173
liancheng wants to merge 1 commit intoapache:masterfrom
liancheng:avro-schema-converter-comment-fix

Conversation

@liancheng
Copy link
Contributor

Got confused by the original Javadoc at first and didn't realize AvroSchemaConverter is also capable to convert a Parquet schema to an Avro schema.

Review on Reviewable

@rdblue
Copy link
Contributor

rdblue commented Apr 14, 2015

@liancheng could you open an issue for this and update the PR title? We use the Spark script and only merge PRs with a JIRA ID in the title: "PARQUET-XYZ: Fixes Javadoc..."

Thanks!

@liancheng liancheng changed the title Fixes Javadoc of AvroSchemaConverter PARQUET-253: Fixes Javadoc of AvroSchemaConverter Apr 15, 2015
@liancheng
Copy link
Contributor Author

@rdblue Done. Didn't realize Parquet is more strict than Spark, JIRA ticket is optional there when merging minor PRs :)

@rdblue
Copy link
Contributor

rdblue commented Apr 30, 2015

Thanks for fixing that! Unfortunately because of the org.apache rename, I can't merge this unless you rebase. Thanks!

@rdblue
Copy link
Contributor

rdblue commented May 15, 2015

@liancheng could you rebase this? It isn't passing the merge script.

@asfgit asfgit closed this in 136c5ff May 15, 2015
@rdblue
Copy link
Contributor

rdblue commented May 15, 2015

Nevermind. It looks like the script was lying and I could merge it. It just had to resort to a 3-way merge.

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