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

Ali Alsuliman updated ASTERIXDB-3083:
-------------------------------------
    Description: 
Currently, in order to use index-nested-loop-join algorithm via "indexnl" hint, 
it is required that the outer collection be cast to the type that is compatible 
with the index on the inner collection. This leads to a query that is 
semantically different from the hash-join algorithm and that (rightly) produces 
a different result. However, from a user perspective, using a hash-join or 
nested-loop join should give the same result. Here is an example:
{code:java}
bucket b1 has:
{"f": 1,"o": "f=1","id": 1}
{"f": 8,"o": "f=8","id": 2}
{"f": "1","o": "f=\"1\"","id": 3}

bucket b2 has:
{"f": 1,"o": "f=1","id": 1}
{"f": "1","o": "f=\"1\"","id": 2}
{"f": 9,"o": "f=9","id": 3}

create dataset ds1 on b1;
create dataset ds2 on b2;
create index idx_ds2 on ds2(f: int);
// hash-join
from ds1, ds2 where ds1.f = ds2.f select ds1, ds2;

[
  {
    "ds1": {
      "f": "1",
      "o": "f=\"1\"",
      "id": 3
    },
    "ds2": {
      "f": "1",
      "o": "f=\"1\"",
      "id": 2
    }
  },
  {
    "ds1": {
      "f": 1,
      "o": "f=1",
      "id": 1
    },
    "ds2": {
      "f": 1,
      "o": "f=1",
      "id": 1
    }
  }
]

// indexnl join where ds1 is the outer collection and ds2 is the inner
from ds1, ds2 where to_bigint(ds1.f) /*+ indexnl */ = ds2.f select ds1, ds2;

[
  {
    "ds1": {
      "f": 1,
      "o": "f=1",
      "id": 1
    },
    "ds2": {
      "f": 1,
      "o": "f=1",
      "id": 1
    }
  },
  {
    "ds1": {
      "f": "1",
      "o": "f=\"1\"",
      "id": 3
    },
    "ds2": {
      "f": 1,
      "o": "f=1",
      "id": 1
    }
  }
]

// the plan for the indexnl join
 distribute result [$$28]
-- DISTRIBUTE_RESULT  |PARTITIONED|
  exchange
  -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
    project ([$$28])
    -- STREAM_PROJECT  |PARTITIONED|
      assign [$$28] <- [{"ds1": $$ds1, "ds2": $$ds2}]
      -- ASSIGN  |PARTITIONED|
        project ([$$ds1, $$ds2])
        -- STREAM_PROJECT  |PARTITIONED|
          select (eq($$33, $$ds2.getField("f")))
          -- STREAM_SELECT  |PARTITIONED|
            project ([$$ds1, $$33, $$ds2])
            -- STREAM_PROJECT  |PARTITIONED|
              exchange
              -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
                unnest-map [$$31, $$ds2, $$32] <- index-search("ds2", 0, 
"Default", "ds2", true, false, 1, $$37, 1, $$37, true, true, true)
                -- BTREE_SEARCH  |PARTITIONED|
                  exchange
                  -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
                    order (ASC, $$37)
                    -- STABLE_SORT [$$37(ASC)]  |PARTITIONED|
                      exchange
                      -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
                        project ([$$ds1, $$33, $$37])
                        -- STREAM_PROJECT  |PARTITIONED|
                          exchange
                          -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
                            unnest-map [$$36, $$37] <- index-search("idx_ds2", 
0, "Default", "ds2", true, true, 1, $$33, 1, $$33, true, true, true)
                            -- BTREE_SEARCH  |PARTITIONED|
                              exchange
                              -- BROADCAST_EXCHANGE  |PARTITIONED|
                                assign [$$33] <- 
[to-bigint($$ds1.getField("f"))]
                                -- ASSIGN  |PARTITIONED|
                                  project ([$$ds1])
                                  -- STREAM_PROJECT  |PARTITIONED|
                                    exchange
                                    -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
                                      data-scan []<-[$$29, $$ds1, $$30] <- 
Default.ds1
                                      -- DATASOURCE_SCAN  |PARTITIONED|
                                        exchange
                                        -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
                                          empty-tuple-source
                                          -- EMPTY_TUPLE_SOURCE  
|PARTITIONED|{code}

  was:
