summaryrefslogtreecommitdiff
path: root/sql/item_windowfunc.h
Commit message (Collapse)AuthorAgeFilesLines
* MDEV-20353 Add separate type handlers for unsigned integer data typesAlexander Barkov2019-08-151-3/+8
|
* Merge remote-tracking branch 'origin/10.4' into 10.5Alexander Barkov2019-08-131-19/+6
|\
| * Merge commit '43882e764d6867c6855b1ff057758a3f08b25c55' into 10.4Alexander Barkov2019-08-131-19/+6
| |\
| | * A cleanup for `MDEV-20273 Add class Item_sum_min_max` - removing duplicate codeAlexander Barkov2019-08-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Reusing the MIN()/MAX() fix_length_and_dec() related code for window functions - FIRST_VALUE() - LAST_VALUE() - NTH_VALUE() - LEAD() - LAG
| | * MDEV-20278 PERCENTILE_DISC() returns a wrong data typeAlexander Barkov2019-08-071-11/+0
| | |
| | * MDEV-20273 Add class Item_sum_min_maxAlexander Barkov2019-08-071-10/+5
| | |
* | | MDEV-20052 Add a MEM_ROOT pointer argument to Type_handler::make_xxx_field()Alexander Barkov2019-07-121-1/+1
|/ /
* | Merge 10.3 into 10.4Marko Mäkelä2019-06-121-8/+4
|\ \ | |/
| * Merge 10.2 into 10.3Marko Mäkelä2019-06-111-8/+4
| |\
| | * MDEV-18015: Assertion `global_status_var.global_memory_used == 0' failed ↵Varun Gupta2019-06-041-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | when using UDF , window functions and views Adding destructor for Group_bound_tracker to free Cached_item_str. The Cached_item for window functions are allocated on THD:mem_root but the Cached_item_str has value of type string which is allocated on the heap, so we need to call free() for it
* | | MDEV-19124 Assertion `0' failed in Item::val_nativeAlexander Barkov2019-04-021-0/+10
| | |
* | | A cleanup in Item_sum: removing dead codeAlexander Barkov2019-03-281-4/+4
| | |
* | | Merge 10.3 into 10.4Marko Mäkelä2018-10-171-0/+1
|\ \ \ | |/ /
| * | MDEV-17137: Syntax errors with VIEW using MEDIANVarun Gupta2018-10-161-0/+1
| | | | | | | | | | | | | | | | | | | | | The syntax error happened because we had not implemented a different print for percentile functions. The syntax is a bit different when we use percentile functions as window functions in comparision to normal window functions. Implemented a seperate print function for percentile functions
* | | MDEV-17317 Add THD* parameter into Item::get_date() and stricter data type ↵Alexander Barkov2018-09-281-3/+3
|/ / | | | | | | control to "fuzzydate"
* | Merge 10.2 into 10.3Marko Mäkelä2018-08-281-0/+2
|\ \ | |/
| * MDEV-15204: lag/lead function order list mandatoryzhzhzoo2018-08-251-0/+2
| |
* | Merge commit '6b8802e8dd5467556a024d807a1df23940b00895' into bb-10.3-fix_len_decbb-10.3-fix_len_decOleksandr Byelkin2018-06-191-5/+10
|\ \ | |/
| * MDEV-11071: Assertion `thd->transaction.stmt.is_empty()' failed in ↵Oleksandr Byelkin2018-06-151-3/+6
| | | | | | | | | | | | Locked_tables_list::unlock_locked_table fix_length_and_dec now return result (error/OK)
| * MDEV-13358: FIRST_V throw SQL Fehler (1292): Incorrect datetime valueVicențiu Ciorbaru2017-09-191-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is backport of 25ad623d64e for 10.2. The issue is similar to the one from MDEV-13240. Item::save_in_field() returns an error during tmp table population in a create table from select query as we try to save an empty string as a date value when force_return_blank is set to true for window functions. MDEV-13240 Wrong warning with MAX(datetime_field) OVER (...) The problem resided in Item_window_func implementation, and it was revealed by bb-10.2-ext specific changes: Item_window_func::save_in_field() works differently in bb-10.2-ext vs 10.2: - 10.2 goes through val_str() - bb-10.2-ext goes through get_date(), due to Type_handler related changes. get_date() tries to convert empty string to DATETIME, hence the warning. During a discussion with Vicentiu, it was decided to fix Item_window_func::val_xxx() to return NULL (instead of an "empty" value, such as 0 for numbers and '' for strings) when force_return_blank is set.
| * MDEV-12851: Case with window functions query crashes serverVicențiu Ciorbaru2017-06-281-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "is null" function performs one operation which no other Item_func does, which is to update used tables during fix_length_and_dec(). This however can not be performed before window functions have had a chance to resolve their order by and partition by definitions, which happens after the initial setup_fields call. Consequently, do not call Item_func_isnull update_used_tables during fix_length_and_dec(). There was another issue detected once the crash was resolved. Because window functions did not implement is_null() method, we would end up returning bad results for "is null" and "is not null" functions. Implemented is_null() method for Item_windowfunc.
* | Merge bb-10.2-ext into 10.3Marko Mäkelä2018-02-151-0/+24
|\ \
| * | MDEV-15293 CAST(AS TIME) returns bad results for ↵Alexander Barkov2018-02-131-0/+24
| | | | | | | | | | | | LAST_VALUE(),NAME_CONST(),SP variable
* | | Remove not used mem_root argument from build_clone(), get_copy() and ↵Michael Widenius2017-11-231-27/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | get_item_copy() TODO: - Make get_thd_memroot() inline - To do this, we need to reduce dependence of include files, especially so that sql_class.h is not depending in item.h
* | | Updates the tests for the percentile functionsVarun Gupta2017-11-011-7/+7
| | |
* | | Update the error messages involving percentile functionsVarun Gupta2017-11-011-4/+4
| | |
* | | Changes made according to the review given, mostly fixing coding style errorsVarun Gupta2017-11-011-8/+10
| | |
* | | Remove has_error as a member from Item_sum and use THD::is_error() insteadVicențiu Ciorbaru2017-11-011-6/+0
| | | | | | | | | | | | | | | Additionally, allow a query with window functions to be killed by the user during its execution.
* | | made changes according to the review, mostly removing unused code and fixing ↵Varun Gupta2017-11-011-28/+6
| | | | | | | | | | | | code to follow the coding conventions
* | | Added the errorVarun Gupta2017-11-011-32/+20
| | | | | | | | | | | | | | | | | | 1)ER_ARGUMENT_OUT_OF_RANGE: This error is thrown if the argument of the percentile function is not in the range [0,1] 2)ER_ARGUMENT_NOT_CONSTANT: This error is thrown if the argument of the percnetile function is not constant in the entire partition of the window function
* | | Added the function setting_handler_for_percentile_function() for the ↵Varun Gupta2017-11-011-0/+15
| | | | | | | | | | | | | | | | | | percentile_disc function that would set the type of the result field for percentile_disc. Percentile_cont would habe double precision result type
* | | Val_str function added for the percentile_disc function, as it can have ↵Varun Gupta2017-11-011-0/+11
| | | | | | | | | | | | result type as STRING_RESULT
* | | Percentile class implemented, most of the functions have the same ↵Varun Gupta2017-11-011-2/+159
| | | | | | | | | | | | functionalite as the percentile cont class
* | | Implemented the implementation of percentile functions using Item_cache ↵Varun Gupta2017-11-011-45/+37
| | | | | | | | | | | | instead of Cache_Item
* | | implementation of add() function added to the Item_sum_percentile_disc classVarun Gupta2017-11-011-8/+56
| | |
* | | Setting handler to have the return type as that of the element by which we ↵Varun Gupta2017-11-011-4/+8
| | | | | | | | | | | | are ordering the partition
* | | A basic implementation of the add function is addedVarun Gupta2017-11-011-4/+31
| | |
* | | Added value field to Item_sum_percentile_discVarun Gupta2017-11-011-1/+28
| | | | | | | | | | | | Check for single element in the order_list is added
* | | MDEV-12985: support percentile and median window functionsVarun Gupta2017-11-011-0/+69
| | | | | | | | | | | | Finalised the synatax and have started implementing the class for the PERCENTILE_DISC
* | | Enusure that my_global.h is included firstMichael Widenius2017-08-241-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added sql/mariadb.h file that should be included first by files in sql directory, if sql_plugin.h is not used (sql_plugin.h adds SHOW variables that must be done before my_global.h is included) - Removed a lot of include my_global.h from include files - Removed include's of some files that my_global.h automatically includes - Removed duplicated include's of my_sys.h - Replaced include my_config.h with my_global.h
* | | Added copyright message to some filesMichael Widenius2017-08-241-0/+16
|/ /
* | MDEV-13240 Wrong warning with MAX(datetime_field) OVER (...)Alexander Barkov2017-07-041-8/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The problem resided in Item_window_func implementation, and it was revealed by bb-10.2-ext specific changes: Item_window_func::save_in_field() works differently in bb-10.2-ext vs 10.2: - 10.2 goes through val_str() - bb-10.2-ext goes through get_date(), due to Type_handler related changes. get_date() tries to convert empty string to DATETIME, hence the warning. During a discussion with Vicentiu, it was decided to fix Item_window_func::val_xxx() to return NULL (instead of an "empty" value, such as 0 for numbers and '' for strings) when force_return_blank is set.
* | MDEV-12719 Determine Item::result_type() from Item::type_handler()Alexander Barkov2017-05-071-5/+0
| |
* | MDEV-12718 Determine Item::cmp_type() from Item::type_handler()Alexander Barkov2017-05-071-2/+0
| |
* | MDEV-12714 Determine Item::field_type() from Item::type_handler()Alexander Barkov2017-05-061-9/+0
| |
* | MDEV-12713 Define virtual type_handler() for all Item classesAlexander Barkov2017-05-061-0/+9
| |
* | Cleanup: changing set_handler_by_field_type(field_type()) to ↵Alexander Barkov2017-05-061-2/+2
| | | | | | | | | | | | set_handler(type_handler()). This gives better performance, as excludes handler-by-type lookup.
* | Changing field::field_name and Item::name to LEX_CSTRINGMonty2017-04-231-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Benefits of this patch: - Removed a lot of calls to strlen(), especially for field_string - Strings generated by parser are now const strings, less chance of accidently changing a string - Removed a lot of calls with LEX_STRING as parameter (changed to pointer) - More uniform code - Item::name_length was not kept up to date. Now fixed - Several bugs found and fixed (Access to null pointers, access of freed memory, wrong arguments to printf like functions) - Removed a lot of casts from (const char*) to (char*) Changes: - This caused some ABI changes - lex_string_set now uses LEX_CSTRING - Some fucntions are now taking const char* instead of char* - Create_field::change and after changed to LEX_CSTRING - handler::connect_string, comment and engine_name() changed to LEX_CSTRING - Checked printf() related calls to find bugs. Found and fixed several errors in old code. - A lot of changes from LEX_STRING to LEX_CSTRING, especially related to parsing and events. - Some changes from LEX_STRING and LEX_STRING & to LEX_CSTRING* - Some changes for char* to const char* - Added printf argument checking for my_snprintf() - Introduced null_clex_str, star_clex_string, temp_lex_str to simplify code - Added item_empty_name and item_used_name to be able to distingush between items that was given an empty name and items that was not given a name This is used in sql_yacc.yy to know when to give an item a name. - select table_name."*' is not anymore same as table_name.* - removed not used function Item::rename() - Added comparision of item->name_length before some calls to my_strcasecmp() to speed up comparison - Moved Item_sp_variable::make_field() from item.h to item.cc - Some minimal code changes to avoid copying to const char * - Fixed wrong error message in wsrep_mysql_parse() - Fixed wrong code in find_field_in_natural_join() where real_item() was set when it shouldn't - ER_ERROR_ON_RENAME was used with extra arguments. - Removed some (wrong) ER_OUTOFMEMORY, as alloc_root will already give the error. TODO: - Check possible unsafe casts in plugin/auth_examples/qa_auth_interface.c - Change code to not modify LEX_CSTRING for database name (as part of lower_case_table_names)
* | MDEV-12338 Split Item_type_holder::get_real_type() into virtual ↵Alexander Barkov2017-03-241-2/+0
|/ | | | Item::real_type_handler()
* MDEV-11746: Wrong result upon using FIRST_VALUE with a window frameVicențiu Ciorbaru2017-02-141-12/+1
| | | | | Reimplement FIRST_VALUE to act as NTH_VALUE with 0 offset. The previous implementation was flawed when the window frame would remove values.
View Lorry Controller Status