diff options
| author | Sergei Golubchik <serg@mariadb.org> | 2022-09-06 19:28:42 +0200 |
|---|---|---|
| committer | Sergei Golubchik <serg@mariadb.org> | 2022-10-26 15:30:38 +0200 |
| commit | 8d2ec37a40117996897c97904ac91b674a448cb8 (patch) | |
| tree | d62024aeeb8bd9d0197d21852f05d2568489fb9e /sql/table.cc | |
| parent | a2cda88631b47aaeec53a8fdd5ced48e04b8ceb0 (diff) | |
| download | mariadb-git-8d2ec37a40117996897c97904ac91b674a448cb8.tar.gz | |
MDEV-16546 post-review fixes
* clarify the help text for --system-versioning-insert-history
* move the vers_write=false check from Item_field::fix_fields()
next to other vers field checks in find_field_in_table()
* move row_start validation from handler::write_row() next to
vers_update_fields()
* make secure_timestamp check to happen in one place only,
extract it into a function is_set_timestamp_vorbidden().
* overwriting vers fields is an error, just like setting @@timestamp
* don't run vers_insert_history() for every row
Diffstat (limited to 'sql/table.cc')
| -rw-r--r-- | sql/table.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sql/table.cc b/sql/table.cc index e94c8eabed0..202ff6b708f 100644 --- a/sql/table.cc +++ b/sql/table.cc @@ -9155,9 +9155,6 @@ bool TABLE::check_period_overlaps(const KEY &key, void TABLE::vers_update_fields() { - if (!vers_write) - return; - if (versioned(VERS_TIMESTAMP)) { if (vers_start_field()->store_timestamp(in_use->query_start(), |