Commit 953f6531 authored by Alexey Botchkov's avatar Alexey Botchkov

Bug#31435 ha_innodb.cc:3983: ulint convert_search_mode_to_innobase(ha_rkey_function): Asse

    I think we don't need to issue an error statement in the convert_search_mode_to_innobase().
    Returning the PAGE_CUR_UNSUPP value is enough as allows to handle this
    case depending on the requirements.

per-file comments:
  sql/ha_innodb.cc 
Bug#31435 ha_innodb.cc:3983: ulint convert_search_mode_to_innobase(ha_rkey_function): Asse 
     no error issued in convert_search_mode_to_innobase.
     ha_innobase::records_in_range() returns HA_POS_ERROR if search mode isn't supported.
parent 94ac4f70
......@@ -3723,7 +3723,6 @@ convert_search_mode_to_innobase(
case HA_READ_MBR_WITHIN:
case HA_READ_MBR_DISJOINT:
case HA_READ_MBR_EQUAL:
my_error(ER_TABLE_CANT_HANDLE_SPKEYS, MYF(0));
return(PAGE_CUR_UNSUPP);
/* do not use "default:" in order to produce a gcc warning:
enumeration value '...' not handled in switch
......@@ -5204,7 +5203,7 @@ ha_innobase::records_in_range(
mode2);
} else {
n_rows = 0;
n_rows = HA_POS_ERROR;
}
dtuple_free_for_mysql(heap1);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment