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

Proposed by Laurynas Biveinis
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 631
Proposed branch: lp:~laurynas-biveinis/percona-server/minor-fixes-5.6-porting-5.6
Merge into: lp:percona-server/5.6
Diff against target: 107 lines (+2/-31)
5 files modified
extra/innochecksum.cc (+2/-14)
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.6
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Review via email: mp+227856@code.launchpad.net

Description of the change

Automerge + implementation of

Fix bug 1182049 (Make innochecksum fully use InnoDB headers) by
removing PAGE_ZIP_MIN_SIZE and the various DICT_TF_* defines from
innochecksum.cc and using DICT_TF_HAS_ATOMIC_BLOBS instead.

http://jenkins.percona.com/job/percona-server-5.6-param/660/

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 'extra/innochecksum.cc'
--- extra/innochecksum.cc 2014-02-17 11:12:40 +0000
+++ extra/innochecksum.cc 2014-07-23 04:01:43 +0000
@@ -52,17 +52,6 @@
52#include "mach0data.h" /* mach_read_from_4() */52#include "mach0data.h" /* mach_read_from_4() */
53#include "ut0crc32.h" /* ut_crc32_init() */53#include "ut0crc32.h" /* ut_crc32_init() */
5454
55/** Smallest compressed page size */
56#define PAGE_ZIP_MIN_SIZE (1 << 10)
57
58#define DICT_TF_FORMAT_SHIFT 5 /* file format */
59#define DICT_TF_FORMAT_MASK \
60 ((~(~0 << (DICT_TF_BITS - DICT_TF_FORMAT_SHIFT))) << DICT_TF_FORMAT_SHIFT)
61#define DICT_TF_FORMAT_51 0 /*!< InnoDB/MySQL up to 5.1 */
62#define DICT_TF_FORMAT_ZIP 1 /*!< InnoDB plugin for 5.1:
63 compressed tables,
64 new BLOB treatment */
65
66#ifdef UNIV_NONINL55#ifdef UNIV_NONINL
67# include "fsp0fsp.ic"56# include "fsp0fsp.ic"
68# include "mach0data.ic"57# include "mach0data.ic"
@@ -324,14 +313,13 @@
324 }313 }
325 else if (page_type == FIL_PAGE_TYPE_FSP_HDR)314 else if (page_type == FIL_PAGE_TYPE_FSP_HDR)
326 {315 {
327 ulint format = flags & DICT_TF_FORMAT_MASK >> DICT_TF_FORMAT_SHIFT;
328 ulint zip_size = fsp_flags_get_zip_size(flags);316 ulint zip_size = fsp_flags_get_zip_size(flags);
329317
330 if (!flags)318 if (!flags)
331 {319 {
332 printf("Detected file format: Antelope (5.1.7 or newer).\n");320 printf("Detected file format: Antelope (5.1.7 or newer).\n");
333 }321 }
334 else if (format == DICT_TF_FORMAT_ZIP)322 else if (DICT_TF_HAS_ATOMIC_BLOBS(flags))
335 {323 {
336 printf("Detected file format: Barracuda ");324 printf("Detected file format: Barracuda ");
337 if (!zip_size)325 if (!zip_size)
@@ -340,7 +328,7 @@
340 printf("(compressed with KEY_BLOCK_SIZE=%lu).\n", zip_size);328 printf("(compressed with KEY_BLOCK_SIZE=%lu).\n", zip_size);
341 }329 }
342 else330 else
343 printf("Unknown file format: %lu\n", format);331 printf("Unknown file format flags: %lu\n", flags);
344 }332 }
345 else333 else
346 {334 {
347335
=== modified file 'sql/handler.h'
--- sql/handler.h 2014-06-23 12:07:10 +0000
+++ sql/handler.h 2014-07-23 04:01:43 +0000
@@ -37,10 +37,6 @@
3737
38class Alter_info;38class Alter_info;
3939
40#if MAX_KEY > 128
41#error MAX_KEY is too large. Values up to 128 are supported.
42#endif
43
44// the following is for checking tables40// the following is for checking tables
4541
46#define HA_ADMIN_ALREADY_DONE 142#define HA_ADMIN_ALREADY_DONE 1
4743
=== modified file 'sql/sql_show.cc'
--- sql/sql_show.cc 2014-06-04 14:48:06 +0000
+++ sql/sql_show.cc 2014-07-23 04:01:43 +0000
@@ -1196,8 +1196,6 @@
1196}1196}
11971197
11981198
1199#define LIST_PROCESS_HOST_LEN 64
1200
1201/**1199/**
1202 Print "ON UPDATE" clause of a field into a string.1200 Print "ON UPDATE" clause of a field into a string.
12031201
12041202
=== modified file 'storage/innobase/handler/ha_innodb.cc'
--- storage/innobase/handler/ha_innodb.cc 2014-06-04 14:48:06 +0000
+++ storage/innobase/handler/ha_innodb.cc 2014-07-23 04:01:43 +0000
@@ -11383,16 +11383,6 @@
11383 return(innobase_buffer_pool_size);11383 return(innobase_buffer_pool_size);
11384}11384}
1138511385
11386UNIV_INTERN
11387bool
11388ha_innobase::is_corrupt() const
11389{
11390 if (share->ib_table)
11391 return ((bool)share->ib_table->is_corrupt);
11392 else
11393 return (FALSE);
11394}
11395
11396/*********************************************************************//**11386/*********************************************************************//**
11397Calculates the key number used inside MySQL for an Innobase index. We will11387Calculates the key number used inside MySQL for an Innobase index. We will
11398first check the "index translation table" for a match of the index to get11388first check the "index translation table" for a match of the index to get
1139911389
=== modified file 'storage/innobase/handler/ha_innodb.h'
--- storage/innobase/handler/ha_innodb.h 2014-06-04 14:48:06 +0000
+++ storage/innobase/handler/ha_innodb.h 2014-07-23 04:01:43 +0000
@@ -130,7 +130,6 @@
130 double read_time(uint index, uint ranges, ha_rows rows);130 double read_time(uint index, uint ranges, ha_rows rows);
131 longlong get_memory_buffer_size() const;131 longlong get_memory_buffer_size() const;
132 my_bool is_fake_change_enabled(THD *thd);132 my_bool is_fake_change_enabled(THD *thd);
133 bool is_corrupt() const;
134133
135 int write_row(uchar * buf);134 int write_row(uchar * buf);
136 int update_row(const uchar * old_data, uchar * new_data);135 int update_row(const uchar * old_data, uchar * new_data);

Subscribers

People subscribed via source and target branches