[ 
https://issues.apache.org/jira/browse/TAJO-1265?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266454#comment-14266454
 ] 

ASF GitHub Bot commented on TAJO-1265:
--------------------------------------

Github user hyunsik commented on a diff in the pull request:

    https://github.com/apache/tajo/pull/315#discussion_r22537086
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/engine/function/builtin/AvgLong.java ---
    @@ -18,6 +18,7 @@
     
     package org.apache.tajo.engine.function.builtin;
     
    +import org.apache.commons.lang.ObjectUtils;
    --- End diff --
    
    unused import


> min(), max() does not handle null properly
> ------------------------------------------
>
>                 Key: TAJO-1265
>                 URL: https://issues.apache.org/jira/browse/TAJO-1265
>             Project: Tajo
>          Issue Type: Bug
>            Reporter: Keuntae Park
>            Assignee: Keuntae Park
>            Priority: Minor
>
> In other DBs like oracle and postgreSQL, 
> null is excluded from calculation of min() and max() value until there is no 
> non-null value in the given column, in which case, it returns null. 
> However, current implementation of Tajo considers null as
> 0 for int or long column, 0.0 for float or double column, or "" for text 
> column.
> It needs to handle null value separately from non-null values in min() and 
> max() calculation. 



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

Reply via email to