You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues-all@impala.apache.org by "Csaba Ringhofer (Jira)" <ji...@apache.org> on 2020/06/10 09:34:00 UTC

[jira] [Updated] (IMPALA-5078) Break up expr-test.cc

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

Csaba Ringhofer updated IMPALA-5078:
------------------------------------
    Labels: newbie ramp-up  (was: newbie)

> Break up expr-test.cc
> ---------------------
>
>                 Key: IMPALA-5078
>                 URL: https://issues.apache.org/jira/browse/IMPALA-5078
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Backend
>            Reporter: Henry Robinson
>            Priority: Minor
>              Labels: newbie, ramp-up
>
> {{expr-test.cc}} clocks in at 7129 lines, which is about enough for my emacs to start slowing down a bit. Let's see if we can refactor it enough to have a couple of test files. Maybe moving all the string instructions into a separate {{expr-string-test.cc}}, and having a common header will be enough to make it a bit more manageable. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org