[
https://issues.apache.org/jira/browse/DRILL-4956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15724499#comment-15724499
]
ASF GitHub Bot commented on DRILL-4956:
---------------------------------------
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/666#discussion_r90690234
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/WriterRecordBatch.java
---
@@ -174,10 +174,15 @@ protected void setupNewSchema() throws IOException {
schema = container.getSchema();
}
+ /** Partially written data will be removed, otherwise clean up will be
performed. */
private void closeWriter() {
if (recordWriter != null) {
try {
- recordWriter.cleanup();
+ if (processed) {
+ recordWriter.cleanup();
+ } else {
+ recordWriter.abort();
+ }
--- End diff --
Can these fail? If so, should we set the recordWriter to null in the
finally section so that the writer is marked as closed even in the failed-close
case?
Also, we mark the context as failed below. Must the caller check this
status and propagate the error up the call stack? Does this happen? Should this
method throw an exception instead to force the attention of the caller?
> Temporary tables support
> ------------------------
>
> Key: DRILL-4956
> URL: https://issues.apache.org/jira/browse/DRILL-4956
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.8.0
> Reporter: Arina Ielchiieva
> Assignee: Paul Rogers
> Labels: doc-impacting
> Fix For: Future
>
>
> Link to design doc -
> https://docs.google.com/document/d/1gSRo_w6q2WR5fPx7SsQ5IaVmJXJ6xCOJfYGyqpVOC-g/edit
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
