Merge lp:~sergei.glushchenko/percona-server/56-bug1157037 into lp:percona-server/5.6

Proposed by Sergei Glushchenko
Status: Rejected
Rejected by: Laurynas Biveinis
Proposed branch: lp:~sergei.glushchenko/percona-server/56-bug1157037
Merge into: lp:percona-server/5.6
Diff against target: 17 lines (+1/-6)
1 file modified
Percona-Server/sql/sql_tmp_table.cc (+1/-6)
To merge this branch: bzr merge lp:~sergei.glushchenko/percona-server/56-bug1157037
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Disapprove
Review via email: mp+154853@code.launchpad.net

Description of the change

Bug 1157037 create_duplicate_weedout_tmp_table() leaves key_part_flag uninitialized
Manual initialization of key_part_info fields has been replaced with
KEY_PART_INFO::init_from_field invocation

http://jenkins.percona.com/view/PS%205.6/job/percona-server-5.6-param/60/

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :
review: Disapprove

Unmerged revisions

323. By Sergei Glushchenko

Bug 1157037 create_duplicate_weedout_tmp_table() leaves key_part_flag uninitialized
Manual initialization of key_part_info fields has been replaced with
KEY_PART_INFO::init_from_field invocation.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Percona-Server/sql/sql_tmp_table.cc'
--- Percona-Server/sql/sql_tmp_table.cc 2013-02-12 07:47:19 +0000
+++ Percona-Server/sql/sql_tmp_table.cc 2013-03-22 04:56:25 +0000
@@ -1395,12 +1395,7 @@
1395 keyinfo->algorithm= HA_KEY_ALG_UNDEF;1395 keyinfo->algorithm= HA_KEY_ALG_UNDEF;
1396 keyinfo->name= (char*) "weedout_key";1396 keyinfo->name= (char*) "weedout_key";
1397 {1397 {
1398 key_part_info->null_bit=0;1398 key_part_info->init_from_field(field);
1399 key_part_info->field= field;
1400 key_part_info->offset= field->offset(table->record[0]);
1401 key_part_info->length= (uint16) field->key_length();
1402 key_part_info->type= (uint8) field->key_type();
1403 key_part_info->key_type = FIELDFLAG_BINARY;
1404 if (!using_unique_constraint)1399 if (!using_unique_constraint)
1405 {1400 {
1406 key_field= field->new_key_field(thd->mem_root, table, group_buff);1401 key_field= field->new_key_field(thd->mem_root, table, group_buff);

Subscribers

People subscribed via source and target branches