Merge lp:~laurynas-biveinis/percona-server/minor-fixes-5.6-porting-5.5 into lp:percona-server/5.5

Proposed by Laurynas Biveinis
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 684
Proposed branch: lp:~laurynas-biveinis/percona-server/minor-fixes-5.6-porting-5.5
Merge into: lp:percona-server/5.5
Diff against target: 60 lines (+0/-17)
4 files modified
sql/handler.h (+0/-4)
sql/sql_show.cc (+0/-2)
storage/innobase/handler/ha_innodb.cc (+0/-10)
storage/innobase/handler/ha_innodb.h (+0/-1)
To merge this branch: bzr merge lp:~laurynas-biveinis/percona-server/minor-fixes-5.6-porting-5.5
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Review via email: mp+227855@code.launchpad.net

Description of the change

Fix assorted minor issues discovered in the 5.6 porting and its code review.

673. By Laurynas Biveinis 14 hours ago
Fix bug 1182050 (Duplicate LIST_PROCESS_HOST_LEN definition) by
removing a copy from sql_show.cc, so that there is only one definition
and that upstream changes to it result in a merge conflict.

672. By Laurynas Biveinis 14 hours ago
Fix bug 1182072 (Unused ha_innobase::is_corrupt() method) by removing
it.

671. By Laurynas Biveinis 14 hours ago
Fix bug 1182046 (Unnecessary MAX_KEY <= 128 compile-time check in
userstat) by removing the check.

http://jenkins.percona.com/job/percona-server-5.5-param/1029/

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sql/handler.h'
--- sql/handler.h 2013-08-22 13:16:55 +0000
+++ sql/handler.h 2014-07-23 03:59:03 +0000
@@ -36,10 +36,6 @@
36#include <ft_global.h>36#include <ft_global.h>
37#include <keycache.h>37#include <keycache.h>
3838
39#if MAX_KEY > 128
40#error MAX_KEY is too large. Values up to 128 are supported.
41#endif
42
43// the following is for checking tables39// the following is for checking tables
4440
45#define HA_ADMIN_ALREADY_DONE 141#define HA_ADMIN_ALREADY_DONE 1
4642
=== modified file 'sql/sql_show.cc'
--- sql/sql_show.cc 2014-06-03 13:14:25 +0000
+++ sql/sql_show.cc 2014-07-23 03:59:03 +0000
@@ -1062,8 +1062,6 @@
1062}1062}
10631063
10641064
1065#define LIST_PROCESS_HOST_LEN 64
1066
1067static bool get_field_default_value(THD *thd, Field *timestamp_field,1065static bool get_field_default_value(THD *thd, Field *timestamp_field,
1068 Field *field, String *def_value,1066 Field *field, String *def_value,
1069 bool quoted)1067 bool quoted)
10701068
=== modified file 'storage/innobase/handler/ha_innodb.cc'
--- storage/innobase/handler/ha_innodb.cc 2014-03-27 15:42:21 +0000
+++ storage/innobase/handler/ha_innodb.cc 2014-07-23 03:59:03 +0000
@@ -8995,16 +8995,6 @@
8995 return(ranges + (double) rows / (double) total_rows * time_for_scan);8995 return(ranges + (double) rows / (double) total_rows * time_for_scan);
8996}8996}
89978997
8998UNIV_INTERN
8999bool
9000ha_innobase::is_corrupt() const
9001{
9002 if (share->ib_table)
9003 return ((bool)share->ib_table->is_corrupt);
9004 else
9005 return (FALSE);
9006}
9007
9008/*********************************************************************//**8998/*********************************************************************//**
9009Calculates the key number used inside MySQL for an Innobase index. We will8999Calculates the key number used inside MySQL for an Innobase index. We will
9010first check the "index translation table" for a match of the index to get9000first check the "index translation table" for a match of the index to get
90119001
=== modified file 'storage/innobase/handler/ha_innodb.h'
--- storage/innobase/handler/ha_innodb.h 2014-03-27 15:42:21 +0000
+++ storage/innobase/handler/ha_innodb.h 2014-07-23 03:59:03 +0000
@@ -141,7 +141,6 @@
141 double scan_time();141 double scan_time();
142 double read_time(uint index, uint ranges, ha_rows rows);142 double read_time(uint index, uint ranges, ha_rows rows);
143 my_bool is_fake_change_enabled(THD *thd);143 my_bool is_fake_change_enabled(THD *thd);
144 bool is_corrupt() const;
145144
146 int write_row(uchar * buf);145 int write_row(uchar * buf);
147 int update_row(const uchar * old_data, uchar * new_data);146 int update_row(const uchar * old_data, uchar * new_data);

Subscribers

People subscribed via source and target branches