Return-Path: X-Original-To: apmail-drill-issues-archive@minotaur.apache.org Delivered-To: apmail-drill-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 0F81619633 for ; Wed, 9 Mar 2016 22:54:41 +0000 (UTC) Received: (qmail 88495 invoked by uid 500); 9 Mar 2016 22:54:41 -0000 Delivered-To: apmail-drill-issues-archive@drill.apache.org Received: (qmail 88466 invoked by uid 500); 9 Mar 2016 22:54:40 -0000 Mailing-List: contact issues-help@drill.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@drill.apache.org Delivered-To: mailing list issues@drill.apache.org Received: (qmail 88446 invoked by uid 99); 9 Mar 2016 22:54:40 -0000 Received: from arcas.apache.org (HELO arcas) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 09 Mar 2016 22:54:40 +0000 Received: from arcas.apache.org (localhost [127.0.0.1]) by arcas (Postfix) with ESMTP id C75AC2C14F4 for ; Wed, 9 Mar 2016 22:54:40 +0000 (UTC) Date: Wed, 9 Mar 2016 22:54:40 +0000 (UTC) From: "ASF GitHub Bot (JIRA)" To: issues@drill.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (DRILL-4476) Enhance Union-All operator for dealing with empty left input or empty both inputs MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/DRILL-4476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15188252#comment-15188252 ] ASF GitHub Bot commented on DRILL-4476: --------------------------------------- Github user hsuanyi commented on a diff in the pull request: https://github.com/apache/drill/pull/407#discussion_r55605493 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionAllRecordBatch.java --- @@ -491,6 +556,25 @@ private void inferOutputFieldsFromLeftSide() { } } + private void inferOutputFieldsFromRightSide() { --- End diff -- Inferencing happens only once when Drill receives the very first batches from left and right. (Schema-change is not yet supported for Union-All). Let me summarize the inference in four different situations: First of all, the field names are always determined by the left side (even when the left side is from empty file, we have the column names. Please see the comment above.) 1. Left: non-empty; Right: non-empty=> types determined by both sides with implicit casting involved 2. Left: empty; Right: non-empty=> type from the right 3. Left: non-empty; Right: empty=> types from the left 4. Left: empty; Right: empty=> types are nullable integer > Enhance Union-All operator for dealing with empty left input or empty both inputs > --------------------------------------------------------------------------------- > > Key: DRILL-4476 > URL: https://issues.apache.org/jira/browse/DRILL-4476 > Project: Apache Drill > Issue Type: Bug > Components: Execution - Relational Operators > Reporter: Sean Hsuan-Yi Chu > Assignee: Sean Hsuan-Yi Chu > > Union-All operator does not deal with the situation where left side comes from empty source. > Due to DRILL-2288's enhancement for empty sources, Union-All operator now can be allowed to support this scenario. -- This message was sent by Atlassian JIRA (v6.3.4#6332)