diff options
| author | Sergei Golubchik <serg@mariadb.org> | 2018-02-18 12:24:51 +0100 |
|---|---|---|
| committer | Sergei Golubchik <serg@mariadb.org> | 2018-02-23 15:33:23 +0100 |
| commit | c4c81a5b048149c267d92764183cc3dd69def818 (patch) | |
| tree | b636a1ad8e9fed21ed40ce6e1a7c9d7f1dadd23c /sql/ha_partition.cc | |
| parent | f38ef430138fcd67fa8fe354c48c8807e06aaf19 (diff) | |
| download | mariadb-git-c4c81a5b048149c267d92764183cc3dd69def818.tar.gz | |
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 26a3ff0a22e. Broken system versioning will be fixed in
following commits.
Diffstat (limited to 'sql/ha_partition.cc')
| -rw-r--r-- | sql/ha_partition.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/ha_partition.cc b/sql/ha_partition.cc index b990baf49fb..9043d56e867 100644 --- a/sql/ha_partition.cc +++ b/sql/ha_partition.cc @@ -8017,7 +8017,7 @@ int ha_partition::compare_number_of_records(ha_partition *me, ::info() is used to return information to the optimizer. Currently this table handler doesn't implement most of the fields really needed. SHOW also makes use of this data - Another note, if your handler doesn't proved exact record count, + Another note, if your handler doesn't provide exact record count, you will probably want to have the following in your code: if (records < 2) records = 2; |