• gkodinov/kgeorge@magare.gmz's avatar
    Bug #26815: · 4f2ec8f3
    gkodinov/kgeorge@magare.gmz authored
     When creating a temporary table the concise column type
     of a string expression is decided based on its length:
     - if its length is under 512 it is stored as either 
       varchar or char.
     - otherwise it is stored as a BLOB.
     
     There is a flag (convert_blob_length) to create_tmp_field 
     that, when >0 allows to force creation of a varchar if the
     max blob length is under convert_blob_length.
     However it must be verified that convert_blob_length 
     (settable through a SQL option in some cases) is 
     under the maximum that can be stored in a varchar column.
     While performing that check for expressions in 
     create_tmp_field_from_item the max length of the blob was
     used instead. This causes blob columns to be created in the
     heap temp table used by GROUP_CONCAT (where blobs must not
     be created in the temp table because of the constant 
     convert_blob_length that is passed to create_tmp_field() ).
     And since these blob columns are not expected in that place
     we get wrong results.
     Fixed by checking that the value of the flag variable is 
     in the limits that fit into VARCHAR instead of the max length
     of the blob column.
    4f2ec8f3
item_sum.cc 82.1 KB