Skip to content

Commit c4c81a5

Browse files
committed
cleanup: partition_info::check_constants
partition_info had a bunch of function pointers to avoid if()'s when invoking part_type specific functionality (like get_part_id, etc). But check_range_constants() and check_list_constants() were still invoked conditionally, with if()'s. Create partition_info::check_constants function pointer, get rid of if()'s Also remove alloc argument of check_range_constants(), added in 26a3ff0. Broken system versioning will be fixed in following commits.
1 parent f38ef43 commit c4c81a5

File tree

4 files changed

+403
-439
lines changed

4 files changed

+403
-439
lines changed

sql/ha_partition.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8017,7 +8017,7 @@ int ha_partition::compare_number_of_records(ha_partition *me,
80178017
::info() is used to return information to the optimizer.
80188018
Currently this table handler doesn't implement most of the fields
80198019
really needed. SHOW also makes use of this data
8020-
Another note, if your handler doesn't proved exact record count,
8020+
Another note, if your handler doesn't provide exact record count,
80218021
you will probably want to have the following in your code:
80228022
if (records < 2)
80238023
records = 2;

0 commit comments

Comments
 (0)