kangkaisen commented on issue #1356: Wrong query result with constant 
URL: 
https://github.com/apache/incubator-doris/issues/1356#issuecomment-504619540
 
 
   Yes. I see And I agree with you.
   
   The issue should be here:
   
   ```
               if (!expr.isBound(desc.getId())) {
                   continue;
               }
   ```
   
   If we think conjunct -1 in (-1) could get here,  we will need change 
`expr.isBound` method, change the `expr.isBound` value to false for 
`LiteralExpr`.  
   
   If we think conjunct -1 in (-1) could not get here, we should simplify 
constant Expr earlier.
   
   I don't know the origin design, but I guess  conjunct -1 in (-1)  should 
simplify earlier.
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@doris.apache.org
For additional commands, e-mail: dev-h...@doris.apache.org

Reply via email to