1. 16 Feb, 2020 6 commits
  2. 14 Feb, 2020 2 commits
    • Sergei Petrunia's avatar
      MDEV-21628: Index condition pushdown condition ... not used with BKA · 9f718041
      Sergei Petrunia authored
      Partitioning storage now supports MRR but doesn't support Index Condition
      Pushdown (aka ICP). This causes counter-intuitive query plans for queries
      that use BKA and conditions that depend on index fields:
      - If the condition refers to other tables, BKA's variant of ICP is used
         to handle it.
      - If the condition depends on this table only, the optimizer will try to
        use regular ICP for it, which will fail because the storage engine
        doesn't support ICP.
      
      Make the optimizer be smarter in the second case: if we were not able to
      use regular ICP, use BKA's variant of ICP..
      9f718041
    • Daniel Black's avatar
      mysys: remove windac my_security_attr_create (#1391) · 8eb03845
      Daniel Black authored
      No longer used.
      8eb03845
  3. 13 Feb, 2020 2 commits
    • Eugene Kosov's avatar
      micro optimization: avoid std::string copy · c400a73d
      Eugene Kosov authored
      c400a73d
    • Eugene Kosov's avatar
      MDEV-21669 InnoDB: Table ... contains <n> indexes inside InnoDB, which is... · 1394216e
      Eugene Kosov authored
      MDEV-21669 InnoDB: Table ... contains <n> indexes inside InnoDB, which is different from the number of indexes <n> defined in the MariaDB
      
      compare_keys_but_name(): do not use KEY_PART_INFO::field for
      Field::is_equal(). Following the logic of that code we need to
      compare fields of a table. But KEY_PART_INFO::field sometimes
      (when key part is shorter than table field) is a different field.
      In that case Field::is_equal() returns incorrect result and
      problems occur.
      
      KEY_PART_INFO::field may become some strange field in
      open_frm_error open_table_from_share(). I think this is an
      incorrect logic, some tecnhical debt. I'm not fixing it right now,
      because I don't have time. But I'm making Field::field_length
      a const class member. Then, the only fishy code which changed that
      field requires now a const_cast<>. I'm bringing attention to that
      code with it. This change should not affect logic of the
      program in any way.
      1394216e
  4. 12 Feb, 2020 1 commit
    • Sergey Vojtovich's avatar
      MDEV-20867 - Perform careful review of "Server crashes with BACKUP STAGE and... · c5e00fea
      Sergey Vojtovich authored
      MDEV-20867 - Perform careful review of "Server crashes with BACKUP STAGE and FLUSH TABLE table_name"
      
      Reverted original patch (c2e0a0b1).
      
      For consistency with "LOCK TABLE <table_name> READ" and "FLUSH TABLES
      WITH READ LOCK", which are forbidden under "BACKUP STAGE", forbid "FLUSH
      TABLE <table_name> FOR EXPORT" and "FLUSH TABLE <table_name> WITH READ
      LOCK" as well.
      
      It'd allow consistent fixes for problems like MDEV-18643.
      c5e00fea
  5. 11 Feb, 2020 1 commit
  6. 10 Feb, 2020 5 commits
  7. 09 Feb, 2020 4 commits
  8. 08 Feb, 2020 2 commits
  9. 07 Feb, 2020 4 commits
  10. 06 Feb, 2020 3 commits
  11. 05 Feb, 2020 4 commits
  12. 04 Feb, 2020 4 commits
  13. 03 Feb, 2020 2 commits