[
https://issues.apache.org/jira/browse/HIVE-7094?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14031197#comment-14031197
]
David Chen commented on HIVE-7094:
----------------------------------
Thanks, [~cwsteinbach]! I have addressed the remaining formatting issues using
the Eclipse formatter and uploaded a new patch.
> Separate out static/dynamic partitioning code in FileRecordWriterContainer
> --------------------------------------------------------------------------
>
> Key: HIVE-7094
> URL: https://issues.apache.org/jira/browse/HIVE-7094
> Project: Hive
> Issue Type: Sub-task
> Components: HCatalog
> Reporter: David Chen
> Assignee: David Chen
> Attachments: HIVE-7094.1.patch, HIVE-7094.3.patch
>
>
> There are two major places in FileRecordWriterContainer that have the {{if
> (dynamicPartitioning)}} condition: the constructor and write().
> This is the approach that I am taking:
> # Move the DP and SP code into two subclasses:
> DynamicFileRecordWriterContainer and StaticFileRecordWriterContainer.
> # Make FileRecordWriterContainer an abstract class that contains the common
> code for both implementations. For write(), FileRecordWriterContainer will
> call an abstract method that will provide the local RecordWriter,
> ObjectInspector, SerDe, and OutputJobInfo.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
