1. 02 Nov, 2011 1 commit
  2. 01 Nov, 2011 3 commits
    • unknown's avatar
      Fix bug lp:833702 · 7755bd9f
      unknown authored
      Analysis:
      Equality propagation propagated the constant '7' into
      args[0] of the Item_in_optimizer that stands for the
      "< ANY" predicate. At the same the min/max subquery
      rewrite swapped the order of the left and right operands
      of the "<" predicate, but used Item_in_subselect::left_expr.
      
      As a result, when the <ANY predicate is executed early in the
      execution phase as a contant condition, instead of a constant
      right (swapped) argument of the < predicate, there was a field
      (t3.a). This field had no data, since the whole predicate is
      considered constant, and it is evaluated before any tables are
      read. Having junk in the field row buffer produced wrong result
      
      Solution:
      Fix create_swap to pick the correct Item_in_optimizer left
      argument.
      7755bd9f
    • unknown's avatar
      Fix of typo. · c1364856
      unknown authored
      c1364856
    • Sergey Petrunya's avatar
      BUG#884631: Table elimination works 5.3 release builds even if turned off · de5d6fd2
      Sergey Petrunya authored
      - Make table elimination to actually switch itself on/off in release builds.
      de5d6fd2
  3. 31 Oct, 2011 1 commit
    • Sergey Petrunya's avatar
      BUG#882994: Crash in QUICK_RANGE_SELECT::reset with derived_with_keys · 27694c5e
      Sergey Petrunya authored
      - The bug was caused by the following scenario:
        = a quick select is created with get_quick_select_for_ref. The quick 
          select refers to temporary (derived) table. It saves table->file, which
          refers to a ha_heap object.
        = When temp table is populated, ha_heap reaches max. size and is converted
          to a ha_myisam.  However, quick->file remains pointing to where ha_heap 
          was. 
        = Attempt to use the quick select causes crash.
      - Fixed by introducing QUICK_SELECT_I::replace_handler(). Note that it will 
        not work for index_merge quick selects. Which is fine, because these
        quick selects are never created for derived tables.
      27694c5e
  4. 28 Oct, 2011 2 commits
  5. 27 Oct, 2011 4 commits
  6. 26 Oct, 2011 3 commits
    • Michael Widenius's avatar
      Fixed lp:879939 "assertion in ha_maria::enable_indexes with derived_with_keys=on" · d12d48ac
      Michael Widenius authored
      Honor unique/not unique when creating keys for internal tempory tables.
      Added new variables to be used to limit how keys are created for internal temporary tables.
      
      
      include/maria.h:
        Added maria_max_key_length() and maria_max_key_segments()
      include/myisam.h:
        Added myisam_max_key_length() and myisam_max_key_segments()
      mysql-test/r/mysql.result:
        Drop all used tables
      mysql-test/r/subselect4.result:
        Added test case for lp:879939
      mysql-test/t/mysql.test:
        Drop all used tables
      mysql-test/t/subselect4.test:
        Added test case for lp:879939
      sql/mysql_priv.h:
        Added internal_tmp_table_max_key_length and internal_tmp_table_max_key_segments to be used to limit how keys for derived tables are created.
      sql/mysqld.cc:
        Added internal_tmp_table_max_key_length and internal_tmp_table_max_key_segments to be used to limit how keys for derived tables are created.
      sql/share/errmsg.txt:
        Added new error message for internal errors
      sql/sql_select.cc:
        Give error if we try to create a wrong key (this error should never happen)
        Honor unique/not unique when creating keys for internal tempory tables.
      storage/maria/ha_maria.cc:
        Added change_table_ptr() to ensure that external_ref points always to the correct table.
        (Not having this caused an assert in the included test)
      storage/maria/ha_maria.h:
        Added change_table_ptr() to ensure that external_ref points always to the correct table.
      storage/maria/ma_check.c:
        Fixed bug in Duplicate key error printing (now row position is printed correctly)
      storage/maria/ma_create.c:
        maria_max_key_length() -> _ma_max_key_length()
      storage/maria/ma_info.c:
        Added extern function maria_max_key_length() to calculate the max key length based on current block size.
      storage/maria/ma_open.c:
        maria_max_key_length() -> _ma_max_key_length()
      storage/maria/maria_def.h:
        maria_max_key_length() -> _ma_max_key_length()
      storage/myisam/ha_myisam.cc:
        Added change_table_ptr() to ensure that external_ref points always to the correct table.
        (Not having this caused an assert in the included test)
      storage/myisam/ha_myisam.h:
        Added change_table_ptr() to ensure that external_ref points always to the correct table.
      d12d48ac
    • Igor Babaev's avatar
      Fixed LP bug #881449. · fff7acef
      Igor Babaev authored
      The function SELECT_LEX::update_used_tables first must clean up
      all bitmaps to be recalculated for all tables that require it
      and only after this walk through on conditions attached to the
      tables to update these bitmaps.
        
      fff7acef
    • Sergey Petrunya's avatar
      Merge fix for BUG#877288 · 429a435a
      Sergey Petrunya authored
      429a435a
  7. 25 Oct, 2011 2 commits
    • Sergey Petrunya's avatar
      BUG#877288: Wrong result with semijoin + materialization + multipart key · 213eaebe
      Sergey Petrunya authored
      - when create_ref_for_key() is constructing a ref access for
        a table that's inside a SJ-Materialization nest, it may not 
        use references to fields of tables that are unside the nest (because 
        these fields will not yet have values when ref access will be used)
        
        The check was performed in the first of create_ref_for_key's loops (the 
        one which counts how many key parts are usable) but not in the second
        (the one which actually fills the TABLE_REF structure). 
      213eaebe
    • Igor Babaev's avatar
      Fixed LP bug #881318. · dafb466f
      Igor Babaev authored
      If a materialized derived table / view is empty then for this table
      the value of file->ref is 0. This was not taken into account by
      the function JOIN_CACHE::write_record_data. As a result a query
      using an empty materialized derived tables as inner tables of outer
      joins and IN subqueries in WHERE conditions could cause server crashes
      when the optimizer employed join caches and duplicate elimination for
      semi-joins.
      dafb466f
  8. 24 Oct, 2011 1 commit
  9. 23 Oct, 2011 1 commit
    • Igor Babaev's avatar
      Fixed LP bug #879882. · 568d3427
      Igor Babaev authored
      This bug happened because the function Item_cond::eval_not_null_tables
      erroneously did not initialize the value of not_null_tables_cache.
      568d3427
  10. 22 Oct, 2011 2 commits
    • Igor Babaev's avatar
      Fixed LP bug #878769. · b8513f86
      Igor Babaev authored
      The method DsMrr_impl::dsmrr_init erroneously tried to get a KEY descriptor
      for key with number MAX_KEY. This caused valgrind complains.
      b8513f86
    • Igor Babaev's avatar
      Fixed LP bug #874378. · cc306396
      Igor Babaev authored
      This bug happened for the queries over multi-table mergeable views
      because the bitmap TABLE::read_set of the underlying tables were not
      updated after the views had been merged into the query.
      Now this bitmaps are updated properly.
      Also the bitmap TABLE::merge_keys now is updated in prevention of
      future bugs.
      cc306396
  11. 20 Oct, 2011 4 commits
    • Igor Babaev's avatar
    • Igor Babaev's avatar
      Fixed LP bug #878199. · 5b2b6bef
      Igor Babaev authored
      The function JOIN::drop_unused_derived_keys could erroneously set
      the value of REF::key to 0 for a joined materialized view/derived table
      in the case when no REF access to the table was used by the query
      execution plan. This could cause a crash of the server.
      5b2b6bef
    • Igor Babaev's avatar
      Merge. · aab6e4f2
      Igor Babaev authored
      aab6e4f2
    • Igor Babaev's avatar
      Fixed LP bug #877316. · 9634dcae
      Igor Babaev authored
      This bug happened due to incompleteness of the fix for bug 872735:
      the occurrences of the fields in the conditions of correlated
      subqueries were not taken into account when recalculating
      covering keys bit maps.
      9634dcae
  12. 19 Oct, 2011 3 commits
  13. 18 Oct, 2011 3 commits
  14. 17 Oct, 2011 2 commits
  15. 16 Oct, 2011 3 commits
    • Igor Babaev's avatar
      Fixed LP bug #874006. · 3f9e14f9
      Igor Babaev authored
      This bug manifested itself with queries containing non-correlated
      IN subqueries over materialized views/derived tables.  
      The bug happened because the code of the function generate_derived_keys did
      not take into account that the function could be called twice when the
      optimizer was deciding whether in-exist transformation should be applied.
      3f9e14f9
    • Michael Widenius's avatar
      Remove extra MariaDB- from binary tar.gz file name · f15724c5
      Michael Widenius authored
      Print server version name to .err file on crash 
      
      scripts/make_binary_distribution.sh:
        Remove extra MariaDB- from binary tar.gz file name
      sql/mysqld.cc:
        Print server version name to .err file on crash (to ensure people paste also that when they report a crash)
      f15724c5
    • Michael Widenius's avatar
      Fixed wrong info message for mysqld --general-log · 87b5f9ab
      Michael Widenius authored
      Fixed wrong parameter type for --general-log. Now one can enable it with --general-log= 1 | true | on
      Fixed that bool parameters can also take 'on' and 'off' as parameters. This is in line with the values assigned to them in mysqld.
      
      
      mysys/my_getopt.c:
        Fixed that bool parameters can also take 'on' and 'off' as parameters.
      sql/mysqld.cc:
        Fixed wrong info message for mysqld --general-log
        Fixed wrong parameter type for --general-log. Now one can enable it with --general-log= 1 | true | on
      87b5f9ab
  16. 14 Oct, 2011 5 commits