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
1=== modified file 'extra/innochecksum.cc'
2--- extra/innochecksum.cc 2014-02-17 11:12:40 +0000
3+++ extra/innochecksum.cc 2014-07-23 04:01:43 +0000
4@@ -52,17 +52,6 @@
5 #include "mach0data.h" /* mach_read_from_4() */
6 #include "ut0crc32.h" /* ut_crc32_init() */
7
8-/** Smallest compressed page size */
9-#define PAGE_ZIP_MIN_SIZE (1 << 10)
10-
11-#define DICT_TF_FORMAT_SHIFT 5 /* file format */
12-#define DICT_TF_FORMAT_MASK \
13- ((~(~0 << (DICT_TF_BITS - DICT_TF_FORMAT_SHIFT))) << DICT_TF_FORMAT_SHIFT)
14-#define DICT_TF_FORMAT_51 0 /*!< InnoDB/MySQL up to 5.1 */
15-#define DICT_TF_FORMAT_ZIP 1 /*!< InnoDB plugin for 5.1:
16- compressed tables,
17- new BLOB treatment */
18-
19 #ifdef UNIV_NONINL
20 # include "fsp0fsp.ic"
21 # include "mach0data.ic"
22@@ -324,14 +313,13 @@
23 }
24 else if (page_type == FIL_PAGE_TYPE_FSP_HDR)
25 {
26- ulint format = flags & DICT_TF_FORMAT_MASK >> DICT_TF_FORMAT_SHIFT;
27 ulint zip_size = fsp_flags_get_zip_size(flags);
28
29 if (!flags)
30 {
31 printf("Detected file format: Antelope (5.1.7 or newer).\n");
32 }
33- else if (format == DICT_TF_FORMAT_ZIP)
34+ else if (DICT_TF_HAS_ATOMIC_BLOBS(flags))
35 {
36 printf("Detected file format: Barracuda ");
37 if (!zip_size)
38@@ -340,7 +328,7 @@
39 printf("(compressed with KEY_BLOCK_SIZE=%lu).\n", zip_size);
40 }
41 else
42- printf("Unknown file format: %lu\n", format);
43+ printf("Unknown file format flags: %lu\n", flags);
44 }
45 else
46 {
47
48=== modified file 'sql/handler.h'
49--- sql/handler.h 2014-06-23 12:07:10 +0000
50+++ sql/handler.h 2014-07-23 04:01:43 +0000
51@@ -37,10 +37,6 @@
52
53 class Alter_info;
54
55-#if MAX_KEY > 128
56-#error MAX_KEY is too large. Values up to 128 are supported.
57-#endif
58-
59 // the following is for checking tables
60
61 #define HA_ADMIN_ALREADY_DONE 1
62
63=== modified file 'sql/sql_show.cc'
64--- sql/sql_show.cc 2014-06-04 14:48:06 +0000
65+++ sql/sql_show.cc 2014-07-23 04:01:43 +0000
66@@ -1196,8 +1196,6 @@
67 }
68
69
70-#define LIST_PROCESS_HOST_LEN 64
71-
72 /**
73 Print "ON UPDATE" clause of a field into a string.
74
75
76=== modified file 'storage/innobase/handler/ha_innodb.cc'
77--- storage/innobase/handler/ha_innodb.cc 2014-06-04 14:48:06 +0000
78+++ storage/innobase/handler/ha_innodb.cc 2014-07-23 04:01:43 +0000
79@@ -11383,16 +11383,6 @@
80 return(innobase_buffer_pool_size);
81 }
82
83-UNIV_INTERN
84-bool
85-ha_innobase::is_corrupt() const
86-{
87- if (share->ib_table)
88- return ((bool)share->ib_table->is_corrupt);
89- else
90- return (FALSE);
91-}
92-
93 /*********************************************************************//**
94 Calculates the key number used inside MySQL for an Innobase index. We will
95 first check the "index translation table" for a match of the index to get
96
97=== modified file 'storage/innobase/handler/ha_innodb.h'
98--- storage/innobase/handler/ha_innodb.h 2014-06-04 14:48:06 +0000
99+++ storage/innobase/handler/ha_innodb.h 2014-07-23 04:01:43 +0000
100@@ -130,7 +130,6 @@
101 double read_time(uint index, uint ranges, ha_rows rows);
102 longlong get_memory_buffer_size() const;
103 my_bool is_fake_change_enabled(THD *thd);
104- bool is_corrupt() const;
105
106 int write_row(uchar * buf);
107 int update_row(const uchar * old_data, uchar * new_data);

Subscribers

People subscribed via source and target branches