diff options
author | Nikita Malyavin <nikitamalyavin@gmail.com> | 2019-02-22 22:17:41 +1000 |
---|---|---|
committer | Sergei Golubchik <serg@mariadb.org> | 2019-03-29 12:51:19 +0100 |
commit | e6230e844c096e321294f5489d6088cfd8f0293f (patch) | |
tree | c1eb8c463db1ae4f4124c0b0427f6ef2ad6fa126 /sql/partition_element.h | |
parent | f6ee132491f45dfe1856a627a686d29efbfa3034 (diff) | |
download | mariadb-git-e6230e844c096e321294f5489d6088cfd8f0293f.tar.gz |
MDEV-15951 system versioning by trx id doesn't work with partitioning
Fix partitioning for trx_id-versioned tables.
`partition by hash`, `range` and others now work.
`partition by system_time` is forbidden.
Currently we cannot use row_start and row_end in `partition by`, because
insertion of versioned field is done by engine's handler, as well as
row_start/row_end's value set up, which is a transaction id -- so it's
also forbidden.
The drawback is that it's now impossible to use `partition by key()`
without parameters for such tables, because it references row_start and
row_end implicitly.
* add handler::vers_can_native()
* drop Table_scope_and_contents_source_st::vers_native()
* drop partition_element::find_engine_flag as unused
* forbid versioning partitioning for trx_id as not supported
* adopt vers tests for trx_id partitioning
* forbid any row_end referencing in `partition by` clauses,
including implicit `by key()`
Diffstat (limited to 'sql/partition_element.h')
-rw-r--r-- | sql/partition_element.h | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/sql/partition_element.h b/sql/partition_element.h index 45900c77cfc..2c89562adda 100644 --- a/sql/partition_element.h +++ b/sql/partition_element.h @@ -176,21 +176,6 @@ public: DBUG_ASSERT(ev->col_val_array); return ev->col_val_array[idx]; } - - bool find_engine_flag(uint32 flag) - { - if (ha_check_storage_engine_flag(engine_type, flag)) - return true; - - List_iterator_fast<partition_element> it(subpartitions); - while (partition_element *element= it++) - { - if (element->find_engine_flag(flag)) - return true; - } - - return false; - } }; #endif /* PARTITION_ELEMENT_INCLUDED */ |