[
https://issues.apache.org/jira/browse/PARQUET-350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alex Levenson resolved PARQUET-350.
-----------------------------------
Resolution: Fixed
> ThriftRecordConverter throws NPE for unrecognized enum values
> -------------------------------------------------------------
>
> Key: PARQUET-350
> URL: https://issues.apache.org/jira/browse/PARQUET-350
> Project: Parquet
> Issue Type: Bug
> Components: parquet-mr
> Reporter: Alex Levenson
> Assignee: Alex Levenson
>
> currently:
> {noformat}
> @Override
> public void addBinary(final Binary value) {
> final int id = enumLookup.get(value);
> events.add(new ParquetProtocol("readI32() enum") {
> @Override
> public int readI32() throws TException {
> return id;
> }
> });
> }
> {noformat}
> the auto-unboxing from Integer to into throws a NPE when
> enumLookup.get(value) == null -- we should throw a better exception here that
> includes the value in question.
> This was actually triggered by someone *renaming* an enum, and because
> parquet stores enums by *name* instead of ID it is not compatible. I'm not
> sure why we store enums as strings, but we might want to reconsider that.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
