[
https://issues.apache.org/jira/browse/PARQUET-345?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alex Levenson resolved PARQUET-345.
-----------------------------------
Resolution: Fixed
> ThriftMetaData toString() should not try to load class reflectively
> -------------------------------------------------------------------
>
> Key: PARQUET-345
> URL: https://issues.apache.org/jira/browse/PARQUET-345
> Project: Parquet
> Issue Type: Bug
> Components: parquet-mr
> Reporter: Alex Levenson
> Assignee: Alex Levenson
> Priority: Minor
>
> Currently:
> {noformat}
> @Override
> public String toString() {
> return "ThriftMetaData" + toExtraMetaData();
> }
> /**
> * generates a map of key values to store in the footer
> * @return the key values
> */
> public Map<String, String> toExtraMetaData() {
> final Map<String, String> map = new HashMap<String, String>();
> map.put(THRIFT_CLASS, getThriftClass().getName());
> map.put(THRIFT_DESCRIPTOR, descriptor.toJSON());
> return map;
> }
> {noformat}
> There's no guarantee that getThriftClass() won't throw (a class in the file
> footer doesn't actually need to be on the classpath if its never used) --
> this masks real error when the error message is being generated.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