Currently, in order to use index-nested-loop-join algorithm via "indexnl" hint, 
it is required that the outer collection be cast to the type that is compatible 
with the index on the inner collection. This leads to a query that is 
semantically different from the hash-join algorithm and that (rightly) produces 
a different result. However, from a user perspective, using a hash-join or 
nested-loop join should give the same result. Here is an example:
{code:java}
bucket b1 has:
{"f": 1,"o": "f=1","id": 1}
{"f": 8,"o": "f=8","id": 2}
{"f": "1","o": "f=\"1\"","id": 3}

bucket b2 has:
{"f": 1,"o": "f=1","id": 1}
{"f": "1","o": "f=\"1\"","id": 2}
{"f": 9,"o": "f=9","id": 3}

create dataset ds1 on b1;
create dataset ds2 on b2;
create index idx_ds2 on ds2(f: int);
// hash-join
from ds1, ds2 where ds1.f = ds2.f select ds1, ds2;

[  {    "ds1": {      "f": "1",      "o": "f=\"1\"",      "id": 3    },    
"ds2": {      "f": "1",      "o": "f=\"1\"",      "id": 2    }  },  {    "ds1": 
{      "f": 1,      "o": "f=1",      "id": 1    },    "ds2": {      "f": 1,     
 "o": "f=1",      "id": 1    }  }]

// indexnl join where ds1 is the outer collection and ds2 is the inner
from ds1, ds2 where to_bigint(ds1.f) /*+ indexnl */ = ds2.f select ds1, ds2;

[  {    "ds1": {      "f": 1,      "o": "f=1",      "id": 1    },    "ds2": {   
   "f": 1,      "o": "f=1",      "id": 1    }  },  {    "ds1": {      "f": "1", 
     "o": "f=\"1\"",      "id": 3    },    "ds2": {      "f": 1,      "o": 
"f=1",      "id": 1    }  }]

// the plan for the indexnl join

distribute result [$$28]-- DISTRIBUTE_RESULT  |PARTITIONED|  exchange  -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|    project ([$$28])    -- STREAM_PROJECT  
|PARTITIONED|      assign [$$28] <- [{"ds1": $$ds1, "ds2": $$ds2}]      -- 
ASSIGN  |PARTITIONED|        project ([$$ds1, $$ds2])        -- STREAM_PROJECT  
|PARTITIONED|          select (eq($$33, $$ds2.getField("f")))          -- 
STREAM_SELECT  |PARTITIONED|            project ([$$ds1, $$33, $$ds2])          
  -- STREAM_PROJECT  |PARTITIONED|              exchange              -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|                unnest-map [$$31, $$ds2, 
$$32] <- index-search("ds2", 0, "Default", "ds2", true, false, 1, $$37, 1, 
$$37, true, true, true)                -- BTREE_SEARCH  |PARTITIONED|           
       exchange                  -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|          
          order (ASC, $$37)                    -- STABLE_SORT [$$37(ASC)]  
|PARTITIONED|                      exchange                      -- 
ONE_TO_ONE_EXCHANGE  |PARTITIONED|                        project ([$$ds1, 
$$33, $$37])                        -- STREAM_PROJECT  |PARTITIONED|            
              exchange                          -- ONE_TO_ONE_EXCHANGE  
|PARTITIONED|                            unnest-map [$$36, $$37] <- 
index-search("idx_ds2", 0, "Default", "ds2", true, true, 1, $$33, 1, $$33, 
true, true, true)                            -- BTREE_SEARCH  |PARTITIONED|     
                         exchange                              -- 
BROADCAST_EXCHANGE  |PARTITIONED|                                assign [$$33] 
<- [to-bigint($$ds1.getField("f"))]                                -- ASSIGN  
|PARTITIONED|                                  project ([$$ds1])                
                  -- STREAM_PROJECT  |PARTITIONED|                              
      exchange                                    -- ONE_TO_ONE_EXCHANGE  
|PARTITIONED|                                      data-scan []<-[$$29, $$ds1, 
$$30] <- Default.ds1                                      -- DATASOURCE_SCAN  
|PARTITIONED|                                        exchange                   
                     -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|                      
                    empty-tuple-source                                          
-- EMPTY_TUPLE_SOURCE  |PARTITIONED| {code}


