Skip to content

Conversation

@tfriedr
Copy link

@tfriedr tfriedr commented Nov 26, 2015

I made a few updates to the original patch PARQUET-334-1.patch proposed by Daniel. As the inputschema is maintained in EvalFunc, any reference to the private class variable inputSchema should be changed to getInputSchema(), and inputSchema can be removed because it will be null always.

@julienledem
Copy link
Member

+1

@julienledem
Copy link
Member

@tfriedr can you update the description to PARQUET-334:...? (with a dash, not an underscore)

@tfriedr tfriedr changed the title PARQUET_334: UT test failure with Pig 0.15 PARQUET-334: UT test failure with Pig 0.15 Dec 8, 2015
@tfriedr
Copy link
Author

tfriedr commented Dec 8, 2015

Thanks, Julien. Updated the title.

@asfgit asfgit closed this in fa7588c Dec 12, 2015
@julienledem
Copy link
Member

Thank you @tfriedr !

piyushnarang pushed a commit to piyushnarang/parquet-mr that referenced this pull request Jun 15, 2016
I made a few updates to the original patch PARQUET-334-1.patch proposed by Daniel. As the inputschema is maintained in EvalFunc, any reference to the private class variable inputSchema should be changed to getInputSchema(), and inputSchema can be removed because it will be null always.

Author: Thomas Friedrich <tfriedr@us.ibm.com>

Closes apache#292 from tfriedr/parquet-334 and squashes the following commits:

012563e [Thomas Friedrich] PARQUET_334: UT test failure with Pig 0.15
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