Return-Path: X-Original-To: apmail-hive-dev-archive@www.apache.org Delivered-To: apmail-hive-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 6942A11279 for ; Mon, 16 Jun 2014 10:09:03 +0000 (UTC) Received: (qmail 25292 invoked by uid 500); 16 Jun 2014 10:09:02 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 25214 invoked by uid 500); 16 Jun 2014 10:09:02 -0000 Mailing-List: contact dev-help@hive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hive.apache.org Delivered-To: mailing list dev@hive.apache.org Received: (qmail 25058 invoked by uid 500); 16 Jun 2014 10:09:02 -0000 Delivered-To: apmail-hadoop-hive-dev@hadoop.apache.org Received: (qmail 25002 invoked by uid 99); 16 Jun 2014 10:09:02 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 16 Jun 2014 10:09:02 +0000 Date: Mon, 16 Jun 2014 10:09:02 +0000 (UTC) From: "Gunther Hagleitner (JIRA)" To: hive-dev@hadoop.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (HIVE-7159) For inner joins push a 'is not null predicate' to the join sources for every non nullSafe join condition 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/HIVE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gunther Hagleitner updated HIVE-7159: ------------------------------------- Attachment: HIVE-7159.6.patch More golden file updates in 6. There seems to be a couple of issues though that are either wrong with this patch or might be exposing problems with the original code: a) subquery transformation doesn't generate the nullsafe array (already addressed in patch) b) join pushdown does not seem to correctly populate the nullsafe array (size(join condns) != size(nullsafe)) c) Somehow this patch seems to change ptf/windowing tests results (actual query results, not just explain) [~rhbutani] any thoughts? > For inner joins push a 'is not null predicate' to the join sources for every non nullSafe join condition > -------------------------------------------------------------------------------------------------------- > > Key: HIVE-7159 > URL: https://issues.apache.org/jira/browse/HIVE-7159 > Project: Hive > Issue Type: Bug > Reporter: Harish Butani > Assignee: Harish Butani > Attachments: HIVE-7159.1.patch, HIVE-7159.2.patch, HIVE-7159.3.patch, HIVE-7159.4.patch, HIVE-7159.5.patch, HIVE-7159.6.patch > > > A join B on A.x = B.y > can be transformed to > (A where x is not null) join (B where y is not null) on A.x = B.y > Apart from avoiding shuffling null keyed rows it also avoids issues with reduce-side skew when there are a lot of null values in the data. > Thanks to [~gopalv] for the analysis and coming up with the solution. -- This message was sent by Atlassian JIRA (v6.2#6252)