> Indexnl gives different result than hash-join
> ---------------------------------------------
>
>                 Key: ASTERIXDB-3083
>                 URL: https://issues.apache.org/jira/browse/ASTERIXDB-3083
>             Project: Apache AsterixDB
>          Issue Type: Bug
>          Components: IDX - Indexes
>    Affects Versions: 0.9.3
>            Reporter: Ali Alsuliman
>            Assignee: Ali Alsuliman
>            Priority: Major
>             Fix For: 0.9.7
>
>
> Currently, in order to use index-nested-loop-join algorithm via "indexnl" 
> hint, it is required that the outer collection be cast to the type that is 
> compatible with the index on the inner collection. This leads to a query that 
> is semantically different from the hash-join algorithm and that (rightly) 
> produces a different result. However, from a user perspective, using a 
> hash-join or nested-loop join should give the same result. Here is an example:
> {code:java}
> bucket b1 has:
> {"f": 1,"o": "f=1","id": 1}
> {"f": 8,"o": "f=8","id": 2}
> {"f": "1","o": "f=\"1\"","id": 3}
> bucket b2 has:
> {"f": 1,"o": "f=1","id": 1}
> {"f": "1","o": "f=\"1\"","id": 2}
> {"f": 9,"o": "f=9","id": 3}
> create dataset ds1 on b1;
> create dataset ds2 on b2;
> create index idx_ds2 on ds2(f: int);
> // hash-join
> from ds1, ds2 where ds1.f = ds2.f select ds1, ds2;
> [
>   {
>     "ds1": {
>       "f": "1",
>       "o": "f=\"1\"",
>       "id": 3
>     },
>     "ds2": {
>       "f": "1",
>       "o": "f=\"1\"",
>       "id": 2
>     }
>   },
>   {
>     "ds1": {
>       "f": 1,
>       "o": "f=1",
>       "id": 1
>     },
>     "ds2": {
>       "f": 1,
>       "o": "f=1",
>       "id": 1
>     }
>   }
> ]
> // indexnl join where ds1 is the outer collection and ds2 is the inner
> from ds1, ds2 where to_bigint(ds1.f) /*+ indexnl */ = ds2.f select ds1, ds2;
> [
>   {
>     "ds1": {
>       "f": 1,
>       "o": "f=1",
>       "id": 1
>     },
>     "ds2": {
>       "f": 1,
>       "o": "f=1",
>       "id": 1
>     }
>   },
>   {
>     "ds1": {
>       "f": "1",
>       "o": "f=\"1\"",
>       "id": 3
>     },
>     "ds2": {
>       "f": 1,
>       "o": "f=1",
>       "id": 1
>     }
>   }
> ]
> // the plan for the indexnl join
>  distribute result [$$28]
> -- DISTRIBUTE_RESULT  |PARTITIONED|
>   exchange
>   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>     project ([$$28])
>     -- STREAM_PROJECT  |PARTITIONED|
>       assign [$$28] <- [{"ds1": $$ds1, "ds2": $$ds2}]
>       -- ASSIGN  |PARTITIONED|
>         project ([$$ds1, $$ds2])
>         -- STREAM_PROJECT  |PARTITIONED|
>           select (eq($$33, $$ds2.getField("f")))
>           -- STREAM_SELECT  |PARTITIONED|
>             project ([$$ds1, $$33, $$ds2])
>             -- STREAM_PROJECT  |PARTITIONED|
>               exchange
>               -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>                 unnest-map [$$31, $$ds2, $$32] <- index-search("ds2", 0, 
> "Default", "ds2", true, false, 1, $$37, 1, $$37, true, true, true)
>                 -- BTREE_SEARCH  |PARTITIONED|
>                   exchange
>                   -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>                     order (ASC, $$37)
>                     -- STABLE_SORT [$$37(ASC)]  |PARTITIONED|
>                       exchange
>                       -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>                         project ([$$ds1, $$33, $$37])
>                         -- STREAM_PROJECT  |PARTITIONED|
>                           exchange
>                           -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>                             unnest-map [$$36, $$37] <- 
> index-search("idx_ds2", 0, "Default", "ds2", true, true, 1, $$33, 1, $$33, 
> true, true, true)
>                             -- BTREE_SEARCH  |PARTITIONED|
>                               exchange
>                               -- BROADCAST_EXCHANGE  |PARTITIONED|
>                                 assign [$$33] <- 
> [to-bigint($$ds1.getField("f"))]
>                                 -- ASSIGN  |PARTITIONED|
>                                   project ([$$ds1])
>                                   -- STREAM_PROJECT  |PARTITIONED|
>                                     exchange
>                                     -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>                                       data-scan []<-[$$29, $$ds1, $$30] <- 
> Default.ds1
>                                       -- DATASOURCE_SCAN  |PARTITIONED|
>                                         exchange
>                                         -- ONE_TO_ONE_EXCHANGE  |PARTITIONED|
>                                           empty-tuple-source
>                                           -- EMPTY_TUPLE_SOURCE  
> |PARTITIONED|{code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to