diff options
| author | malff/marcsql@weblab.(none) <> | 2007-01-11 14:58:05 -0700 |
|---|---|---|
| committer | malff/marcsql@weblab.(none) <> | 2007-01-11 14:58:05 -0700 |
| commit | 9655f5fb0330d9eb67effe6755dec45f28f1e446 (patch) | |
| tree | 53f7cc34c8a22ed9ceaf16d39f046e08a1c24778 /sql/item_sum.h | |
| parent | c55862698ebc7aafdfeeba47170c9f7b89b66eec (diff) | |
| download | mariadb-git-9655f5fb0330d9eb67effe6755dec45f28f1e446.tar.gz | |
Bug#22687 (Functions UNIQUE_USERS, GROUP_UNIQUE_USERS)
According to some internal communication, these two functions are place
holders for future enhancements. Because they use a variable number of
parameters, the implementation defined a reserved keyword for them in the
parser grammar.
Unfortunately, doing so creates a bug similar to Bug 21114 reported for the
function FORMAT.
In the 5.1 code base, due to improvements in the code implemented with bug
21114, having a reserved keyword for functions with a variable number of
arguments is not needed any more by the implementation.
As a result, this fix removes the place-holder implementation, and removes
the unnecessary reserved keywords. Should the functions UNIQUE_USERS and
GROUP_UNIQUE_USERS be finally implemented in a later release, the
implementation should sub class Create_native_func in sql/item_create.cc.
For example, see the class Create_func_concat.
Diffstat (limited to 'sql/item_sum.h')
| -rw-r--r-- | sql/item_sum.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/item_sum.h b/sql/item_sum.h index acd23d1ff4c..33bd23602b2 100644 --- a/sql/item_sum.h +++ b/sql/item_sum.h @@ -223,7 +223,7 @@ class Item_sum :public Item_result_field public: enum Sumfunctype { COUNT_FUNC, COUNT_DISTINCT_FUNC, SUM_FUNC, SUM_DISTINCT_FUNC, AVG_FUNC, - AVG_DISTINCT_FUNC, MIN_FUNC, MAX_FUNC, UNIQUE_USERS_FUNC, STD_FUNC, + AVG_DISTINCT_FUNC, MIN_FUNC, MAX_FUNC, STD_FUNC, VARIANCE_FUNC, SUM_BIT_FUNC, UDF_SUM_FUNC, GROUP_CONCAT_FUNC }; |