[jira] [Updated] (HIVE-2260) ExecDriver::addInputPaths should pass the table properties to the record writer

["Jakob Homan (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HIVE-2260?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jakob Homan updated HIVE-2260:
------------------------------

    Attachment: HIVE-2260.patch

Patch takes the Properties from the table/partition and passes it to the RecordWriter.  This fixes the issue Haivvreo runs into.  

The patch doesn't include a unit test, since this is pretty far into the guts of ExecDriver.  I'd be happy to write one up and introduce Mockito, which I think is the only reasonable way of getting to this code path, if others think it's worthwhile.

> ExecDriver::addInputPaths should pass the table properties to the record writer
> -------------------------------------------------------------------------------
>
>                 Key: HIVE-2260
>                 URL: https://issues.apache.org/jira/browse/HIVE-2260
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 0.7.0
>            Reporter: Jakob Homan
>            Assignee: Jakob Homan
>         Attachments: HIVE-2260.patch
>
>
> Currently when ExecDriver encounters a non-existent partition, it creates an empty file so that the query will be valid (and return 0 results).  However, when it does this and calls {{getHiveRecordWriter()}}, it creates a new instance of Properties, rather than providing the Properties associated with the table.
> This causes RecordWriters that pull information from the table through the Properties to fail (such as [Haivvreo|http://bit.ly/iwEQzJ]).  The RecordWriter should be provided the table's Properties, as it is in all other cases where it's called.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira