Mailing-List: contact issues-help@drill.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@drill.apache.org
Date: Fri, 6 Mar 2015 01:44:38 +0000 (UTC)
From: "Jason Altekruse (JIRA)" <jira@apache.org>
To: issues@drill.apache.org
Message-ID: <JIRA.12779955.1425606044000.17024.1425606278929@Atlassian.JIRA>
In-Reply-To: <JIRA.12779955.1425606044000@Atlassian.JIRA>
References: <JIRA.12779955.1425606044000@Atlassian.JIRA>
 <JIRA.12779955.1425606044684@arcas>
Subject: [jira] [Assigned] (DRILL-2395) Improve interpreted expression
 evaluation to only call the setup method once per batch
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Jason Altekruse reassigned DRILL-2395:
--------------------------------------

    Assignee: Jason Altekruse  (was: Daniel Barclay (Drill))

> Improve interpreted expression evaluation to only call the setup method once per batch
> --------------------------------------------------------------------------------------
>
>                 Key: DRILL-2395
>                 URL: https://issues.apache.org/jira/browse/DRILL-2395
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Functions - Drill
>            Reporter: Jason Altekruse
>            Assignee: Jason Altekruse
>            Priority: Minor
>
> This enhancement request came out of the review for a patch for DRILL-2060. Copied below is the discussion from there:
> Jinfeng:
> Do you have a plan to move setup() call into places such that setup() will be called once for each VectorAccessible input?
> In the code compile + evaluation model, doSetup() will be called per batch, in stead of per row.
> Jason:
> I have started working on a fix for this. Its a little complicated with setting constant inputs before the setup method is called. I'm trying to figure out the best way to share code with the rest of the input passing used in the EvalVisitor. Would you be okay with this being opened as an enhancement request to be merged a little later? Considering the current use of the interpreter this won't have an impact on any actual queries today.


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