gistfile1.txt

The code used to be:

      if ( s->quick->get_type() == QUICK_SELECT_I::QS_TYPE_RANGE)
      {
        ...
        if (filter)
        {
          ...
        }
        type= JT_RANGE;
      } 
      else // spetrunia: that is: s->quick->get_type() != QUICK_SELECT_I::QS_TYPE_RANGE
      {
        type= JT_INDEX_MERGE;

After the patch, the code is:


      if ( s->quick->get_type() == QUICK_SELECT_I::QS_TYPE_RANGE)
      {
        if (access_cost_factor > 0.0)
        {
          ...
          type= JT_RANGE;
        }
        // spetrunia: QUESTION: why do we not set type= JT_RANGE when s->quick->get_type()=INDEX_MERGE,
        //   but access_cost_factor <0?
      }
      else // spetrunia: that is: s->quick->get_type() != QUICK_SELECT_I::QS_TYPE_RANGE
      {
        type= JT_INDEX_MERGE;