1. 14 Dec, 2017 4 commits
  2. 13 Dec, 2017 14 commits
    • Marko Mäkelä's avatar
      Merge 10.0 into 10.1 · 7be5b6f0
      Marko Mäkelä authored
      7be5b6f0
    • Marko Mäkelä's avatar
      Follow-up fix for MDEV-12352: Plug a memory leak · 9d76b274
      Marko Mäkelä authored
      trx_rollback_active(): When aborting the rollback, free the query graph.
      9d76b274
    • Marko Mäkelä's avatar
      Fix a typo: schedule, scheduling · e9cc486c
      Marko Mäkelä authored
      e9cc486c
    • Marko Mäkelä's avatar
      Fix the grammar of an error message · 2fe990df
      Marko Mäkelä authored
      2fe990df
    • Marko Mäkelä's avatar
      Merge 10.0 into 10.1 · 46305b00
      Marko Mäkelä authored
      46305b00
    • Marko Mäkelä's avatar
      MDEV-12323 Rollback progress log messages during crash recovery are intermixed... · b1977a39
      Marko Mäkelä authored
      MDEV-12323 Rollback progress log messages during crash recovery are intermixed with unrelated log messages
      
      trx_roll_must_shutdown(): During the rollback of recovered transactions,
      report progress and check if the rollback should be interrupted because
      of a pending shutdown.
      
      trx_roll_max_undo_no, trx_roll_progress_printed_pct: Remove, along with
      the messages that were interleaved with other messages.
      b1977a39
    • Marko Mäkelä's avatar
      08d0ea1f
    • Marko Mäkelä's avatar
      MDEV-12352 InnoDB shutdown should not be blocked by a large transaction rollback · b46fa627
      Marko Mäkelä authored
      row_undo_step(), trx_rollback_active(): Abort the rollback of a
      recovered ordinary transaction if fast shutdown has been initiated.
      
      trx_rollback_resurrected(): Convert an aborted-rollback transaction
      into a fake XA PREPARE transaction, so that fast shutdown can proceed.
      b46fa627
    • Marko Mäkelä's avatar
      MDEV-13797 InnoDB may hang if shutdown is initiated soon after startup while... · 6559ba71
      Marko Mäkelä authored
      MDEV-13797 InnoDB may hang if shutdown is initiated soon after startup while rolling back recovered incomplete transactions
      
      trx_rollback_resurrected(): If shutdown was initiated, fake all
      remaining active transactions to XA PREPARE state, so that shutdown
      can proceed. Also, make the parameter "all" an output that will be
      assigned to FALSE in this case.
      
      trx_rollback_or_clean_recovered(): Remove the shutdown check
      (it was moved to trx_rollback_resurrected()).
      
      trx_undo_free_prepared(): Relax assertions.
      6559ba71
    • Marko Mäkelä's avatar
      MDEV-14422 Assertion failure in trx_purge_run() on shutdown · 58eb4e5d
      Marko Mäkelä authored
      row_quiesce_table_start(), row_quiesce_table_complete():
      Use the more appropriate predicate srv_undo_sources for skipping
      purge control. (This change alone is insufficient; it is possible
      that this predicate will change during the call to trx_purge_stop()
      or trx_purge_run().)
      
      trx_purge_stop(), trx_purge_run(): Tolerate PURGE_STATE_EXIT.
      It is very well possible to initiate shutdown soon after the statement
      FLUSH TABLES FOR EXPORT has been submitted to execution.
      
      srv_purge_coordinator_thread(): Ensure that the wait for purge_sys->event
      in trx_purge_stop() will terminate when the coordinator thread exits.
      58eb4e5d
    • Alexander Barkov's avatar
      Removing a dead code in sql_load.cc · 701e22d5
      Alexander Barkov authored
      The loop in read_xml_field(), unlike the same loop in read_sep_field(),
      cannot end with item<>NULL, as it does not have any "break" statements.
      The entire block "if (item) {...}" was a dead code.
      701e22d5
    • Marko Mäkelä's avatar
      Try to prevent sporadic test failures · b93a87f1
      Marko Mäkelä authored
      On Windows, sometimes more files stay open:
      
      [Warning] InnoDB: innodb_open_files=13 is exceeded (15 files stay open)
      b93a87f1
    • Alexander Barkov's avatar
      MDEV-14628 Wrong autoinc value assigned by LOAD XML in the NO_AUTO_VALUE_ON_ZERO mode · a53e087e
      Alexander Barkov authored
      The fixes for these bugs:
      
      Bug#27586 Wrong autoinc value assigned by LOAD DATA in the NO_AUTO_VALUE_ON_ZERO mode
      Bug#22372 Disable spatial key, load data, enable spatial key, crashes table
      
      fixed only LOAD DATA INFILE, but did not fix LOAD XML INFILE.
      
      This patch does for LOAD XML FILE what patches for Bug#27586 and Bug#22372
      earlier did for LOAD DATA INFILE.
      
      1. Fixing the auto_increment problem:
         a. table->auto_increment_field_not_null is not set to TRUE
            anymore when a column does not have a corresponding XML tag.
         b. Adding "table->auto_increment_field_not_null= false"
            in the end of read_xml_field().
         These two changes resemble the patch for Bug#27586.
      
      2. Fixing the GEOMETRY problem:
         The result for "reset()" was not tested for errors in read_xml_field(),
         which made it possible for empty string to sneak into a "GEOMETRY NOT NULL"
         column when this column does not have a corresponding XML tag with data.
         After this patch the result of reset() is tested and and an error is
         returned in such cases.
         This change effectively resembles the patch for Bug#22372
      
      3. Spliting the code into a new virtual method Field::load_data_set_null().
      
         Rationale:
         a. To avoid duplicate code in read_sep_field() and read_xml_field():
            Changes #1 and #2 made the code handling NULL values for Field
            exactly the same in read_sep_field() and read_xml_field().
      
        b. To avoid tests for field_type(), which is not friendly to
           upcoming data type plugins.
           This change makes it possible for data type plugins
           to implement their own special way for handling NULL values in LOAD DATA
           by overriding Field_xxx::load_data_set_null(),
           like Field_geom and Field_timestamp do.
      a53e087e
    • Sergey Vojtovich's avatar
      MDEV-14505 - Threads_running becomes scalability bottleneck · 159a6c2e
      Sergey Vojtovich authored
      Instead of updating global counter, calculate Threads_running on the fly.
      All threads having command != COM_SLEEP are included.
      
      Behaviour changes:
      
      Previously SHOW STATUS and SHOW GLOBAL STATUS returned the same values
      representing global status. Now SHOW STATUS always returns 1 indicating that
      current session has 1 thread running.
      
      Previously only threads that were executing dispatch_command() or running events
      were accounted by Threads_running. Now it is rough equivalent of
      SELECT COUNT(*) FROM INFORMATION_SCHEMA.PROCESSLIST WHERE state!='Sleep'
      159a6c2e
  3. 12 Dec, 2017 15 commits
  4. 11 Dec, 2017 7 commits
    • Sergei Petrunia's avatar
      MDEV-14389: MyRocks and NOPAD collations · e12f77a7
      Sergei Petrunia authored
      Disallow use of NOPAD collations in indexed columns.
      e12f77a7
    • Vladislav Vaintroub's avatar
      MDEV-14113 Use abortive TCP close, in case server closes the connection · 70d6c944
      Vladislav Vaintroub authored
      first, and we do not not care whether client has received all data.
      
      This is a TCP optimization to avoid TIME_WAIT in TCP connection teardown.
      
      This patch would abort connection on timeout, which usually happens when
      client reads a large result set, at slower pace then the server can
      write.
      
      The patch also cleans up socket timeout handling, so that Windows
      is consistent with another platforms (using nonblocking socket IO
       + waiting in poll/select on single socket, rather than setsockopt).
      This makes identifying timeouts easier.
      
      
      Also removed the superficial shutdown() before closesocket() in a few
      places where it was used, because it  was never needed , and
      reportedly breaks SO_LINGER on Windows.
      70d6c944
    • Marko Mäkelä's avatar
      8695c816
    • Marko Mäkelä's avatar
      Merge 10.2 into bb-10.2-ext · e312a407
      Marko Mäkelä authored
      e312a407
    • Marko Mäkelä's avatar
      MDEV-14589 InnoDB should not lock a delete-marked record · 13b9ec65
      Marko Mäkelä authored
      When the transaction isolation level is SERIALIZABLE, or when
      a locking read is performed in the REPEATABLE READ isolation level,
      InnoDB must lock delete-marked records in order to prevent another
      transaction from inserting something.
      
      However, at READ UNCOMMITTED or READ COMMITTED isolation level or
      when the parameter innodb_locks_unsafe_for_binlog is set, the
      repeatability of the reads does not matter, and there is no need
      to lock any records.
      
      row_search_mvcc(): Skip locks on delete-marked committed records upfront,
      instead of invoking row_unlock_for_mysql() afterwards. The unlocking
      never worked for secondary index records.
      13b9ec65
    • Marko Mäkelä's avatar
      MDEV-14614 InnoDB: Failing assertion in dict_stats_rename_table() · 434c9e6f
      Marko Mäkelä authored
      dict_stats_rename_table(): After DB_LOCK_WAIT_TIMEOUT
      or DB_DUPLICATE_KEY, reset the trx->error_state before retrying.
      Also, properly treat DB_DEADLOCK as a hard error.
      434c9e6f
    • Marko Mäkelä's avatar
      1e6ac944