Mailing-List: contact issues-help@drill.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@drill.apache.org
Date: Thu, 8 Oct 2015 20:50:27 +0000 (UTC)
From: "Mehant Baid (JIRA)" <jira@apache.org>
To: issues@drill.apache.org
Message-ID: <JIRA.12841759.1435687460000.60844.1444337427450@Atlassian.JIRA>
In-Reply-To: <JIRA.12841759.1435687460000@Atlassian.JIRA>
References: <JIRA.12841759.1435687460000@Atlassian.JIRA>
 <JIRA.12841759.1435687460949@arcas>
Subject: [jira] [Updated] (DRILL-3429) DrillAvgVarianceConvertlet may
 produce wrong results while rewriting stddev, variance
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Mehant Baid updated DRILL-3429:
-------------------------------
    Attachment: DRILL-3429.patch

> DrillAvgVarianceConvertlet may produce wrong results while rewriting stddev, variance
> -------------------------------------------------------------------------------------
>
>                 Key: DRILL-3429
>                 URL: https://issues.apache.org/jira/browse/DRILL-3429
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Mehant Baid
>            Assignee: Mehant Baid
>            Priority: Critical
>             Fix For: 1.3.0
>
>         Attachments: DRILL-3429.patch
>
>
> DrillAvgVarianceConvertlet currently rewrites aggregate functions like avg, stddev, variance to simple computations. 
> Eg: 
> Stddev( x ) => power(
>          (sum(x * x) - sum( x ) * sum( x ) / count( x ))
>          / count( x ),
>          .5)
> Consider the case when the input is an integer. Now the rewrite contains multiplication and division, which will bind to functions that operate on integers however the expected result should be a double and since double has more precision than integer we should be operating on double during the multiplication and division.  


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)