phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cancylin <...@git.apache.org>
Subject [GitHub] phoenix pull request: PHOENIX-1225
Date Wed, 03 Sep 2014 07:12:15 GMT
Github user cancylin commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/9#discussion_r17034179
  
    --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/WhereOptimizer.java ---
    @@ -685,7 +685,7 @@ private KeySlots orKeySlots(OrExpression orExpression, List<KeySlots>
childSlots
                         for (KeySlot slot : childSlot) {
                             // We have a nested OR with nothing for this slot, so continue
                             if (slot == null) {
    -                            continue; // FIXME: I don't think this is ever necessary
    +                            return null; //If one childSlot does not have the PK columns,
let Phoenix scan all the key ranges of the table. 
    --- End diff --
    
    I download the DBTools from http://www.dbtools.com.br/EN/queryit/.
    Test the query in postgresql.
    ![psqloutput](https://cloud.githubusercontent.com/assets/8636229/4130494/715d628c-3338-11e4-99c1-e2a077820415.png)
    
    After modify the WhereOptiMizer.java.
    Phoenix outputs the same result.
                 ID       NAME    ADDRESS       SBIN      MONEY     NUMBER INDATE        
PASSWD 
    ---------- ---------- ---------- ---------- ---------- ---------- ---------- ----------

             1    Tester1    HSINCHU                   0.0        1.1 2012-12-23 10:00:00
6655447788 
             2    Tester2                              0.0        2.1 2012-10-25 14:00:00
  11223344 
             3    Tester3                              0.0        2.0 2012-12-25 10:09:53
6654321897 
             6 LikeTesterEnd                              0.0        7.1 2012-12-25 11:00:00
9876543210 
             8    Tester3     Taipei                   0.0        1.0 2011-12-25 10:00:00
      5678 
            25    Tester1   HSINCHUU                   0.0        1.1 2012-12-23 10:00:00
6655447788 
            26  Tester100 HSINCHUUXX                 12.03      100.1 2010-10-10 10:10:10
      pass 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message