Skip to content

Conversation

@trxcllnt
Copy link
Contributor

Forgot to serialize schema metadata. Resolves https://issues.apache.org/jira/browse/ARROW-2640

Copy link
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

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

+1. There's a JIRA about adding integration tests for metadata round-trip; since we also have ARROW-906 pending for C++, we should probably take that on soon (will also require work in Java)

@trxcllnt
Copy link
Contributor Author

@wesm btw I've never run the merge script, so that's why I haven't merged this. I'm happy to give it a shot if you're fine with it.

@wesm
Copy link
Member

wesm commented May 31, 2018

Sure, it's a good thing to have working. Let me know if you run into trouble

@wesm wesm closed this in 9aa7c52 Jun 4, 2018
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